On Jul 30, 2013, at 3:50 AM, Paul Eggleton <[email protected]> wrote:
> On Tuesday 30 July 2013 01:52:42 Khem Raj wrote: >> On Jul 26, 2013, at 12:01 AM, Koen Kooi <[email protected]> wrote: >>> Op 25 jul. 2013, om 16:51 heeft Koen Kooi <[email protected]> het > volgende geschreven: >>>> The ??= operator is too weak and it's setting a non-existent >>>> PACKAGECONFIG option ('libpam' instead of 'pam'). >>>> >>>> Signed-off-by: Koen Kooi <[email protected]> >>>> --- >>>> meta/recipes-core/util-linux/util-linux.inc | 4 ++-- >>>> 1 file changed, 2 insertions(+), 2 deletions(-) >>>> >>>> diff --git a/meta/recipes-core/util-linux/util-linux.inc >>>> b/meta/recipes-core/util-linux/util-linux.inc index d373cec..3d1198a >>>> 100644 >>>> --- a/meta/recipes-core/util-linux/util-linux.inc >>>> +++ b/meta/recipes-core/util-linux/util-linux.inc >>>> @@ -47,9 +47,9 @@ EXTRA_OECONF = "--libdir=${base_libdir} >>>> --disable-use-tty-group \>> >>>> --enable-libuuid --enable-libblkid --enable-fsck >>>> --without-udev \ >>>> >>>> usrsbin_execdir='${sbindir}' \ >>>> >>>> " >>>> -PACKAGECONFIG_class-target ??= "${@base_contains('DISTRO_FEATURES', >>>> 'pam', 'libpam', '', d)} \ +PACKAGECONFIG_class-target ?= >>>> "${@base_contains('DISTRO_FEATURES', 'pam', 'pam', '', d)} \>> >>>> ${@base_contains('DISTRO_FEATURES', >>>> 'systemd', 'systemd', '', d)} " >>>> >>>> -PACKAGECONFIG[pam] = "--enable-su --enable-runuser,--disable-su >>>> --disable-runuser, pam," +PACKAGECONFIG[pam] = "--enable-su >>>> --enable-runuser,--disable-su --disable-runuser, libpam," >>>> >>>> # Respect the systemd feature for uuidd >>>> PACKAGECONFIG[systemd] = "--enable-socket-activation >>>> --with-systemdsystemunitdir=${systemd_unitdir}/system/, >>>> --disable-socket-activation --without-systemdsystemunitdir"> >>> And of course this will trigger another bug: >>> | * check_data_file_clashes: Package util-linux wants to install file >>> | /build/v2013.06/build/tmp-angstrom_v2013_06-eglibc/work/beaglebone-ang >>> | strom-linux-gnueabi/cloud9-image/1.0-r0/rootfs/usr/bin/su> | >>> | But that file is already provided by package * shadow >>> >>> *sigh* >> >> +*sigh* >> >> has this been fixed yet ? just ran into same issue > > The patch to fix it is in Saul's most recent consolidated pull. > which one is it ? I am at commit 09deeef20ee5a0c12ad4fd89cace6e0fb832d5b1 Author: Saul Wold <[email protected]> Date: Thu Jul 25 14:15:18 2013 -0700 external-sourcery: add missing providers > Cheers, > Paul > > -- > > Paul Eggleton > Intel Open Source Technology Centre _______________________________________________ Openembedded-core mailing list [email protected] http://lists.openembedded.org/mailman/listinfo/openembedded-core
