Re: [meta-networking][PATCH v2] Samba: use built-in libtevent

[Martin Jansa <martin.jansa@gmail.com>]

[-- Attachment #1: Type: text/plain, Size: 5644 bytes --]

On Mon, Feb 20, 2017 at 01:04:58PM +0100, Andreas Müller wrote:
> On Sat, Feb 11, 2017 at 3:17 PM, Gyorgy Szombathelyi <gyurco@freemail.hu> wrote:
> > Since Samba uses libtevent-internal.h in some places, it is
> > incompatible with external libtevent versions if they're not
> > the same as the built-in, and just crashes.
> >
> > Signed-off-by: Gyorgy Szombathelyi <gyurco@freemail.hu>
> > ---
> >  meta-networking/recipes-connectivity/samba/samba_4.4.5.bb | 4 ++--
> >  1 file changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/meta-networking/recipes-connectivity/samba/samba_4.4.5.bb b/meta-networking/recipes-connectivity/samba/samba_4.4.5.bb
> > index 0ba59c7..754455e 100644
> > --- a/meta-networking/recipes-connectivity/samba/samba_4.4.5.bb
> > +++ b/meta-networking/recipes-connectivity/samba/samba_4.4.5.bb
> > @@ -33,7 +33,7 @@ inherit systemd waf-samba cpan-base perlnative update-rc.d
> >  # remove default added RDEPENDS on perl
> >  RDEPENDS_${PN}_remove = "perl"
> >
> > -DEPENDS += "readline virtual/libiconv zlib popt libtalloc libtdb libtevent libldb krb5 libbsd libaio libpam"
> > +DEPENDS += "readline virtual/libiconv zlib popt libtalloc libtdb libldb krb5 libbsd libaio libpam"
> >  DEPENDS_append_libc-musl = " libtirpc"
> >  CFLAGS_append_libc-musl = " -I${STAGING_INCDIR}/tirpc"
> >  LDFLAGS_append_libc-musl = " -ltirpc"
> > @@ -74,7 +74,7 @@ SAMBA4_PDB_MODULES="pdb_tdbsam,${@bb.utils.contains('PACKAGECONFIG', 'ldap', 'pd
> >  SAMBA4_AUTH_MODULES="auth_unix,auth_wbc,auth_server,auth_netlogond,auth_script,auth_samba4"
> >  SAMBA4_MODULES="${SAMBA4_IDMAP_MODULES},${SAMBA4_PDB_MODULES},${SAMBA4_AUTH_MODULES}"
> >
> > -SAMBA4_LIBS="heimdal,!zlib,!popt,!talloc,!pytalloc,!pytalloc-util,!tevent,!pytevent,!tdb,!pytdb,!ldb,!pyldb"
> > +SAMBA4_LIBS="heimdal,!zlib,!popt,!talloc,!pytalloc,!pytalloc-util,tevent,!pytevent,!tdb,!pytdb,!ldb,!pyldb"
> >
> >  EXTRA_OECONF += "--enable-fhs \
> >                   --with-piddir=/run \
> > --
> > 2.9.3
> >
> This one causes conflict with libldb. Unfortunately these kind of
> conflicts remain undetected (I had error with gvfs but that isn't
> build in Martins's world - I guess) [1]. Without RSS I get:

In last build I've noticed only this new issue related to samba:

ctdb-2.5.6: ctdb rdepends on samba, but it isn't a build dependency, missing samba in DEPENDS or PACKAGECONFIG? [build-deps]

Maybe this dependency was created from samba being libldb provider in
this build.

gvfs and whole meta-gnome layers is still included in my world, maybe it
didn't fail because some gvfs dependency was already failing or one of
samba related changes in master-next is fixing this one.

Regards,

> 
> ERROR: samba-4.4.5-r0 do_populate_sysroot: The recipe samba is trying
> to install files into a shared area when those files already exist.
> Those files and their manifest location are:
>    /home/a.mueller/tmp/oe-core-glibc/sysroots/raspberrypi2/usr/lib/python2.7/site-packages/_ldb_text.py
>  Matched in b'manifest-raspberrypi2-libldb.populate_sysroot'
>  /home/a.mueller/tmp/oe-core-glibc/sysroots/raspberrypi2/usr/lib/python2.7/site-packages/ldb.so
>  Matched in b'manifest-raspberrypi2-libldb.populate_sysroot'
>  /home/a.mueller/tmp/oe-core-glibc/sysroots/raspberrypi2/usr/lib/python2.7/site-packages/_tevent.so
>  Matched in b'manifest-raspberrypi2-libtevent.populate_sysroot'
>  /home/a.mueller/tmp/oe-core-glibc/sysroots/raspberrypi2/usr/lib/python2.7/site-packages/tevent.py
>  Matched in b'manifest-raspberrypi2-libtevent.populate_sysroot'
> Please verify which recipe should provide the above files.
> The build has stopped as continuing in this scenario WILL break
> things, if not now, possibly in the future (we've seen builds fail
> several months later). If the system knew how to recover from this
> automatically it would however there are several different scenarios
> which can result in this and we don't know which one this is. It may
> be you have switched providers of something like virtual/kernel (e.g.
> from linux-yocto to linux-yocto-dev), in that case you need to execute
> the clean task for both recipes and it will resolve this error. It may
> be you changed DISTRO_FEATURES from systemd to udev or vice versa.
> Cleaning those recipes should again resolve this error however
> switching DISTRO_FEATURES on an existing build directory is not
> supported, you should really clean out tmp and rebuild (reusing sstate
> should be safe). It could be the overlapping files detected are
> harmless in which case adding them to SSTATE_DUPWHITELIST may be the
> correct solution. It could also be your build is including two
> different conflicting versions of things (e.g. bluez 4 and bluez 5 and
> the correct solution for that would be to resolve the conflict. If in
> doubt, please ask on the mailing list, sharing the error and filelist
> above.
> ERROR: samba-4.4.5-r0 do_populate_sysroot: If the above message is too
> much, the simpler version is you're advised to wipe out tmp and
> rebuild (reusing sstate is fine). That will likely fix things in most
> (but not all) cases.
> ERROR: samba-4.4.5-r0 do_populate_sysroot: Function failed: sstate_task_postfunc
> 
> [1] http://lists.openembedded.org/pipermail/openembedded-core/2017-February/132938.html
> 
> Andreas
> -- 
> _______________________________________________
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-devel

-- 
Martin 'JaMa' Jansa     jabber: Martin.Jansa@gmail.com

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 201 bytes --]