[Buildroot] [PATCH 1/1] package/libnss: fix build on powerpc

[Joel Stanley]

Hi Fabrice,

On Tue, 5 Apr 2022 at 21:35, Fabrice Fontaine
<fontaine.fabrice at gmail.com> wrote:
>
> Set NSS_DISABLE_CRYPTO_VSX which is available since version 3.64 and
> https://github.com/nss-dev/nss/commit/9dab43371d4d924419523e18ba84f02804880533
> to avoid the following build failure on powerpc:
>
> cc1: warning: '-mvsx' requires hardware floating point
> cc1: error: '-mno-vsx' turns off '-mcrypto'
>
> Fixes:
>  - http://autobuild.buildroot.org/results/6bedb5b658f6c9c16c26c73a524a995e5e84fcc8
>
> Signed-off-by: Fabrice Fontaine <fontaine.fabrice at gmail.com>
> ---
>  package/libnss/libnss.mk | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/package/libnss/libnss.mk b/package/libnss/libnss.mk
> index 2f7a265136..4582c55ada 100644
> --- a/package/libnss/libnss.mk
> +++ b/package/libnss/libnss.mk
> @@ -51,6 +51,7 @@ LIBNSS_BUILD_VARS = \
>         NSPR_INCLUDE_DIR=$(STAGING_DIR)/usr/include/nspr \
>         NSPR_LIB_DIR=$(STAGING_DIR)/usr/lib \
>         NS_USE_GCC=1 \
> +       NSS_DISABLE_CRYPTO_VSX=1 \

How about we introduce a BR2_POWERPC_CPU_HAS_VSX, in a similar fashion
to BR2_POWERPC_CPU_HAS_ALITIVEC?

I've prepared a patch to do that, and rebased your change to use it:

 https://github.com/shenki/buildroot/commits/powerpc-vsx

I did a build test of the autobuilder config you linked to and it succeeded.

I'll send them out tomorrow unless you have an alternative proposal.

Cheers,

Joel

>         NSS_DISABLE_GTESTS=1 \
>         NSS_USE_SYSTEM_SQLITE=1 \
>         NATIVE_CC="$(HOSTCC)" \
> --
> 2.35.1
>
> _______________________________________________
> buildroot mailing list
> buildroot at buildroot.org
> https://lists.buildroot.org/mailman/listinfo/buildroot