From: Peter Korsgaard <peter@korsgaard.com>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH 08/10] package/freerdp: drop gstreamer 0.10.x support
Date: Fri, 7 Feb 2020 15:50:23 +0100 [thread overview]
Message-ID: <20200207145026.28384-9-peter@korsgaard.com> (raw)
In-Reply-To: <20200207145026.28384-1-peter@korsgaard.com>
With the upcoming removal of gstreamer 0.10, the logic for building freerdp
with support for it must go as well.
As there is now a single option for gstreamer (1.x) support, convert the
gstreamer support choice to a normal option for simplicity.
Signed-off-by: Peter Korsgaard <peter@korsgaard.com>
---
Config.in.legacy | 7 +++++++
package/freerdp/Config.in | 24 +++---------------------
package/freerdp/freerdp.mk | 9 +--------
3 files changed, 11 insertions(+), 29 deletions(-)
diff --git a/Config.in.legacy b/Config.in.legacy
index 34f99b54de..96848f7f06 100644
--- a/Config.in.legacy
+++ b/Config.in.legacy
@@ -146,6 +146,13 @@ endif
comment "Legacy options removed in 2020.02"
+config BR2_PACKAGE_FREERDP_GSTREAMER
+ bool "freerdp gstreamer 0.10.x support removed"
+ select BR2_LEGACY
+ help
+ Gstreamer 0.10.x is no longer available in Buildroot, so
+ neither is the support in freerdp.
+
config BR2_PACKAGE_OPENCV3_WITH_GSTREAMER
bool "opencv3 gstreamer 0.10.x support removed"
select BR2_LEGACY
diff --git a/package/freerdp/Config.in b/package/freerdp/Config.in
index 59bc18b672..5705ce0e0c 100644
--- a/package/freerdp/Config.in
+++ b/package/freerdp/Config.in
@@ -28,36 +28,18 @@ 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"
+ bool "gstreamer support"
depends on BR2_PACKAGE_GSTREAMER1
+ depends on BR2_PACKAGE_XORG7 # xlib-libxrandr
# 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
+ depends on BR2_PACKAGE_GSTREAMER1
config BR2_PACKAGE_FREERDP_SERVER
bool "server"
diff --git a/package/freerdp/freerdp.mk b/package/freerdp/freerdp.mk
index 293b6b5c01..a477e74cb7 100644
--- a/package/freerdp/freerdp.mk
+++ b/package/freerdp/freerdp.mk
@@ -12,14 +12,7 @@ FREERDP_LICENSE_FILES = LICENSE
FREERDP_INSTALL_STAGING = YES
-FREERDP_CONF_OPTS = -DWITH_MANPAGES=OFF -Wno-dev
-
-ifeq ($(BR2_PACKAGE_FREERDP_GSTREAMER),y)
-FREERDP_CONF_OPTS += -DWITH_GSTREAMER_0_10=ON
-FREERDP_DEPENDENCIES += gstreamer gst-plugins-base libxml2
-else
-FREERDP_CONF_OPTS += -DWITH_GSTREAMER_0_10=OFF
-endif
+FREERDP_CONF_OPTS = -DWITH_MANPAGES=OFF -Wno-dev -DWITH_GSTREAMER_0_10=OFF
ifeq ($(BR2_PACKAGE_FREERDP_GSTREAMER1),y)
FREERDP_CONF_OPTS += -DWITH_GSTREAMER_1_0=ON
--
2.20.1
next prev parent reply other threads:[~2020-02-07 14:50 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-02-07 14:50 [Buildroot] [PATCH 00/10] Remove gstreamer 0.10 Peter Korsgaard
2020-02-07 14:50 ` [Buildroot] [PATCH 01/10] package/classpath: drop gstreamer 0.10.x support Peter Korsgaard
2020-02-07 14:50 ` [Buildroot] [PATCH 02/10] package/gupnp-dlna: " Peter Korsgaard
2020-02-07 14:50 ` [Buildroot] [PATCH 03/10] package/libnice: " Peter Korsgaard
2020-02-07 14:50 ` [Buildroot] [PATCH 04/10] package/qt5multimedia: " Peter Korsgaard
2020-02-07 14:50 ` [Buildroot] [PATCH 05/10] package/libplayer: remove package Peter Korsgaard
2020-02-07 14:50 ` [Buildroot] [PATCH 06/10] package/opencv: drop gstreamer 0.10.x support Peter Korsgaard
2020-02-07 14:50 ` [Buildroot] [PATCH 07/10] package/opencv3: " Peter Korsgaard
2020-02-07 14:50 ` Peter Korsgaard [this message]
2020-02-07 14:50 ` [Buildroot] [PATCH 09/10] package/nvidia-tegra23-binaries: " Peter Korsgaard
2020-02-07 14:50 ` [Buildroot] [PATCH 10/10] package/gstreamer/*: remove packages Peter Korsgaard
2020-02-08 21:58 ` [Buildroot] [PATCH 00/10] Remove gstreamer 0.10 Yann E. MORIN
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20200207145026.28384-9-peter@korsgaard.com \
--to=peter@korsgaard.com \
--cc=buildroot@busybox.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.