* [PATCH][eclipse-poky] plugin/sdk.ide: Pre-populate project specific profile form [not found] <cover.1372173542.git.timo.mueller@bmw-carit.de> @ 2013-06-25 15:32 ` Timo Mueller 2013-06-25 21:52 ` Zhang, Jessica 0 siblings, 1 reply; 3+ messages in thread From: Timo Mueller @ 2013-06-25 15:32 UTC (permalink / raw) To: yocto; +Cc: Timo Mueller From: Timo Mueller <timo.mueller@bmw-carit.de> On initial setup the profile form is populated with the values from the previously selected target profile. This allows the user to quickly make project specific adaptions of an existing profile. If a project specific profile is already defined the form will show the values from this configuration. Signed-off-by: Timo Mueller <timo.mueller@bmw-carit.de> --- .../sdk/ide/preferences/YoctoSDKProjectPropertyPage.java | 14 ++++++++++++-- 1 file changed, 12 insertions(+), 2 deletions(-) diff --git a/plugins/org.yocto.sdk.ide/src/org/yocto/sdk/ide/preferences/YoctoSDKProjectPropertyPage.java b/plugins/org.yocto.sdk.ide/src/org/yocto/sdk/ide/preferences/YoctoSDKProjectPropertyPage.java index 9f99caf..408ce76 100644 --- a/plugins/org.yocto.sdk.ide/src/org/yocto/sdk/ide/preferences/YoctoSDKProjectPropertyPage.java +++ b/plugins/org.yocto.sdk.ide/src/org/yocto/sdk/ide/preferences/YoctoSDKProjectPropertyPage.java @@ -31,14 +31,15 @@ import org.osgi.service.prefs.BackingStoreException; import org.yocto.sdk.ide.YoctoProfileElement; import org.yocto.sdk.ide.YoctoProfileSetting; import org.yocto.sdk.ide.YoctoProjectSpecificSetting; +import org.yocto.sdk.ide.YoctoSDKChecker; import org.yocto.sdk.ide.YoctoSDKChecker.SDKCheckRequestFrom; import org.yocto.sdk.ide.YoctoSDKChecker.SDKCheckResults; import org.yocto.sdk.ide.YoctoSDKMessages; -import org.yocto.sdk.ide.utils.YoctoSDKUtils; -import org.yocto.sdk.ide.utils.YoctoSDKUtilsConstants; import org.yocto.sdk.ide.YoctoSDKPlugin; import org.yocto.sdk.ide.YoctoUIElement; import org.yocto.sdk.ide.YoctoUISetting; +import org.yocto.sdk.ide.utils.YoctoSDKUtils; +import org.yocto.sdk.ide.utils.YoctoSDKUtilsConstants; public class YoctoSDKProjectPropertyPage extends PropertyPage implements IWorkbenchPropertyPage { @@ -288,6 +289,15 @@ public class YoctoSDKProjectPropertyPage extends PropertyPage implements public void switchToProjectSpecificProfile() { YoctoUIElement profileElement = getElemFromProjectPreferences(getProject()); + SDKCheckResults result = YoctoSDKChecker.checkYoctoSDK(profileElement); + + if ((result != SDKCheckResults.SDK_PASS)) { + /* Project specific profile has not yet been defined, + * leave settings from previously selected profile + */ + return; + } + yoctoUISetting.setCurrentInput(profileElement); } -- 1.8.1.4 ^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH][eclipse-poky] plugin/sdk.ide: Pre-populate project specific profile form 2013-06-25 15:32 ` [PATCH][eclipse-poky] plugin/sdk.ide: Pre-populate project specific profile form Timo Mueller @ 2013-06-25 21:52 ` Zhang, Jessica 2013-06-26 6:46 ` Timo Müller 0 siblings, 1 reply; 3+ messages in thread From: Zhang, Jessica @ 2013-06-25 21:52 UTC (permalink / raw) To: Timo Mueller, yocto@yoctoproject.org; +Cc: Timo Mueller Hi Timo, How am I supposed to apply this patch? I tried to apply it against master first, then the patch2 of your v3 patch series won't apply. If I apply your v3 patch series first, then this patch won't apply after that... Thanks, Jessica -----Original Message----- From: yocto-bounces@yoctoproject.org [mailto:yocto-bounces@yoctoproject.org] On Behalf Of Timo Mueller Sent: Tuesday, June 25, 2013 8:33 AM To: yocto@yoctoproject.org Cc: Timo Mueller Subject: [yocto] [PATCH][eclipse-poky] plugin/sdk.ide: Pre-populate project specific profile form From: Timo Mueller <timo.mueller@bmw-carit.de> On initial setup the profile form is populated with the values from the previously selected target profile. This allows the user to quickly make project specific adaptions of an existing profile. If a project specific profile is already defined the form will show the values from this configuration. Signed-off-by: Timo Mueller <timo.mueller@bmw-carit.de> --- .../sdk/ide/preferences/YoctoSDKProjectPropertyPage.java | 14 ++++++++++++-- 1 file changed, 12 insertions(+), 2 deletions(-) diff --git a/plugins/org.yocto.sdk.ide/src/org/yocto/sdk/ide/preferences/YoctoSDKProjectPropertyPage.java b/plugins/org.yocto.sdk.ide/src/org/yocto/sdk/ide/preferences/YoctoSDKProjectPropertyPage.java index 9f99caf..408ce76 100644 --- a/plugins/org.yocto.sdk.ide/src/org/yocto/sdk/ide/preferences/YoctoSDKProjectPropertyPage.java +++ b/plugins/org.yocto.sdk.ide/src/org/yocto/sdk/ide/preferences/YoctoS +++ DKProjectPropertyPage.java @@ -31,14 +31,15 @@ import org.osgi.service.prefs.BackingStoreException; import org.yocto.sdk.ide.YoctoProfileElement; import org.yocto.sdk.ide.YoctoProfileSetting; import org.yocto.sdk.ide.YoctoProjectSpecificSetting; +import org.yocto.sdk.ide.YoctoSDKChecker; import org.yocto.sdk.ide.YoctoSDKChecker.SDKCheckRequestFrom; import org.yocto.sdk.ide.YoctoSDKChecker.SDKCheckResults; import org.yocto.sdk.ide.YoctoSDKMessages; -import org.yocto.sdk.ide.utils.YoctoSDKUtils; -import org.yocto.sdk.ide.utils.YoctoSDKUtilsConstants; import org.yocto.sdk.ide.YoctoSDKPlugin; import org.yocto.sdk.ide.YoctoUIElement; import org.yocto.sdk.ide.YoctoUISetting; +import org.yocto.sdk.ide.utils.YoctoSDKUtils; +import org.yocto.sdk.ide.utils.YoctoSDKUtilsConstants; public class YoctoSDKProjectPropertyPage extends PropertyPage implements IWorkbenchPropertyPage { @@ -288,6 +289,15 @@ public class YoctoSDKProjectPropertyPage extends PropertyPage implements public void switchToProjectSpecificProfile() { YoctoUIElement profileElement = getElemFromProjectPreferences(getProject()); + SDKCheckResults result = +YoctoSDKChecker.checkYoctoSDK(profileElement); + + if ((result != SDKCheckResults.SDK_PASS)) { + /* Project specific profile has not yet been defined, + * leave settings from previously selected profile + */ + return; + } + yoctoUISetting.setCurrentInput(profileElement); } -- 1.8.1.4 _______________________________________________ yocto mailing list yocto@yoctoproject.org https://lists.yoctoproject.org/listinfo/yocto ^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH][eclipse-poky] plugin/sdk.ide: Pre-populate project specific profile form 2013-06-25 21:52 ` Zhang, Jessica @ 2013-06-26 6:46 ` Timo Müller 0 siblings, 0 replies; 3+ messages in thread From: Timo Müller @ 2013-06-26 6:46 UTC (permalink / raw) To: Zhang, Jessica; +Cc: yocto@yoctoproject.org, Timo Mueller Hi Jessica, Zhang, Jessica wrote, On 25.06.2013 23:52: > Hi Timo, > > How am I supposed to apply this patch? I tried to apply it against master first, then the patch2 of your v3 patch series won't apply. If I apply your v3 patch series first, then this patch won't apply after that... I've messed up with the prefix, that's why it didn't apply. Sorry about that. I'll resend the patch that will apply to the master. > > Thanks, > Jessica > > -----Original Message----- > From: yocto-bounces@yoctoproject.org [mailto:yocto-bounces@yoctoproject.org] On Behalf Of Timo Mueller > Sent: Tuesday, June 25, 2013 8:33 AM > To: yocto@yoctoproject.org > Cc: Timo Mueller > Subject: [yocto] [PATCH][eclipse-poky] plugin/sdk.ide: Pre-populate project specific profile form > > From: Timo Mueller <timo.mueller@bmw-carit.de> > > On initial setup the profile form is populated with the values from the previously selected target profile. This allows the user to quickly make project specific adaptions of an existing profile. > > If a project specific profile is already defined the form will show the values from this configuration. > > Signed-off-by: Timo Mueller <timo.mueller@bmw-carit.de> > --- > .../sdk/ide/preferences/YoctoSDKProjectPropertyPage.java | 14 ++++++++++++-- > 1 file changed, 12 insertions(+), 2 deletions(-) > > diff --git a/plugins/org.yocto.sdk.ide/src/org/yocto/sdk/ide/preferences/YoctoSDKProjectPropertyPage.java b/plugins/org.yocto.sdk.ide/src/org/yocto/sdk/ide/preferences/YoctoSDKProjectPropertyPage.java > index 9f99caf..408ce76 100644 > --- a/plugins/org.yocto.sdk.ide/src/org/yocto/sdk/ide/preferences/YoctoSDKProjectPropertyPage.java > +++ b/plugins/org.yocto.sdk.ide/src/org/yocto/sdk/ide/preferences/YoctoS > +++ DKProjectPropertyPage.java > @@ -31,14 +31,15 @@ import org.osgi.service.prefs.BackingStoreException; > import org.yocto.sdk.ide.YoctoProfileElement; > import org.yocto.sdk.ide.YoctoProfileSetting; > import org.yocto.sdk.ide.YoctoProjectSpecificSetting; > +import org.yocto.sdk.ide.YoctoSDKChecker; > import org.yocto.sdk.ide.YoctoSDKChecker.SDKCheckRequestFrom; > import org.yocto.sdk.ide.YoctoSDKChecker.SDKCheckResults; > import org.yocto.sdk.ide.YoctoSDKMessages; > -import org.yocto.sdk.ide.utils.YoctoSDKUtils; > -import org.yocto.sdk.ide.utils.YoctoSDKUtilsConstants; > import org.yocto.sdk.ide.YoctoSDKPlugin; import org.yocto.sdk.ide.YoctoUIElement; import org.yocto.sdk.ide.YoctoUISetting; > +import org.yocto.sdk.ide.utils.YoctoSDKUtils; > +import org.yocto.sdk.ide.utils.YoctoSDKUtilsConstants; > > public class YoctoSDKProjectPropertyPage extends PropertyPage implements > IWorkbenchPropertyPage { > @@ -288,6 +289,15 @@ public class YoctoSDKProjectPropertyPage extends PropertyPage implements > public void switchToProjectSpecificProfile() > { > YoctoUIElement profileElement = getElemFromProjectPreferences(getProject()); > + SDKCheckResults result = > +YoctoSDKChecker.checkYoctoSDK(profileElement); > + > + if ((result != SDKCheckResults.SDK_PASS)) { > + /* Project specific profile has not yet been defined, > + * leave settings from previously selected profile > + */ > + return; > + } > + > yoctoUISetting.setCurrentInput(profileElement); > } > > -- > 1.8.1.4 > > _______________________________________________ > yocto mailing list > yocto@yoctoproject.org > https://lists.yoctoproject.org/listinfo/yocto > If this is merged, the "Add target profile quick switch" patch series has to be adapted slightly. I'll send a rework of the series in that case. Best regards, Timo ^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2013-06-26 6:47 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <cover.1372173542.git.timo.mueller@bmw-carit.de>
2013-06-25 15:32 ` [PATCH][eclipse-poky] plugin/sdk.ide: Pre-populate project specific profile form Timo Mueller
2013-06-25 21:52 ` Zhang, Jessica
2013-06-26 6:46 ` Timo Müller
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.