gst1-plugins-bad: remove obsolete make rules

PATCH New Benoît Thébaudeau 2 participants 5 messages May 7, 2015 – May 28, 2015

Share & tools

Benoît Thébaudeau

author

2015-05-07 14:58 UTC #0 lore raw

PATCH 1/2 gst1-plugins-bad: remove obsolete make rules

Commit 83e29f2656a237e2c300d523776868837d3bc646 "gst1-plugins-bad: bump version" (from 1.2.4 to 1.4.1) removed the configuration options for the cdaudio and eglglessink plugins as a consequence of their removal in 1.3.1. However, that commit did not remove the corresponding make rules, so remove them now.

Signed-off-by Beno?t Th?baudeau <[email protected]>

---
 .../gst1-plugins-bad/gst1-plugins-bad.mk           | 24 ----------------------
 1 file changed, 24 deletions(-)

package/gstreamer1/gst1-plugins-bad/gst1-plugins-bad.mk +0 ~~-24~~

index 6280fd9..0e00ffd 100644
@@ -544,13 +544,6 @@ else
 GST1_PLUGINS_BAD_CONF_OPTS += --disable-bz2
 endif
 
-ifeq ($(BR2_PACKAGE_GST1_PLUGINS_BAD_PLUGIN_CDAUDIO),y)
-GST1_PLUGINS_BAD_CONF_OPTS += --enable-cdaudio
-GST1_PLUGINS_BAD_DEPENDENCIES += libcdaudio
-else
-GST1_PLUGINS_BAD_CONF_OPTS += --disable-cdaudio
-endif
-
 ifeq ($(BR2_PACKAGE_GST1_PLUGINS_BAD_PLUGIN_CURL),y)
 GST1_PLUGINS_BAD_CONF_OPTS += --enable-curl
 GST1_PLUGINS_BAD_DEPENDENCIES += libcurl

@@ -663,23 +656,6 @@ else
 GST1_PLUGINS_BAD_CONF_OPTS += --disable-rsvg
 endif
 
-ifeq ($(BR2_PACKAGE_GST1_PLUGINS_BAD_PLUGIN_EGLGLES),y)
-GST1_PLUGINS_BAD_CONF_OPTS += --enable-eglgles
-GST1_PLUGINS_BAD_DEPENDENCIES += libegl libgles
-
-ifeq ($(BR2_PACKAGE_RPI_USERLAND),y)
-# RPI has odd locations for several required headers.
-GST1_PLUGINS_BAD_CONF_OPTS += --with-egl-window-system=rpi
-GST1_PLUGINS_BAD_CONF_ENV += \
-	CFLAGS="$(TARGET_CFLAGS) \
-	-I$(STAGING_DIR)/usr/include/IL \
-	-I$(STAGING_DIR)/usr/include/interface/vcos/pthreads \
-	-I$(STAGING_DIR)/usr/include/interface/vmcs_host/linux"
-endif
-else
-GST1_PLUGINS_BAD_CONF_OPTS += --disable-eglgles
-endif
-
 ifeq ($(BR2_PACKAGE_GST1_PLUGINS_BAD_PLUGIN_SDL),y)
 GST1_PLUGINS_BAD_CONF_ENV += ac_cv_path_SDL_CONFIG=$(STAGING_DIR)/usr/bin/sdl-config
 GST1_PLUGINS_BAD_CONF_OPTS += --enable-sdl

Signature · 1 line

2.1.0

Benoît Thébaudeau

author

2015-05-07 14:58 UTC #1 lore raw

PATCH 2/2 gst1-plugins-bad: add support for opengl

Commit 83e29f2656a237e2c300d523776868837d3bc646 "gst1-plugins-bad: bump version" (from 1.2.4 to 1.4.1) removed the configuration option for the eglglessink plugin as a consequence of its removal in 1.3.1. However, that commit did not add support for glimagesink, the plugin element to be used as a replacement, which broke support for the EGL/GLES video sink feature in Buildroot.

