[Buildroot] [PATCH] libusb: add an option to compile examples
Gaël PORTAY
gael.portay at savoirfairelinux.com
Sun Jun 10 22:05:37 UTC 2018
Baruch,
On Sun, Jun 10, 2018 at 08:25:44PM +0300, Baruch Siach wrote:
> (...)
Hi Gaël,
>
> On Sun, Jun 10, 2018 at 12:36:20PM -0400, Gaël PORTAY wrote:
> > On Sun, Jun 10, 2018 at 05:35:29PM +0300, Baruch Siach wrote:
> > > > diff --git a/package/libusb/libusb.mk b/package/libusb/libusb.mk
> > > > index aee3622f8d..89ec91bb42 100644
> > > > --- a/package/libusb/libusb.mk
> > > > +++ b/package/libusb/libusb.mk
> > > > @@ -25,5 +25,16 @@ else
> > > > LIBUSB_CONF_OPTS += --disable-udev
> > > > endif
> > > >
> > > > +ifeq ($(BR2_PACKAGE_LIBUSB_EXAMPLES),y)
> > > > +LIBUSB_CONF_OPTS += --enable-examples-build
> > > > +define LIBUSB_INSTALL_TARGET_EXAMPLES
> > > > + for example in listdevs xusb fxload hotplugtest testlibusb dpfp dpfp_threaded sam3u_benchmark ; do \
> > > > + [ -x $(@D)/examples/$${example} ] || continue; \
> > >
> > > Why is this file presence check needed?
> >
> > Because, some of the examples are compiled under HAVE_xxx condition.
> >
> > See the extract of examples/Makefile.am below.
> >
> > noinst_PROGRAMS = listdevs xusb fxload hotplugtest testlibusb
> >
> > if HAVE_SIGACTION
> > noinst_PROGRAMS += dpfp
> >
> > if THREADS_POSIX
> > dpfp_threaded_CFLAGS = $(AM_CFLAGS)
> > noinst_PROGRAMS += dpfp_threaded
> > endif
> >
> > sam3u_benchmark_SOURCES = sam3u_benchmark.c
> > noinst_PROGRAMS += sam3u_benchmark
> > endif
>
> All Buildroot libcs support sigaction. Also, libusb depends on threads
> support, so THREADS_POSIX is always true in Buildroot.
>
> So I think this check is not needed.
>
Okay, I will send a v2 without this check.
More information about the buildroot
mailing list