<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Sun, Oct 9, 2016 at 4:44 AM, Richard Purdie <span dir="ltr"><<a href="mailto:richard.purdie@linuxfoundation.org" target="_blank">richard.purdie@linuxfoundation.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Thu, 2016-10-06 at 20:09 -0700, Stephano Cetola wrote:<br>
><br>
</span><span class="">> When using subprocess call and check_output, it is better to use<br>
> arrays<br>
> rather than strings when possible to avoid whitespace and quoting<br>
> problems.<br>
><br>
> [ YOCTO #9342 ]<br>
><br>
> Signed-off-by: Stephano Cetola <<a href="mailto:stephano.cetola@linux.intel.com">stephano.cetola@linux.intel.<wbr>com</a>><br>
> ---<br>
> meta/lib/oe/distro_check.py <wbr> | 2 +-<br>
> meta/lib/oe/package.py <wbr> | 13 +--<br>
> meta/lib/oe/package_manager.<wbr>py | 218 ++++++++++++++++++++----------<wbr>-<br>
> ----------<br>
> 3 files changed, 114 insertions(+), 119 deletions(-)<br>
</span><span class="">This triggered a lot of errors on the autobuilder:<br>
<br>
<a href="http://autobuilder.yocto.io:8010/builders/build-appliance/builds/70" rel="noreferrer" target="_blank">http://autobuilder.yocto.io:<wbr>8010/builders/build-appliance/<wbr>builds/70</a><br>
<a href="http://autobuilder.yocto.io:8010/builders/nightly-x86/builds/75" rel="noreferrer" target="_blank">http://autobuilder.yocto.io:<wbr>8010/builders/nightly-x86/<wbr>builds/75</a><br>
<a href="http://autobuilder.yocto.io:8010/builders/nightly-x86-64/builds/73" rel="noreferrer" target="_blank">http://autobuilder.yocto.io:<wbr>8010/builders/nightly-x86-64/<wbr>builds/73</a><br>
<a href="http://autobuilder.yocto.io:8010/builders/nightly-x86-64-lsb/builds/72" rel="noreferrer" target="_blank">http://autobuilder.yocto.io:<wbr>8010/builders/nightly-x86-64-<wbr>lsb/builds/72</a><br>
<a href="http://autobuilder.yocto.io:8010/builders/nightly-x86-lsb/builds/73" rel="noreferrer" target="_blank">http://autobuilder.yocto.io:<wbr>8010/builders/nightly-x86-lsb/<wbr>builds/73</a><br>
<a href="http://autobuilder.yocto.io:8010/builders/nightly-wic/builds/73" rel="noreferrer" target="_blank">http://autobuilder.yocto.io:<wbr>8010/builders/nightly-wic/<wbr>builds/73</a><br>
</span><a href="http://autobuilder.yocto.io:8010/builders/nightly-qa-extras/builds/55" rel="noreferrer" target="_blank">http://autobuilder.yocto.io:<wbr>8010/builders/nightly-qa-<wbr>extras/builds/55</a><br>
<a href="http://autobuilder.yocto.io:8010/builders/nightly-oe-selftest/builds/70" rel="noreferrer" target="_blank">http://autobuilder.yocto.io:<wbr>8010/builders/nightly-oe-<wbr>selftest/builds/70</a><br>
<a href="http://autobuilder.yocto.io:8010/builders/nightly-multilib/builds/78" rel="noreferrer" target="_blank">http://autobuilder.yocto.io:<wbr>8010/builders/nightly-<wbr>multilib/builds/78</a><br>
<br>
and similar errors on the main AB. I've confirmed it is this patch<br>
which causes the issue. Presumably some errors are occurring but are<br>
currently silently being ignored?<br>
<br>
Its probably worth looking into what is going on in case there is some<br>
real issue here but at this point given the complexity of the changes<br>
I'm leaning towwards deferring this for 2.3.</blockquote></div><br>I’m a bit curious about why this was queued for 2.2 anyway, given it’s not a clear bugfix. Are we not past the feature freeze date?<br>-- <br><div class="gmail_signature" data-smartmail="gmail_signature">Christopher Larson<br>clarson at kergoth dot com<br>Founder - BitBake, OpenEmbedded, OpenZaurus<br>Maintainer - Tslib<br>Senior Software Engineer, Mentor Graphics</div>
</div></div>