Re: [PATCH] util-linux: Fix QA warnings about ALTERNATIVES

[Richard Purdie <richard.purdie@linuxfoundation.org>]

On Wed, 2013-11-13 at 16:11 +0800, Chong.Lu@windriver.com wrote:
> From: Lu Chong <Chong.Lu@windriver.com>
> 
> Previously, EXTRA_OECONF would be overwritten by EXTRA_OECONF_class-native
> and EXTRA_OECONF_class-nativesdk when we executed `make export-sdk' command.

We don't have an export-sdk command do we?

> As a result, some applications such as `last', `mesg' and `reset' didn't get
> compiled.
> 
> The patch rewrote EXTRA_OECONF_class-native and EXTRA_OECONF_class-nativesdk
> variables to fix QA warnings.
> 
> usr/share/man/man1/last.1.util-linux does not exist
> usr/share/man/man1/mesg.1.util-linux does not exist
> usr/bin/last.util-linux does not exist
> usr/bin/mesg.util-linux does not exist
> bin/reset does not exist
> 
> Signed-off-by: Lu Chong <Chong.Lu@windriver.com>
> ---
>  meta/recipes-core/util-linux/util-linux_2.23.2.bb |   22 +++++++++++++++++++--
>  1 file changed, 20 insertions(+), 2 deletions(-)
>
> diff --git a/meta/recipes-core/util-linux/util-linux_2.23.2.bb b/meta/recipes-core/util-linux/util-linux_2.23.2.bb
> index b5c221b..b9cdc61 100644
> --- a/meta/recipes-core/util-linux/util-linux_2.23.2.bb
> +++ b/meta/recipes-core/util-linux/util-linux_2.23.2.bb
> @@ -18,5 +18,23 @@ SRC_URI[md5sum] = "39a02ad0b3b37824c394f40930b7aa38"
>  SRC_URI[sha256sum] = "6c5be3f7beec91b4893de14bbd722886fa2521be5bfa0fc079b749d0018633b1"
>  
>  CACHED_CONFIGUREVARS += "scanf_cv_alloc_modifier=as"
> -EXTRA_OECONF_class-native += "--disable-fallocate --disable-use-tty-group"
> -EXTRA_OECONF_class-nativesdk += "--disable-fallocate --disable-use-tty-group"
> +EXTRA_OECONF_class-native = "--disable-use-tty-group \
> +                             --disable-makeinstall-chown --enable-elvtune --enable-init \
> +                             --enable-kill --enable-last --enable-mesg --enable-partx \
> +                             --enable-raw --enable-rdev --enable-reset --disable-login \
> +                             --disable-vipw --disable-newgrp --disable-chfn-chsh \
> +                             --enable-write --enable-arch --enable-mount --with-fsprobe=builtin \
> +                             --enable-libuuid --enable-libblkid --enable-fsck --without-udev \
> +                             usrsbin_execdir='${sbindir}' \
> +                             --disable-fallocate --disable-use-tty-group \
> +"
> +EXTRA_OECONF_class-nativesdk = "--disable-use-tty-group \
> +                                --disable-makeinstall-chown --enable-elvtune --enable-init \
> +                                --enable-kill --enable-last --enable-mesg --enable-partx \
> +                                --enable-raw --enable-rdev --enable-reset --disable-login \
> +                                --disable-vipw --disable-newgrp --disable-chfn-chsh \
> +                                --enable-write --enable-arch --enable-mount --with-fsprobe=builtin \
> +                                --enable-libuuid --enable-libblkid --enable-fsck --without-udev \
> +                                usrsbin_execdir='${sbindir}' \
> +                                --disable-fallocate --disable-use-tty-group \
> +"

We are not duplicating EXTRA_OECONF like that, its horrible. How about
something like:

EXTRA_OECONF = "xxxx ${SOMEVAR}"
SOMEVAR = "z"
SOMEVAR_class-native = "x"
SOMEVAR_class-nativesdk = "y"

? I'm having to guess at what your real problem is though as the commit
message doesn't contain the right information.

Cheers,

Richard