On Thu, Mar 19, 2020 at 12:37 PM Martin Jansa <martin.ja...@gmail.com> wrote:
>
> On Thu, Mar 19, 2020 at 12:33:17PM -0700, Andre McCurdy wrote:
> > On Thu, Mar 19, 2020 at 12:19 PM Martin Jansa <martin.ja...@gmail.com>
> > wrote:
> > >
> > > From: Ross Burton <ross.bur...@intel.com>
> > >
> > > MJ: icu in master doesn't need distutils anymore, because icu 65.1
> > > currently in
> > > dunfell/master doesn't depend on python3-distutils anymore since:
> > > https://github.com/unicode-org/icu/commit/b4d41b0561b6e8de38b99850ce0e4be8ef536bb1
> > >
> > > but the icu-64.2 in zeus and
> > > openembedded-core/meta/recipes-core/ovmf/ovmf_git.bb
> > > still need python3-distutils as described in:
> > > http://lists.openembedded.org/pipermail/openembedded-core/2020-March/293984.html
> > >
> > > Signed-off-by: Ross Burton <ross.bur...@intel.com>
> > > ---
> > > meta/classes/sanity.bbclass | 12 +++++++-----
> > > 1 file changed, 7 insertions(+), 5 deletions(-)
> > >
> > > diff --git a/meta/classes/sanity.bbclass b/meta/classes/sanity.bbclass
> > > index 936fe913b4..5c2f8f9d75 100644
> > > --- a/meta/classes/sanity.bbclass
> > > +++ b/meta/classes/sanity.bbclass
> > > @@ -625,13 +625,14 @@ def check_sanity_version_change(status, d):
> > > # In other words, these tests run once in a given build directory
> > > and then
> > > # never again until the sanity version or host distrubution
> > > id/version changes.
> >
> > s/distrubution/distribution/ (not part of the original patch but maybe
> > worth fixing in a v2).
> >
> > > - # Check the python install is complete. glib-2.0-natives requries
> > > - # xml.parsers.expat
> > > + # Check the python install is complete. Examples that are often
> > > removed in
> > > + # minimal installations: glib-2.0-natives requries #
> > > xml.parsers.expat and icu
> >
> > Stray # left over from reformatting the comment.
> >
> > > + # requires distutils.sysconfig.
> > > try:
> > > import xml.parsers.expat
> > > - except ImportError:
> > > - status.addresult('Your python is not a full install. Please
> > > install the module xml.parsers.expat (python-xml on openSUSE and SUSE
> > > Linux).\n')
> > > - import stat
> > > + import distutils.sysconfig
> > > + except ImportError as e:
> > > + status.addresult('Your Python 3 is not a full install. Please
> > > install the module %s (see the Getting Started guide for further
> > > information).\n' % e.name)
> >
> > Should that be the Yocto Project Quick Start? Googling for "yocto
> > getting started guide" doesn't come up with anything.
> >
> > The version of Yocto Project Quick Start found by google doesn't
> > mention anything about installing python3-distutils though, so maybe
> > it's not the right document...
>
> This is just cherry-pick from master to zeus (I've only updated the
> commit message).
>
> If you want these issues fixed, then it should be fixed first in master
> and then backported to zeus.
Oh yes, didn't notice it's a backport to zeus. Sorry!
> >
> > > status.addresult(check_make_version(d))
> > > status.addresult(check_patch_version(d))
> > > @@ -667,6 +668,7 @@ def check_sanity_version_change(status, d):
> > > status.addresult('Please use ASSUME_PROVIDED +=, not
> > > ASSUME_PROVIDED = in your local.conf\n')
> > >
> > > # Check that TMPDIR isn't on a filesystem with limited filename
> > > length (eg. eCryptFS)
> > > + import stat
> > > tmpdir = d.getVar('TMPDIR')
> > > status.addresult(check_create_long_filename(tmpdir, "TMPDIR"))
> > > tmpdirmode = os.stat(tmpdir).st_mode
> > > --
> > > 2.20.1
> > >
> > > --
> > > _______________________________________________
> > > Openembedded-core mailing list
> > > Openembedded-core@lists.openembedded.org
> > > http://lists.openembedded.org/mailman/listinfo/openembedded-core
--
_______________________________________________
Openembedded-core mailing list
Openembedded-core@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-core
