Re: [PATCH 5/9] python-setuptools: update to 41.0.1

[Paul Eggleton <paul.eggleton@linux.intel.com>]

Hi Alex,

On Thursday, 25 April 2019 4:20:40 AM NZST Alexander Kanavin wrote:
> On Wed, 24 Apr 2019 at 18:02, Richard Purdie <richard.purdie@linuxfoundation.org> wrote:
> > I think this breaks buildhistory.
> >
> > The autobuilder runs:
> >
> > '/home/pokybuild/yocto-worker/qemuppc/yocto-autobuilder-helper/scripts/collect-results', '/home/pokybuild/yocto-worker/qemuppc/build/build', '/srv/autobuilder/autobuilder.yoctoproject.org/pub/non-release/20190424-4/testresults', 'qemuppc'
> >
> > which results in a call like:
> >
> > /home/pokybuild/yocto-worker/qemuppc/build/build/../scripts/buildhistory-diff -p /home/pokybuild/yocto-worker/qemuppc/build/build/buildhistory ab-fetchrev
> >
> > which results in a comparision between:
> >
> >...
> >
> > which trips over:
> >
> > '/opt/poky/2.7+snapshot/sysroots/i686-pokysdk-linux/usr/lib/python3.7/site-packages/setuptools/script (dev).tmpl'
> >
> > and results in:
> >
> > Traceback (most recent call last):
> >   File "/home/pokybuild/yocto-worker/qemuppc/build/build/../scripts/buildhistory-diff", line 139, in <module>
> >     main()
> >   File "/home/pokybuild/yocto-worker/qemuppc/build/build/../scripts/buildhistory-diff", line 132, in main
> >     out = str(chg)
> >   File "/home/pokybuild/yocto-worker/qemuppc/build/meta/lib/oe/buildhistory_analysis.py", line 67, in __str__
> >     return self._str_internal(True)
> >   File "/home/pokybuild/yocto-worker/qemuppc/build/meta/lib/oe/buildhistory_analysis.py", line 133, in _str_internal
> >     depvera = bb.utils.explode_dep_versions2(self.oldvalue, sort=False)
> >   File "/home/pokybuild/yocto-worker/qemuppc/build/bitbake/lib/bb/utils.py", line 231, in explode_dep_versions2
> >     raise VersionStringException('Invalid version specification in "(%s" - invalid or missing operator' % i)
> >
> > This doesn't block the upgrade but it does need a bug and for us to
> > figure out some way of improving bb.utils.explode_dep_versions2 to
> > handle this better.
> 
> It seems as though the parser for package dependency lists is called
> on a plain file list, which should not be happening at all (as
> obviously files are allowed to have just about anything in their
> names). I'll look into it closer.

Did you get anywhere with this?

FYI I filed a bug for the issue:

  https://bugzilla.yoctoproject.org/show_bug.cgi?id=13467

Cheers,
Paul

-- 

Paul Eggleton
Intel Open Source Technology Centre