On 10/09, Richard Purdie wrote: > On Thu, 2016-10-06 at 20:09 -0700, Stephano Cetola wrote: > > > > When using subprocess call and check_output, it is better to use > > arrays > > rather than strings when possible to avoid whitespace and quoting > > problems. > > > > [ YOCTO #9342 ] > > > > Signed-off-by: Stephano Cetola <stephano.cet...@linux.intel.com> > > --- > > meta/lib/oe/distro_check.py | 2 +- > > meta/lib/oe/package.py | 13 +-- > > meta/lib/oe/package_manager.py | 218 ++++++++++++++++++++----------- > > ---------- > > 3 files changed, 114 insertions(+), 119 deletions(-) > This triggered a lot of errors on the autobuilder: > > http://autobuilder.yocto.io:8010/builders/build-appliance/builds/70 > http://autobuilder.yocto.io:8010/builders/nightly-x86/builds/75 > http://autobuilder.yocto.io:8010/builders/nightly-x86-64/builds/73 > http://autobuilder.yocto.io:8010/builders/nightly-x86-64-lsb/builds/72 > http://autobuilder.yocto.io:8010/builders/nightly-x86-lsb/builds/73 > http://autobuilder.yocto.io:8010/builders/nightly-wic/builds/73 > http://autobuilder.yocto.io:8010/builders/nightly-qa-extras/builds/55 > http://autobuilder.yocto.io:8010/builders/nightly-oe-selftest/builds/70 > http://autobuilder.yocto.io:8010/builders/nightly-multilib/builds/78 > > and similar errors on the main AB. I've confirmed it is this patch > which causes the issue. Presumably some errors are occurring but are > currently silently being ignored?
The solution here is probably to write a thorough test for package_manager.py. I imagine that is the culprit. I'm fine with moving this to 2.3 and I'll add a bug to create a robust set of tests. > > Its probably worth looking into what is going on in case there is some > real issue here but at this point given the complexity of the changes > I'm leaning towwards deferring this for 2.3. > > Cheers, > > Richard -- _______________________________________________ Openembedded-core mailing list Openembedded-core@lists.openembedded.org http://lists.openembedded.org/mailman/listinfo/openembedded-core