This new commit adds support for the OpenGL library and the corresponding gl plugin, including the glimagesink element among others.

The plugin needs the library, but the library can also be used separately in order to link custom plugins or applications against it.

The library needs at least one OpenGL API, one OpenGL platform and one compatible window system. The implementations of these requirements that Buildroot can currently provide are:
- APIs: OpenGL, OpenGL ES 2.X,
- platforms: GLX, EGL,
- window systems: X11, Wayland, Dispmanx.

The wayland configure feature switch is shared between the corresponding OpenGL library window system and the waylandsink plugin.

This has been tested with videotestsrc/omxmpeg4videodec/omxh264dec and glimagesink on the Raspberry Pi, using OpenGL ES 2.X, EGL and Dispmanx.

Signed-off-by Beno?t Th?baudeau <[email protected]>

---
 package/gstreamer1/gst1-plugins-bad/Config.in      | 104 +++++++++++++++++++++
 .../gst1-plugins-bad/gst1-plugins-bad.mk           |  73 +++++++++++++--
 2 files changed, 170 insertions(+), 7 deletions(-)

package/gstreamer1/gst1-plugins-bad/Config.in +104 -0

index 44fcdfb..39d0817 100644
@@ -9,6 +9,103 @@ menuconfig BR2_PACKAGE_GST1_PLUGINS_BAD
 
 if BR2_PACKAGE_GST1_PLUGINS_BAD
 
