[Buildroot] [PATCH v2 3/5] package/libv4l: add optional support for alsa-lib

[Yann E. MORIN]

Bernd, All,

On 2016-06-12 22:07 +0200, Bernd Kuhls spake thusly:
> Add alsa-lib as optional dependency to get a reproducable build.
> Without this patch configure logs:
> 
> checking for ALSA... no
> configure: WARNING: ALSA library not available
> 
> After this patch alsa-lib was built before libv4l:
> checking for ALSA... yes
> 
> Signed-off-by: Bernd Kuhls <bernd.kuhls at t-online.de>

I've had a cursory look at their configure.ac, and it seems there is no
way to explicitly disable support for alsa. Hopefully, it will not pick
up host libs when the target is the same as the host and alsa-libs is
disabled...

Reviewed-by: "Yann E. MORIN" <yann.morin.1998 at free.fr>

Regards,
Yann E. MORIN.

> ---
> v2: no changes
> 
>  package/libv4l/libv4l.mk | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/package/libv4l/libv4l.mk b/package/libv4l/libv4l.mk
> index d0a0369..e7a57f0 100644
> --- a/package/libv4l/libv4l.mk
> +++ b/package/libv4l/libv4l.mk
> @@ -18,6 +18,10 @@ LIBV4L_CONF_ENV = ac_cv_prog_cc_c99='-std=gnu99'
>  LIBV4L_LICENSE = GPLv2+ (utilities), LGPLv2.1+ (libraries)
>  LIBV4L_LICENSE_FILES = COPYING COPYING.libv4l lib/libv4l1/libv4l1-kernelcode-license.txt
>  
> +ifeq ($(BR2_PACKAGE_ALSA_LIB),y)
> +LIBV4L_DEPENDENCIES += alsa-lib
> +endif
> +
>  ifeq ($(BR2_PACKAGE_ARGP_STANDALONE),y)
>  LIBV4L_DEPENDENCIES += argp-standalone
>  LIBV4L_LIBS += -largp
> -- 
> 2.8.1
> 
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 223 225 172 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'