On Fri, Feb 17, 2017 at 2:26 PM, Aníbal Limón <[email protected]>
wrote:
> + parser.add_argument('-l', '--layer', metavar='LAYER_DIR',
> + help='Layer to test compatibility with Yocto Project',
> + action='store', required=True)
>
The behavior of this argument is quite unintuitive. First, this path isn’t
actually used as the layer directory, detect_layers searches down from that
path to *find* layers in that subdirectory, so the argument isn’t what it
says it is. Second, why are you making an option mandatory rather than just
using arguments?
That is, why `layer-compat-layer.py -l layerpath` rather than
`layer-compat-layer.py layerpath`. If you change the call to
`parser.add_argument(‘layer’, …)` it’d change to the latter behavior.
--
Christopher Larson
kergoth at gmail dot com
Founder - BitBake, OpenEmbedded, OpenZaurus
Senior Software Engineer, Mentor Graphics
--
_______________________________________________
yocto mailing list
[email protected]
https://lists.yoctoproject.org/listinfo/yocto
