[yocto] [PATCH 0/5][eclipse-poky] Refactor handling of SDK check errors
Timo Müller
mail at timomueller.eu
Fri Mar 1 01:37:07 PST 2013
Hi Jessica,
Zhang, Jessica wrote, On 27.02.2013 23:04:
> Hi Timo,
>
> Thanks for initiating the cleanup in this area that I agree
> definitely need some work. There're couple issues I'm seeing with
> your patch set that probably need some refinements:
>
> 1. After seeing the error message, if I go into the field correct the
> error, I have to explicit click on "Apply" to make it revalidate,
> probably we should track the fields change and do automatic
> re-validation
Yes, I agree it can be confusing. I already played around with enabling
Listeners on the YoctoUISetting element but stopped because I thought
that rechecking the SDK everytime you type in a new character will slow
down the ui to much.
One idea I have in mind is, if the settings are changed (on type) we
delete the error message and show a message telling the user that the
new settings have to be revalidated again. Do you think that would be
sufficient?
Nevertheless I will extend the YoctoUISetting to be able to notify
registered listeners on change. Then we can decide on how we want to
react to this change.
>
> 2. For a scenario that I have 2 toolchains (for different arch, e.g.
> arm and x86) installed under same directory, which means there're 2
> environment setup scripts under the same directory. In this case,
> the Target Architecture drop down list will contain 2 entries for the
> arches. In this case, I'm getting an error message for missing
> environment script file which is not true. By looking at the code,
> the logic basically checking the elem's target architecture setting,
> if there's none, report missing environment script. I think for this
> case, we do need to check the size of the target arch option list to
> see whether it's empty or not.
True, this calls for a separate error message.
>
> 3. For the case of Poky.Env.Script.Nonexist, there's
> Poky.Env.ScriptNoexist.Advise which is more meaningful to user what
> to do if running into the error. But I don't see that Advice is
> posted and by looking at code, it seems it's not being used as well?
I'll try to rephrase the error message.
Regarding the advice I'm not quite sure if I follow correctly. If you
choose "Build system derived toolchain" and the location doesn't contain
a toolchain (so the drop-down is empty) the SDK check fails with
ENV_SETUP_SCRIPT_NONEXIST. If a dialog pops up showing the error, the
advice will also be used and added to the message.
>
> Thanks, Jessica
>
> -----Original Message----- From: yocto-bounces at yoctoproject.org
> [mailto:yocto-bounces at yoctoproject.org] On Behalf Of Timo Mueller
> Sent: Wednesday, February 27, 2013 6:37 AM To:
> yocto at yoctoproject.org Cc: Timo Mueller Subject: [yocto] [PATCH
> 0/5][eclipse-poky] Refactor handling of SDK check errors
>
> From: Timo Mueller <timo.mueller at bmw-carit.de>
>
> Hi,
>
> I tried to display the messages resulting from validating the
> YoctoUISettings in the message area of the properties/preference
> pages instead of showing a dialog. This led to some refactoring of
> the validation functionality and also the messages.
>
> If for example the user entered a SDK toolchain directory that
> doesn't exist in the global preferences a dialog will show up showing
> the error and telling him how he can do "IDE-wide settings". As he is
> currently trying to do "IDE-wide settings" this advice is not really
> necessary and the dialog can be really annoying.
>
> I split up the messages into a one line error message and an advice.
> This message can be shown in the page's message area instead of
> opening a dialog with every configuration error. The advice can still
> be used when we want to show a dialog to help the user, e.g. when
> creating a new project.
>
> On a side note I reactivated the YoctoSDKChecker class and moved all
> the validation code from the utils, getting rid of some duplicated
> and inconsistent code.
>
> Best regards, Timo
>
> PS: This patch set is based on "plugins/sdk.ide: Remove profile edit
> buttons from property page"
Best regards
Timo
More information about the yocto
mailing list