package/freerdp: re-add support for gstreamer

Previously, we expected the user to select gstreamer-0.x on his own,
to enable gstreamer support in FreeRDP. This could have been a bit
confusing to the user, as he may have enabled gst-1.x but FreeRDP did
only support gst-0.x.

Also, gstreamer support needs xlib-libxrandr, which was missing in
FreeRDP's dependencies, so it was never enabled (AFAICS).

(Re-)introduce support for gstreamer-0.x and gstreamer-1.x, since both
are supported.

We're doing it in a choice, and select whichever version the user chooses,
rather than automatically detect it as previosuly done.  We can select the
gstreamer packages, as their dependencies are anyway already covered by the
ones of FreeRDP.

This also now requires xlib-libxrandr, so hide the choice if X.org is
not enabled, still offer the option of not using gstreamer if it is.

[Peter: Hide option if gstreamer{,1} aren't enabled,
	Default to gstreamer{,1} support enabled
	GStreamer 0.10 support needs host-pkgconf and libxml2]
Signed-off-by: "Yann E. MORIN" <yann.morin.1998@free.fr>
Signed-off-by: Peter Korsgaard <peter@korsgaard.com>

diff --git a/package/freerdp/Config.in b/package/freerdp/Config.in
index ab8c3f59a9e9decb61a056dd88aef50f401f6108..12619a5b05287b23af0c71878c66fccc68926759 100644 (file)
--- a/package/freerdp/Config.in
+++ b/package/freerdp/Config.in
@@ -23,6 +23,37 @@ config BR2_PACKAGE_FREERDP
 
 if BR2_PACKAGE_FREERDP
 
+choice
+       bool "gstreamer support"
+       depends on BR2_PACKAGE_XORG7 # xlib-libxrandr
+       depends on BR2_PACKAGE_GSTREAMER || BR2_PACKAGE_GSTREAMER1
+
+config BR2_PACKAGE_FREERDP_GSTREAMER1
+       bool "gstreamer-1.x"
+       depends on BR2_PACKAGE_GSTREAMER1
+       # gstreamer-1.x dependencies already dependencies of FreeRDP
+       select BR2_PACKAGE_GST1_PLUGINS_BASE
+       select BR2_PACKAGE_GST1_PLUGINS_BASE_PLUGIN_APP
+       select BR2_PACKAGE_XLIB_LIBXRANDR
+
+config BR2_PACKAGE_FREERDP_GSTREAMER
+       bool "gstreamer-0.x"
+       depends on BR2_PACKAGE_GSTREAMER
+       # gstreamer-0.x dependencies already dependencies of FreeRDP
+       select BR2_PACKAGE_GST_PLUGINS_BASE
+       select BR2_PACKAGE_GST_PLUGINS_BASE_PLUGIN_APP
+       select BR2_PACKAGE_XLIB_LIBXRANDR
+       select BR2_PACKAGE_LIBXML2
+
+config BR2_PACKAGE_FREERDP_GSTREAMER_NO
+       bool "none"
+
+endchoice
+
+comment "gstreamer support needs X.Org"
+       depends on !BR2_PACKAGE_XORG7
+       depends on BR2_PACKAGE_GSTREAMER || BR2_PACKAGE_GSTREAMER1
+
 config BR2_PACKAGE_FREERDP_SERVER
        bool "server"
        depends on BR2_PACKAGE_XORG7

diff --git a/package/freerdp/freerdp.mk b/package/freerdp/freerdp.mk
index 92051696055e129e5700e82deb4494d79bcea815..30108b6843f5cf4718ec86b8ab149cfd910ccf3f 100644 (file)
--- a/package/freerdp/freerdp.mk
+++ b/package/freerdp/freerdp.mk
@@ -15,7 +15,19 @@ FREERDP_INSTALL_STAGING = YES
 
 FREERDP_CONF_OPTS = -DWITH_MANPAGES=OFF -Wno-dev
 
-FREERDP_CONF_OPTS += -DWITH_GSTREAMER_0_10=OFF -DWITH_GSTREAMER_1_0=OFF
+ifeq ($(BR2_PACKAGE_FREERDP_GSTREAMER),y)
+FREERDP_CONF_OPTS += -DWITH_GSTREAMER_0_10=ON
+FREERDP_DEPENDENCIES += gstreamer gst-plugins-base libxml2 host-pkgconf
+else
+FREERDP_CONF_OPTS += -DWITH_GSTREAMER_0_10=OFF
+endif
+
+ifeq ($(BR2_PACKAGE_FREERDP_GSTREAMER1),y)
+FREERDP_CONF_OPTS += -DWITH_GSTREAMER_1_0=ON
+FREERDP_DEPENDENCIES += gstreamer1 gst1-plugins-base
+else
+FREERDP_CONF_OPTS += -DWITH_GSTREAMER_1_0=OFF
+endif
 
 ifeq ($(BR2_PACKAGE_CUPS),y)
 FREERDP_CONF_OPTS += -DWITH_CUPS=ON