[Buildroot] [PATCH 02/18] package/mplayer: add optional giflib support

Thomas Petazzoni thomas.petazzoni at free-electrons.com
Mon Mar 30 20:56:58 UTC 2015


Dear Bernd Kuhls,

On Sun, 29 Mar 2015 19:11:18 +0200, Bernd Kuhls wrote:
> 
> Signed-off-by: Bernd Kuhls <bernd.kuhls at t-online.de>
> ---
>  package/mplayer/mplayer.mk |    6 ++++++
>  1 file changed, 6 insertions(+)
> 
> diff --git a/package/mplayer/mplayer.mk b/package/mplayer/mplayer.mk
> index 192a42e..f004ca9 100644
> --- a/package/mplayer/mplayer.mk
> +++ b/package/mplayer/mplayer.mk
> @@ -100,6 +100,11 @@ else
>  MPLAYER_CONF_OPTS += --disable-live
>  endif
>  
> +ifeq ($(BR2_PACKAGE_GIFLIB),y)
> +MPLAYER_DEPENDENCIES += giflib
> +MPLAYER_EXTRA_LIBS += -lgif
> +endif

Any reason not to use --enable-gif / --disable-gif here? Then you don't
need to pass a --extra-libs value. I've applied the patch with this
change, as it seems to build fine. Let me know if that causes a problem.

Thanks,

Thomas
-- 
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com



More information about the buildroot mailing list