+comment "libraries with external dependencies"
+
+menu "OpenGL"
+
+config BR2_PACKAGE_GST1_PLUGINS_BAD_HAS_LIB_OPENGL
+	def_bool BR2_PACKAGE_GST1_PLUGINS_BAD_LIB_OPENGL_HAS_WINDOW
+
+menu "APIs"
+
+config BR2_PACKAGE_GST1_PLUGINS_BAD_LIB_OPENGL_HAS_API
+	bool
+
+config BR2_PACKAGE_GST1_PLUGINS_BAD_LIB_OPENGL_OPENGL
+	bool "opengl"
+	depends on BR2_PACKAGE_HAS_LIBGL
+	select BR2_PACKAGE_LIBGLU
+	select BR2_PACKAGE_GST1_PLUGINS_BAD_LIB_OPENGL_HAS_API
+
+config BR2_PACKAGE_GST1_PLUGINS_BAD_LIB_OPENGL_GLES2
+	bool "gles2"
+	depends on BR2_PACKAGE_HAS_LIBGLES
+	select BR2_PACKAGE_GST1_PLUGINS_BAD_LIB_OPENGL_HAS_API
+
+endmenu
+
+menu "Platforms"
+depends on BR2_PACKAGE_GST1_PLUGINS_BAD_LIB_OPENGL_HAS_API
+
+config BR2_PACKAGE_GST1_PLUGINS_BAD_LIB_OPENGL_HAS_PLATFORM
+	bool
+
+config BR2_PACKAGE_GST1_PLUGINS_BAD_LIB_OPENGL_GLX
+	bool "glx"
+	depends on BR2_PACKAGE_GST1_PLUGINS_BAD_LIB_OPENGL_OPENGL
+	depends on BR2_PACKAGE_XSERVER_XORG_SERVER_MODULAR # mesa needs big X
+	select BR2_PACKAGE_XLIB_LIBX11
+	select BR2_PACKAGE_XLIB_LIBXRENDER
+	select BR2_PACKAGE_XLIB_LIBXEXT
+	select BR2_PACKAGE_XPROTO_GLPROTO
+	select BR2_PACKAGE_GST1_PLUGINS_BAD_LIB_OPENGL_HAS_PLATFORM
+	help
+	  OpenGL Extension to the X Window System
+
+config BR2_PACKAGE_GST1_PLUGINS_BAD_LIB_OPENGL_EGL
+	bool "egl"
+	depends on BR2_PACKAGE_HAS_LIBEGL
+	select BR2_PACKAGE_GST1_PLUGINS_BAD_LIB_OPENGL_HAS_PLATFORM
+
+endmenu
+
+comment "platforms need an API"
+	depends on !BR2_PACKAGE_GST1_PLUGINS_BAD_LIB_OPENGL_HAS_API
+
+menu "Window Systems"
+depends on BR2_PACKAGE_GST1_PLUGINS_BAD_LIB_OPENGL_HAS_PLATFORM
+
+config BR2_PACKAGE_GST1_PLUGINS_BAD_LIB_OPENGL_HAS_WINDOW
+	bool
+
+config BR2_PACKAGE_GST1_PLUGINS_BAD_LIB_OPENGL_X11
+	bool "x11"
+	depends on !BR2_PACKAGE_RPI_USERLAND
+	depends on BR2_PACKAGE_XSERVER_XORG_SERVER_MODULAR # mesa needs big X
+	select BR2_PACKAGE_XLIB_LIBX11
+	select BR2_PACKAGE_XLIB_LIBXRENDER
+	select BR2_PACKAGE_XLIB_LIBXEXT
+	select BR2_PACKAGE_XPROTO_GLPROTO
+	select BR2_PACKAGE_GST1_PLUGINS_BAD_LIB_OPENGL_HAS_WINDOW
+
+if BR2_PACKAGE_GST1_PLUGINS_BAD_LIB_OPENGL_EGL
+
+config BR2_PACKAGE_GST1_PLUGINS_BAD_LIB_OPENGL_WAYLAND
+	bool "wayland"
+	depends on !BR2_STATIC_LIBS # wayland
+	depends on BR2_TOOLCHAIN_HAS_THREADS # wayland
+	select BR2_PACKAGE_WAYLAND
+	select BR2_PACKAGE_GST1_PLUGINS_BAD_LIB_OPENGL_HAS_WINDOW
+
+config BR2_PACKAGE_GST1_PLUGINS_BAD_LIB_OPENGL_DISPMANX
+	bool "dispmanx"
+	depends on BR2_PACKAGE_RPI_USERLAND
+	select BR2_PACKAGE_GST1_PLUGINS_BAD_LIB_OPENGL_HAS_WINDOW
+	help
+	  Raspberry Pi's Dispmanx windowing system
+
+endif
+
+endmenu
+
+comment "window systems need a platform"
+	depends on !BR2_PACKAGE_GST1_PLUGINS_BAD_LIB_OPENGL_HAS_PLATFORM
+
+comment "opengl needs an API, a platform and a window system"
+	depends on !BR2_PACKAGE_GST1_PLUGINS_BAD_HAS_LIB_OPENGL
+
+endmenu
+
 comment "dependency-less plugins"
 
 config BR2_PACKAGE_GST1_PLUGINS_BAD_PLUGIN_ACCURIP

@@ -469,6 +566,13 @@ comment "rsvg plugin needs a toolchain w/ C++"
 	depends on !BR2_INSTALL_LIBSTDCPP
 	depends on BR2_ARCH_HAS_ATOMICS
 
+config BR2_PACKAGE_GST1_PLUGINS_BAD_PLUGIN_GL
+	bool "gl"
+	depends on BR2_PACKAGE_GST1_PLUGINS_BAD_HAS_LIB_OPENGL
+
+comment "gl needs the gst1-plugins-bad opengl library"
+	depends on !BR2_PACKAGE_GST1_PLUGINS_BAD_HAS_LIB_OPENGL
+
 config BR2_PACKAGE_GST1_PLUGINS_BAD_PLUGIN_SDL
 	bool "sdl"
 	select BR2_PACKAGE_SDL

package/gstreamer1/gst1-plugins-bad/gst1-plugins-bad.mk +66 -7

91 lines not on this page · raw message

Signature · 1 line

2.1.0

Peter Korsgaard

2015-05-17 21:20 UTC #2 lore raw

PATCH 2/2 gst1-plugins-bad: add support for opengl

