[Buildroot] [PATCH] mediastreamer: fix build with libX11 and sdl without x11 support

[Buildroot] [PATCH] mediastreamer: fix build with libX11 and sdl without x11 support

[Peter Korsgaard]

Fixes http://autobuild.buildroot.net/results/a8f2db4a40c0f53c0d25b2632fe87c2ce136f770

Mediastreamer assumes SDL has X11 support if SDL and libX11 are found,
which is not necessarily true in BR.

Signed-off-by: Peter Korsgaard <jacmet@sunsite.dk>
---
 package/mediastreamer/mediastreamer.mk |    5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/package/mediastreamer/mediastreamer.mk b/package/mediastreamer/mediastreamer.mk
index 9e67ed4..1f4e1d4 100644
--- a/package/mediastreamer/mediastreamer.mk
+++ b/package/mediastreamer/mediastreamer.mk
@@ -48,9 +48,10 @@ else
 MEDIASTREAMER_CONF_OPT += --disable-sdl
 endif
 
-ifeq ($(BR2_PACKAGE_XLIB_LIBX11),y)
+# mediastreamer assumes SDL has X11 support if --enable-x11 (and X11 support
+# is only used for SDL output)
+ifeq ($(BR2_PACKAGE_SDL_X11),y)
 MEDIASTREAMER_CONF_OPT += --enable-x11
-MEDIASTREAMER_DEPENDENCIES += xlib_libX11
 else
 MEDIASTREAMER_CONF_OPT += --disable-x11
 endif
-- 
1.7.10

[Buildroot] [PATCH] mediastreamer: fix build with libX11 and sdl without x11 support

[Thomas Petazzoni]

Le Mon, 23 Jul 2012 11:49:19 +0200,
Peter Korsgaard <jacmet@sunsite.dk> a ?crit :

> -ifeq ($(BR2_PACKAGE_XLIB_LIBX11),y)
> +# mediastreamer assumes SDL has X11 support if --enable-x11 (and X11 support
> +# is only used for SDL output)
> +ifeq ($(BR2_PACKAGE_SDL_X11),y)
>  MEDIASTREAMER_CONF_OPT += --enable-x11
> -MEDIASTREAMER_DEPENDENCIES += xlib_libX11
>  else
>  MEDIASTREAMER_CONF_OPT += --disable-x11
>  endif

So this means that it is not possible to use mediastreamer on X11
without SDL?

I understand that when SDL support is enabled, it wants SDL to support
X11. But when X11 is enabled, does it requires SDL ?

Thanks!

Thomas
-- 
Thomas Petazzoni, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com

[Buildroot] [PATCH] mediastreamer: fix build with libX11 and sdl without x11 support

[Peter Korsgaard]

>>>>> "Thomas" == Thomas Petazzoni <thomas.petazzoni@free-electrons.com> writes:

 Thomas> Le Mon, 23 Jul 2012 11:49:19 +0200,
 Thomas> Peter Korsgaard <jacmet@sunsite.dk> a ?crit :

 >> -ifeq ($(BR2_PACKAGE_XLIB_LIBX11),y)
 >> +# mediastreamer assumes SDL has X11 support if --enable-x11 (and X11 support
 >> +# is only used for SDL output)
 >> +ifeq ($(BR2_PACKAGE_SDL_X11),y)
 >> MEDIASTREAMER_CONF_OPT += --enable-x11
 >> -MEDIASTREAMER_DEPENDENCIES += xlib_libX11
 >> else
 >> MEDIASTREAMER_CONF_OPT += --disable-x11
 >> endif

 Thomas> So this means that it is not possible to use mediastreamer on X11
 Thomas> without SDL?

Indeed. The Xlib support is only used for the SDL videoouput:

grep -rls HAVE_X11_XLIB_H mediastreamer-2.8.2
mediastreamer-2.8.2/configure
mediastreamer-2.8.2/mediastreamer-config.h.in
mediastreamer-2.8.2/src/videoout.c
mediastreamer-2.8.2/mediastreamer-config.h
mediastreamer-2.8.2/tests/test_x11window.c

Where videoout.c is:

grep -B1 -A1 videoout mediastreamer-2.8.2/src/Makefile.am
if BUILD_SDL
libmediastreamer_la_SOURCES+=videoout.c
endif

 Thomas> I understand that when SDL support is enabled, it wants SDL to support
 Thomas> X11. But when X11 is enabled, does it requires SDL ?

It does (and ffmpeg). The option should probably have been called
--enable-sdl-x11 instead.

-- 
Bye, Peter Korsgaard

[Buildroot] [PATCH] mediastreamer: fix build with libX11 and sdl without x11 support

[Thomas Petazzoni]

Le Mon, 23 Jul 2012 11:49:19 +0200,
Peter Korsgaard <jacmet@sunsite.dk> a ?crit :

> Fixes http://autobuild.buildroot.net/results/a8f2db4a40c0f53c0d25b2632fe87c2ce136f770
> 
> Mediastreamer assumes SDL has X11 support if SDL and libX11 are found,
> which is not necessarily true in BR.
> 
> Signed-off-by: Peter Korsgaard <jacmet@sunsite.dk>

Applied, thanks!

Thomas
-- 
Thomas Petazzoni, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com