[Buildroot] [PATCH-FOR-NEXT v1 1/6] package: add libnpsr host package
Gaël PORTAY
gael.portay at savoirfairelinux.com
Thu Feb 22 14:29:13 UTC 2018
Thomas,
On Wed, Feb 21, 2018 at 10:27:00PM +0100, Thomas Petazzoni wrote:
> Hello,
>
> On Wed, 21 Feb 2018 09:27:56 -0500, Gaël PORTAY wrote:
> > Signed-off-by: Gaël PORTAY <gael.portay at savoirfairelinux.com>
> > ---
> > package/libnspr/Config.in.host | 10 ++++++++++
> > package/libnspr/libnspr.mk | 7 +++++++
> > 2 files changed, 17 insertions(+)
> > create mode 100644 package/libnspr/Config.in.host
> >
> > diff --git a/package/libnspr/Config.in.host b/package/libnspr/Config.in.host
> > new file mode 100644
> > index 0000000000..a80b55e122
> > --- /dev/null
> > +++ b/package/libnspr/Config.in.host
> > @@ -0,0 +1,10 @@
> > +config BR2_PACKAGE_HOST_LIBNSPR
> > + bool "host libnspr"
> > + help
> > + NSPR is the Netscape Portable Runtime library which provides
> > + a platform-neutral API for system level and libc like
> > + functions. The library is used by the Mozilla project as well
> > + as others.
> > +
> > + http://www.mozilla.org/projects/nspr/
> > +
>
> It is not necessary to add a Config.in.host option for this package, it
> is just a build dependency of another package.
>
Okay. I have not noticed that.
> > diff --git a/package/libnspr/libnspr.mk b/package/libnspr/libnspr.mk
> > index 0c782ae309..c1662947f7 100644
> > --- a/package/libnspr/libnspr.mk
> > +++ b/package/libnspr/libnspr.mk
> > @@ -49,4 +49,11 @@ LIBNSPR_INSTALL_TARGET_OPTS = DESTDIR=$(TARGET_DIR) LIBRARY= install
> > LIBNSPR_INSTALL_STAGING_OPTS = DESTDIR=$(STAGING_DIR) LIBRARY= install
> > endif
> >
> > +ifeq ($(HOSTARCH),x86_64)
> > +HOST_LIBNSPR_CONF_OPTS += --enable-64bit
>
> x86_64 is not the only 64-bit architecture in the world: we have an
> autobuilder running on a ppc64 machine for example.
>
Okay, great! I did not know that we were supporting other architecture
than intel for build :)
BTW, I picked up the "code" from another package. Probably took it from
wine.mk.
# Wine needs to enable 64-bit build tools on 64-bit host
ifeq ($(HOSTARCH),x86_64)
HOST_WINE_CONF_OPTS += --enable-win64
endif
I may filter to %64 instead. Do very 64bits architectures are named
using 64?
> We don't really have a good way today to test if the host architecture
> is 64 bit or not, so it would need to be added.
>
> Best regards,
>
> Thomas
> --
> Thomas Petazzoni, CTO, Bootlin (formerly Free Electrons)
> Embedded Linux and Kernel engineering
> http://bootlin.com
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot
Regards,
Gael
More information about the buildroot
mailing list