[yocto] [PATCH 0/3] [eclipse-poky] Eclipse plugin on Windows - fixes for progress update on wizards & input validation

[Grigoropol, IoanaX]

Hi Jessica,

I have apparently associated my Execution Environment 1.6 with a jre 1.7. I have sent a patch for the syntax error.
As for the error shown for the existing repository, I have sent a patch to validate the entries when switching between an existing repo and a new clone. 
I think that we should perform some kind of validation on the existing repo (oe-build-env-init script exists, etc.) from the very creation of the project otherwise at some point things will stop working with bitbake and the user will not know why. Indeed, check for an existing git directory does not fall into this category and should be removed. 

Thanks,
Ioana


-----Original Message-----
From: Zhang, Jessica 
Sent: Wednesday, December 05, 2012 12:55 AM
To: Grigoropol, IoanaX; yocto at yoctoproject.org
Subject: RE: [yocto] [PATCH 0/3] [eclipse-poky] Eclipse plugin on Windows - fixes for progress update on wizards & input validation

Hi Ioana,

There's issue as to syntax for YoctoHostShellProcessAdaptor since it's only supported by JDK version 1.7 and above.  Also, when I tried the plugin on Linux, it still failed for creating a project for an existing metadata directory.   With a not very clear error message " Directory /home/jzhang/poky contains a repository, please select validate repository" which is what I did.  It seems we are calling out the git repository for bitbake commander project, which I don't think there's the need. I think we should preserve the existing interface that only provide user the option to clone a new one if there's the need.  Otherwise, we should just validate the project repo behind the scene.  And we shouldn't make the clone a new one as the default as the current implementation.

Thanks,
Jessica

-----Original Message-----
From: yocto-bounces at yoctoproject.org [mailto:yocto-bounces at yoctoproject.org] On Behalf Of Ioana Grigoropol
Sent: Tuesday, December 04, 2012 5:26 AM
To: yocto at yoctoproject.org
Subject: [yocto] [PATCH 0/3] [eclipse-poky] Eclipse plugin on Windows - fixes for progress update on wizards & input validation

- Resending patches with proper header for eclipse-poky
- Include fixes for showing progress information for long time running tasks on wizards for Bitbake Commander project & Bitbake recipe
- Includes validation for the location & name of a new Bitbake Commander in a new repo clone or an existing one
 
Ioana Grigoropol (3):
  Show progress bar for New Bitbake Commander project
  Show progress bar for building new Recipe in Bitbake commander
    perspective
  Validate project name and location for Bitbake command project

 .../src/org/yocto/bc/bitbake/ShellSession.java     |  116 +++-----
 .../yocto/bc/remote/utils/ProcessStreamBuffer.java |    3 +-
 .../org/yocto/bc/remote/utils/RemoteHelper.java    |  155 +++++------
 .../org/yocto/bc/remote/utils/RemoteMachine.java   |    4 +-
 .../remote/utils/YoctoHostShellProcessAdapter.java |  136 +++++-----  .../bc/remote/utils/YoctoRunnableWithProgress.java |  106 ++++++++
 .../bc/ui/wizards/NewBitBakeFileRecipeWizard.java  |    2 +-
 .../ui/wizards/NewBitBakeFileRecipeWizardPage.java |  285 ++++++++++----------  .../yocto/bc/ui/wizards/install/InstallWizard.java |  113 ++------
 .../yocto/bc/ui/wizards/install/OptionsPage.java   |  178 ++++++------
 .../BBConfigurationInitializeOperation.java        |    6 +-
 .../newproject/CreateBBCProjectOperation.java      |   31 +--
 12 files changed, 551 insertions(+), 584 deletions(-)  create mode 100644 plugins/org.yocto.bc.ui/src/org/yocto/bc/remote/utils/YoctoRunnableWithProgress.java

--
1.7.9.5

_______________________________________________
yocto mailing list
yocto at yoctoproject.org
https://lists.yoctoproject.org/listinfo/yocto