On Thu, 2020-12-10 at 16:20 +0000, Ross Burton wrote: > Currently the argument parser uses nargs=+ for both the layers > (positional arguments) and machines/dependencies/addition layers > (optional arguments). This means it's impossible to determine what is > meant by: > > $ yocto-check-layer --machines a b c > > Is this machines=(a,b) and layers=(c), or machines=(a) and layers=(b,c)? > Or even machines=(a,b,c) and layers=(), which results in a parse failure > as the layers are mandatory? > > The workaround is to re-order the arguments to an unambiguous form: > > $ yocto-check-layer b c --machines a > > However this means the user needs to know the problem and how to work > around it. Instead, change the parser to require an explicit --layers > argument: > > $ yocto-check-layer --machines a --layers b c > > Also improve the other arguments by adding clearer metavars and short > options. > > [ YOCTO #14148 ] > > Signed-off-by: Ross Burton <ross.bur...@arm.com> > --- > scripts/yocto-check-layer | 10 +++++----- > 1 file changed, 5 insertions(+), 5 deletions(-)
https://autobuilder.yoctoproject.org/typhoon/#/builders/39/builds/2825 Do we need to update the way we're calling it on the autobuilder? Cheers, Richard
-=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#145558): https://lists.openembedded.org/g/openembedded-core/message/145558 Mute This Topic: https://lists.openembedded.org/mt/78857660/21656 Group Owner: openembedded-core+ow...@lists.openembedded.org Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-