<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Feb 17, 2017 at 2:26 PM, Aníbal Limón <span dir="ltr"><<a href="mailto:anibal.limon@linux.intel.com" target="_blank">anibal.limon@linux.intel.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div id=":dp" class="a3s aXjCH m15a4df4b55dec0a4">+ parser.add_argument('-l', '--layer', metavar='LAYER_DIR',<br>
+ help='Layer to test compatibility with Yocto Project',<br>
+ action='store', required=True)</div></blockquote></div><br>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?</div><div class="gmail_extra"><br></div><div class="gmail_extra">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.<br>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr">Christopher Larson<br>kergoth at gmail dot com<br>Founder - BitBake, OpenEmbedded, OpenZaurus<br>Senior Software Engineer, Mentor Graphics</div></div></div></div>
</div></div>