Quoted 22 lines

Quoted 1 line
Quoted 1 line
Quoted 1 line
Quoted 1 line
"Beno?t" == Beno?t Th?baudeau <[email protected]> writes:
Commit 83e29f2656a237e2c300d523776868837d3bc646 "gst1-plugins-bad: bump version" (from 1.2.4 to 1.4.1) removed the configuration option for the eglglessink plugin as a consequence of its removal in 1.3.1. However, that commit did not add support for glimagesink, the plugin element to be used as a replacement, which broke support for the EGL/GLES video sink feature in Buildroot.
This new commit adds support for the OpenGL library and the corresponding gl plugin, including the glimagesink element among others.
The plugin needs the library, but the library can also be used separately in order to link custom plugins or applications against it.
The library needs at least one OpenGL API, one OpenGL platform and one compatible window system. The implementations of these requirements that Buildroot can currently provide are:
- APIs: OpenGL, OpenGL ES 2.X,
- platforms: GLX, EGL,
- window systems: X11, Wayland, Dispmanx.

Quoted 5 lines

The wayland configure feature switch is shared between the corresponding OpenGL library window system and the waylandsink plugin.
This has been tested with videotestsrc/omxmpeg4videodec/omxh264dec and glimagesink on the Raspberry Pi, using OpenGL ES 2.X, EGL and Dispmanx.

First of all, thanks! Sorry, I haven't had time to review and test this completely, but here's a bit of feedback already.

package/gstreamer1/gst1-plugins-bad/Config.in menuconfig BR2_PACKAGE_GST1_PLUGINS_BAD · line 14

Quoted 3 lines

package/gstreamer1/gst1-plugins-bad/Config.in +3 -0
+comment "libraries with external dependencies"
+
+menu "OpenGL"

Perhaps this should depend on BR2_PACKAGE_HAS_LIBGL || BR2_PACKAGE_HAS_LIBGLES to not clutter the screen when it cannot do anything anyway?

Perhaps it should be a menuconfig instead so that there's a corresponding kconfig symbol to easily enable/disable it?

package/gstreamer1/gst1-plugins-bad/Config.in menuconfig BR2_PACKAGE_GST1_PLUGINS_BAD · line 19

Quoted 5 lines

package/gstreamer1/gst1-plugins-bad/Config.in +5 -0

+
+config BR2_PACKAGE_GST1_PLUGINS_BAD_HAS_LIB_OPENGL
+	def_bool BR2_PACKAGE_GST1_PLUGINS_BAD_LIB_OPENGL_HAS_WINDOW
+
+menu "APIs"

There is a lot of cascading menus which makes it hard to get an overview. Perhaps they should be replaced by comments instead (E.G. comment "APIs")?

Quoted 4 lines

package/gstreamer1/gst1-plugins-bad/Config.in +4 -0
+
+config BR2_PACKAGE_GST1_PLUGINS_BAD_LIB_OPENGL_HAS_API
+	bool
+

package/gstreamer1/gst1-plugins-bad/Config.in menuconfig BR2_PACKAGE_GST1_PLUGINS_BAD · line 33

Quoted 10 lines

package/gstreamer1/gst1-plugins-bad/Config.in +10 -0

+config BR2_PACKAGE_GST1_PLUGINS_BAD_LIB_OPENGL_OPENGL
+	bool "opengl"
+	depends on BR2_PACKAGE_HAS_LIBGL
+	select BR2_PACKAGE_LIBGLU
+	select BR2_PACKAGE_GST1_PLUGINS_BAD_LIB_OPENGL_HAS_API
+
+config BR2_PACKAGE_GST1_PLUGINS_BAD_LIB_OPENGL_GLES2
+	bool "gles2"
+	depends on BR2_PACKAGE_HAS_LIBGLES
+	select BR2_PACKAGE_GST1_PLUGINS_BAD_LIB_OPENGL_HAS_API

