diff mbox

package/freerdp: fix build with gstreamer support

Message ID 1432074933-4039-1-git-send-email-s.martin49@gmail.com
State Accepted
Headers show

Commit Message

Samuel Martin May 19, 2015, 10:35 p.m. UTC
FreeRDP with gstreamer support enabled needs gstreamer and gst-plugins-base
with gstapp.

Fixes:
  http://autobuild.buildroot.net/results/937/9373d4238bc3f027e571581bd3531018c4faed91/
  http://autobuild.buildroot.org/results/f77/f77a2c9eea7973b54a22e12ab33ee8964adaddfd/

Signed-off-by: Samuel Martin <s.martin49@gmail.com>
---
 package/freerdp/Config.in  | 2 ++
 package/freerdp/freerdp.mk | 3 ++-
 2 files changed, 4 insertions(+), 1 deletion(-)

Comments

Gergely Imreh May 20, 2015, 5:02 a.m. UTC | #1
On 20 May 2015 at 06:35, Samuel Martin <s.martin49@gmail.com> wrote:
> FreeRDP with gstreamer support enabled needs gstreamer and gst-plugins-base
> with gstapp.
>
> Fixes:
>   http://autobuild.buildroot.net/results/937/9373d4238bc3f027e571581bd3531018c4faed91/
>   http://autobuild.buildroot.org/results/f77/f77a2c9eea7973b54a22e12ab33ee8964adaddfd/
>
> Signed-off-by: Samuel Martin <s.martin49@gmail.com>

Tested-by: Gergely Imreh <imrehg@gmail.com>
 by running `make freerdp` reproducing the autobuild fails and
successful building with this patch

> ---
>  package/freerdp/Config.in  | 2 ++
>  package/freerdp/freerdp.mk | 3 ++-
>  2 files changed, 4 insertions(+), 1 deletion(-)
>
> diff --git a/package/freerdp/Config.in b/package/freerdp/Config.in
> index 2e879b4..f7d6550 100644
> --- a/package/freerdp/Config.in
> +++ b/package/freerdp/Config.in
> @@ -5,6 +5,8 @@ config BR2_PACKAGE_FREERDP
>         depends on BR2_TOOLCHAIN_HAS_THREADS
>         select BR2_PACKAGE_OPENSSL
>         select BR2_PACKAGE_ZLIB
> +       select BR2_PACKAGE_GST_PLUGINS_BASE if BR2_PACKAGE_GSTREAMER
> +       select BR2_PACKAGE_GST_PLUGINS_BASE_PLUGIN_APP if BR2_PACKAGE_GSTREAMER
>         help
>           FreeRDP is a free implementation of the Remote Desktop
>           Protocol (RDP), released under the Apache license
> diff --git a/package/freerdp/freerdp.mk b/package/freerdp/freerdp.mk
> index c7c933e..0d10ef8 100644
> --- a/package/freerdp/freerdp.mk
> +++ b/package/freerdp/freerdp.mk
> @@ -17,7 +17,8 @@ FREERDP_CONF_OPTS = -DWITH_MANPAGES=OFF -Wno-dev
>
>  ifeq ($(BR2_PACKAGE_GSTREAMER),y)
>  FREERDP_CONF_OPTS += -DWITH_GSTREAMER=ON
> -FREERDP_DEPENDENCIES += gstreamer
> +# freerdp needs gstinterface and gstapp from gst-plugins-base
> +FREERDP_DEPENDENCIES += gstreamer gst-plugins-base
>  else
>  FREERDP_CONF_OPTS += -DWITH_GSTREAMER=OFF
>  endif
> --
> 2.4.1
>
> _______________________________________________
> buildroot mailing list
> buildroot@busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot
Thomas Petazzoni May 20, 2015, 9:23 p.m. UTC | #2
Dear Samuel Martin,

On Wed, 20 May 2015 00:35:33 +0200, Samuel Martin wrote:
> FreeRDP with gstreamer support enabled needs gstreamer and gst-plugins-base
> with gstapp.
> 
> Fixes:
>   http://autobuild.buildroot.net/results/937/9373d4238bc3f027e571581bd3531018c4faed91/
>   http://autobuild.buildroot.org/results/f77/f77a2c9eea7973b54a22e12ab33ee8964adaddfd/
> 
> Signed-off-by: Samuel Martin <s.martin49@gmail.com>
> ---
>  package/freerdp/Config.in  | 2 ++
>  package/freerdp/freerdp.mk | 3 ++-
>  2 files changed, 4 insertions(+), 1 deletion(-)

Applied, thanks.

Thomas
diff mbox

Patch

diff --git a/package/freerdp/Config.in b/package/freerdp/Config.in
index 2e879b4..f7d6550 100644
--- a/package/freerdp/Config.in
+++ b/package/freerdp/Config.in
@@ -5,6 +5,8 @@  config BR2_PACKAGE_FREERDP
 	depends on BR2_TOOLCHAIN_HAS_THREADS
 	select BR2_PACKAGE_OPENSSL
 	select BR2_PACKAGE_ZLIB
+	select BR2_PACKAGE_GST_PLUGINS_BASE if BR2_PACKAGE_GSTREAMER
+	select BR2_PACKAGE_GST_PLUGINS_BASE_PLUGIN_APP if BR2_PACKAGE_GSTREAMER
 	help
 	  FreeRDP is a free implementation of the Remote Desktop
 	  Protocol (RDP), released under the Apache license
diff --git a/package/freerdp/freerdp.mk b/package/freerdp/freerdp.mk
index c7c933e..0d10ef8 100644
--- a/package/freerdp/freerdp.mk
+++ b/package/freerdp/freerdp.mk
@@ -17,7 +17,8 @@  FREERDP_CONF_OPTS = -DWITH_MANPAGES=OFF -Wno-dev
 
 ifeq ($(BR2_PACKAGE_GSTREAMER),y)
 FREERDP_CONF_OPTS += -DWITH_GSTREAMER=ON
-FREERDP_DEPENDENCIES += gstreamer
+# freerdp needs gstinterface and gstapp from gst-plugins-base
+FREERDP_DEPENDENCIES += gstreamer gst-plugins-base
 else
 FREERDP_CONF_OPTS += -DWITH_GSTREAMER=OFF
 endif