On 27 January 2014 12:44, Paul Barker <[email protected]> wrote:
> The case statement for ARGS_OPT_ADD_EXCLUDE added to the argument handling
> switch statement in opkg was missing a "break;" at the end, so it was falling
> through into the handler for ARGS_OPT_NOACTION. Thus when "--add-exclude" was
> specified on the command line it was as if "--noaction" was also being
> specified. This appears to be the root cause of YP bug 5311.
>
> This patch should fix that behaviour and that bug.
>
> Signed-off-by: Paul Barker <[email protected]>
> Cc: Mark Hatle <[email protected]>
> Cc: Saul Wold <[email protected]>

I should have checked this more myself before sending - I didn't run
'rediff' on the patch after manually editing it so it failed to apply
cleanly. I'll update it and send v2.

I'm still not sure how best to test this one myself but it would be
good to see it tested and fix the relevant bug. Once that's done I can
look at upstreaming this feature into opkg.

-- 
Paul Barker

Email: [email protected]
http://www.paulbarker.me.uk
_______________________________________________
Openembedded-core mailing list
[email protected]
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Reply via email to