[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