[OE-core] [PATCH] yocto-compat-layer.py: Add script to YP Compatible Layer validation

[Christopher Larson]

On Fri, Feb 17, 2017 at 2:26 PM, Aníbal Limón <anibal.limon at linux.intel.com>
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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openembedded.org/pipermail/openembedded-core/attachments/20170217/29d82557/attachment-0002.html>