Probably these should be default y so they get enabled if the dependencies are present, as that is most likely what the user wants.

Same comments for platforms and window systems.

package/gstreamer1/gst1-plugins-bad/Config.in comment "rsvg plugin needs a toolchain w/ C++" · line 571

Quoted 3 lines

package/gstreamer1/gst1-plugins-bad/Config.in +3 -0

+config BR2_PACKAGE_GST1_PLUGINS_BAD_PLUGIN_GL
+	bool "gl"
+	depends on BR2_PACKAGE_GST1_PLUGINS_BAD_HAS_LIB_OPENGL

Likewise this should probably be default y.

package/gstreamer1/gst1-plugins-bad/gst1-plugins-bad.mk GST1_PLUGINS_BAD_CONF_OPTS += \ · line 99

Quoted 3 lines

package/gstreamer1/gst1-plugins-bad/gst1-plugins-bad.mk +3 -0

+ifeq ($(BR2_PACKAGE_GST1_PLUGINS_BAD_LIB_OPENGL_OPENGL),y)
+GST1_PLUGINS_BAD_CONF_OPTS += --enable-opengl
+GST1_PLUGINS_BAD_DEPENDENCIES += libgl

I think you are missing libglu here.

package/gstreamer1/gst1-plugins-bad/gst1-plugins-bad.mk GST1_PLUGINS_BAD_CONF_OPTS += \ · line 114

Quoted 4 lines

package/gstreamer1/gst1-plugins-bad/gst1-plugins-bad.mk +4 -0

+ifeq ($(BR2_PACKAGE_GST1_PLUGINS_BAD_LIB_OPENGL_GLX),y)
+GST1_PLUGINS_BAD_CONF_OPTS += --enable-glx
+GST1_PLUGINS_BAD_DEPENDENCIES += xproto_glproto xlib_libX11 xlib_libXrender \
+					xlib_libXext libeet

libeet? What does that have to do with opengl?

package/gstreamer1/gst1-plugins-bad/gst1-plugins-bad.mk GST1_PLUGINS_BAD_CONF_OPTS += \ · line 129

Quoted 4 lines

package/gstreamer1/gst1-plugins-bad/gst1-plugins-bad.mk +4 -0

+ifeq ($(BR2_PACKAGE_GST1_PLUGINS_BAD_LIB_OPENGL_X11),y)
+GST1_PLUGINS_BAD_CONF_OPTS += --enable-x11
+GST1_PLUGINS_BAD_DEPENDENCIES += xproto_glproto xlib_libX11 xlib_libXrender \
+					xlib_libXext libeet

As GLX implies X11 and the dependencies are the same, perhaps we should just handle them together? (E.G. move the enable-x11 / disable-x11 up to the glx option).

Signature · 1 line

Bye, Peter Korsgaard

Benoît Thébaudeau

author

2015-05-28 14:45 UTC #3 lore raw

PATCH 2/2 gst1-plugins-bad: add support for opengl

Dear Peter Korsgaard,

Quoted 22 lines · Peter Korsgaard

Quoted 21 lines
Quoted 1 line
Quoted 1 line
Quoted 1 line
Quoted 1 line
"Beno?t" == Beno?t Th?baudeau <[email protected]> writes:
Commit 83e29f2656a237e2c300d523776868837d3bc646 "gst1-plugins-bad: bump version" (from 1.2.4 to 1.4.1) removed the configuration option for the eglglessink plugin as a consequence of its removal in 1.3.1. However, that commit did not add support for glimagesink, the plugin element to be used as a replacement, which broke support for the EGL/GLES video sink feature in Buildroot.
This new commit adds support for the OpenGL library and the corresponding gl plugin, including the glimagesink element among others.
The plugin needs the library, but the library can also be used separately in order to link custom plugins or applications against it.
The library needs at least one OpenGL API, one OpenGL platform and one compatible window system. The implementations of these requirements that Buildroot can currently provide are:
- APIs: OpenGL, OpenGL ES 2.X,
- platforms: GLX, EGL,
- window systems: X11, Wayland, Dispmanx.

Quoted 8 lines

Quoted 5 lines
The wayland configure feature switch is shared between the corresponding OpenGL library window system and the waylandsink plugin.
This has been tested with videotestsrc/omxmpeg4videodec/omxh264dec and glimagesink on the Raspberry Pi, using OpenGL ES 2.X, EGL and Dispmanx.
First of all, thanks! Sorry, I haven't had time to review and test this completely, but here's a bit of feedback already.

You're welcome. Don't worry. Sorry for the late reply.

Quoted 7 lines

package/gstreamer1/gst1-plugins-bad/Config.in menuconfig BR2_PACKAGE_GST1_PLUGINS_BAD · line 14
Quoted 3 lines
package/gstreamer1/gst1-plugins-bad/Config.in +3 -0
+comment "libraries with external dependencies"
+
+menu "OpenGL"
Perhaps this should depend on BR2_PACKAGE_HAS_LIBGL || BR2_PACKAGE_HAS_LIBGLES to not clutter the screen when it cannot do anything anyway?

Yes, changed in v2.

Quoted 2 lines

Perhaps it should be a menuconfig instead so that there's a corresponding kconfig symbol to easily enable/disable it?

Yes, changed in v2.

Quoted 9 lines

package/gstreamer1/gst1-plugins-bad/Config.in menuconfig BR2_PACKAGE_GST1_PLUGINS_BAD · line 19
Quoted 5 lines
package/gstreamer1/gst1-plugins-bad/Config.in +5 -0
+
+config BR2_PACKAGE_GST1_PLUGINS_BAD_HAS_LIB_OPENGL
+	def_bool BR2_PACKAGE_GST1_PLUGINS_BAD_LIB_OPENGL_HAS_WINDOW
+
+menu "APIs"
There is a lot of cascading menus which makes it hard to get an overview. Perhaps they should be replaced by comments instead (E.G. comment "APIs")?

Yes, changed in v2. In order to keep the overview idea, I've also added comments for hidden options rather than hiding all platforms / window systems under a single comment if unavailable. This is also more helpful for users to know what is doable with this library.

Quoted 18 lines

Quoted 4 lines
package/gstreamer1/gst1-plugins-bad/Config.in +4 -0
+
+config BR2_PACKAGE_GST1_PLUGINS_BAD_LIB_OPENGL_HAS_API
+	bool
+
package/gstreamer1/gst1-plugins-bad/Config.in menuconfig BR2_PACKAGE_GST1_PLUGINS_BAD · line 33
Quoted 10 lines
package/gstreamer1/gst1-plugins-bad/Config.in +10 -0
+config BR2_PACKAGE_GST1_PLUGINS_BAD_LIB_OPENGL_OPENGL
+	bool "opengl"
+	depends on BR2_PACKAGE_HAS_LIBGL
+	select BR2_PACKAGE_LIBGLU
+	select BR2_PACKAGE_GST1_PLUGINS_BAD_LIB_OPENGL_HAS_API
+
+config BR2_PACKAGE_GST1_PLUGINS_BAD_LIB_OPENGL_GLES2
+	bool "gles2"
+	depends on BR2_PACKAGE_HAS_LIBGLES
+	select BR2_PACKAGE_GST1_PLUGINS_BAD_LIB_OPENGL_HAS_API
Probably these should be default y so they get enabled if the dependencies are present, as that is most likely what the user wants.

Quoted 1 line

Same comments for platforms and window systems.

Yes, changed in v2. For consistency of the OpenGL options, I've also changed wayland into a dependency for the window system, rather than selecting it. Otherwise, this would have been confusing, and making it "default y" would have been too much.

Quoted 5 lines

package/gstreamer1/gst1-plugins-bad/Config.in comment "rsvg plugin needs a toolchain w/ C++" · line 571
Quoted 3 lines
package/gstreamer1/gst1-plugins-bad/Config.in +3 -0
+config BR2_PACKAGE_GST1_PLUGINS_BAD_PLUGIN_GL
+	bool "gl"
+	depends on BR2_PACKAGE_GST1_PLUGINS_BAD_HAS_LIB_OPENGL
Likewise this should probably be default y.

Yes, changed in v2.

Quoted 5 lines

package/gstreamer1/gst1-plugins-bad/gst1-plugins-bad.mk GST1_PLUGINS_BAD_CONF_OPTS += \ · line 99
Quoted 3 lines
package/gstreamer1/gst1-plugins-bad/gst1-plugins-bad.mk +3 -0
+ifeq ($(BR2_PACKAGE_GST1_PLUGINS_BAD_LIB_OPENGL_OPENGL),y)
+GST1_PLUGINS_BAD_CONF_OPTS += --enable-opengl
+GST1_PLUGINS_BAD_DEPENDENCIES += libgl
I think you are missing libglu here.

Yes, changed in v2.

Quoted 6 lines

package/gstreamer1/gst1-plugins-bad/gst1-plugins-bad.mk GST1_PLUGINS_BAD_CONF_OPTS += \ · line 114
Quoted 4 lines
package/gstreamer1/gst1-plugins-bad/gst1-plugins-bad.mk +4 -0
+ifeq ($(BR2_PACKAGE_GST1_PLUGINS_BAD_LIB_OPENGL_GLX),y)
+GST1_PLUGINS_BAD_CONF_OPTS += --enable-glx
+GST1_PLUGINS_BAD_DEPENDENCIES += xproto_glproto xlib_libX11 xlib_libXrender \
+					xlib_libXext libeet
libeet? What does that have to do with opengl?

Nothing, removed in v2. It was a leftover of a copy from libevas.

Quoted 8 lines

package/gstreamer1/gst1-plugins-bad/gst1-plugins-bad.mk GST1_PLUGINS_BAD_CONF_OPTS += \ · line 129
Quoted 4 lines
package/gstreamer1/gst1-plugins-bad/gst1-plugins-bad.mk +4 -0
+ifeq ($(BR2_PACKAGE_GST1_PLUGINS_BAD_LIB_OPENGL_X11),y)
+GST1_PLUGINS_BAD_CONF_OPTS += --enable-x11
+GST1_PLUGINS_BAD_DEPENDENCIES += xproto_glproto xlib_libX11 xlib_libXrender \
+					xlib_libXext libeet
As GLX implies X11 and the dependencies are the same, perhaps we should just handle them together? (E.G. move the enable-x11 / disable-x11 up to the glx option).

I've reworked the GLX/X11 options and dependencies in v2. The dependencies for X11 are now lighter, and GLX selects X11.

Note that I have not tested the X11 and Wayland paths other than in menuconfig (i.e. not built, not run). I didn't know how the sub-packages of X.org are organized, so I've tried to figure out the requirements from the gst1-plugins-bad configure script and other packages using X11 with OpenGL. I've not found real-world defconfigs to test these cases, but if you give me pointers, I can launch some builds.

Best regards,
Beno?t

Peter Korsgaard

2015-05-17 21:05 UTC #4 lore raw

PATCH 1/2 gst1-plugins-bad: remove obsolete make rules

In reply to Benoît Thébaudeau

Quoted 9 lines

Quoted 1 line
Quoted 1 line
Quoted 1 line
Quoted 1 line
"Beno?t" == Beno?t Th?baudeau <[email protected]> writes:
Commit 83e29f2656a237e2c300d523776868837d3bc646 "gst1-plugins-bad: bump version" (from 1.2.4 to 1.4.1) removed the configuration options for the cdaudio and eglglessink plugins as a consequence of their removal in 1.3.1. However, that commit did not remove the corresponding make rules, so remove them now.
Signed-off-by Beno?t Th?baudeau <[email protected]>

Committed to next, thanks.

Signature · 1 line

Bye, Peter Korsgaard