diff mbox

[v8,26/28] xbmc: Add option for Goom screensaver

Message ID 1400342276-10303-27-git-send-email-bernd.kuhls@t-online.de
State Superseded
Headers show

Commit Message

Bernd Kuhls May 17, 2014, 3:57 p.m. UTC
Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
---
 package/xbmc/Config.in |   11 +++++++++++
 package/xbmc/xbmc.mk   |    6 ++++++
 2 files changed, 17 insertions(+)

Comments

Yann E. MORIN May 17, 2014, 9:04 p.m. UTC | #1
Bernd, All,

On 2014-05-17 17:57 +0200, Bernd Kuhls spake thusly:
> Signed-off-by: Bernd Kuhls <bernd.kuhls@t-online.de>
> ---
>  package/xbmc/Config.in |   11 +++++++++++
>  package/xbmc/xbmc.mk   |    6 ++++++
>  2 files changed, 17 insertions(+)
> 
> diff --git a/package/xbmc/Config.in b/package/xbmc/Config.in
> index 40a1f63..4b5276c 100644
> --- a/package/xbmc/Config.in
> +++ b/package/xbmc/Config.in
> @@ -131,6 +131,17 @@ config BR2_PACKAGE_XBMC_LIBMICROHTTPD
>  	help
>  	  Enable webserver feature
>  
> +config BR2_PACKAGE_XBMC_GOOM
> +	bool "goom screensaver"
> +	depends on BR2_i386 || BR2_x86_64 # it does not make sense to use goom on embedded archs

Why does it not make sense?

Surely, if an ARM platform has a full openGL implementation, what would
prevent the user to run the Goom screen saver at all?

Remove the depedns on x86.

> +	depends on BR2_PACKAGE_HAS_LIBGL
> +	help
> +	  Enable goom screensaver
> +
> +comment "goom needs an OpenGL backend"
> +	depends on BR2_i386 || BR2_x86_64

Ditto, dependency on HAS_LIBGL should be enough.

Regards,
Yann E. MORIN.

> +	depends on !BR2_PACKAGE_HAS_LIBGL
> +
>  config BR2_PACKAGE_XBMC_LIBNFS
>  	bool "nfs"
>  	depends on BR2_TOOLCHAIN_HAS_NATIVE_RPC
> diff --git a/package/xbmc/xbmc.mk b/package/xbmc/xbmc.mk
> index 361380d..a18b82a 100644
> --- a/package/xbmc/xbmc.mk
> +++ b/package/xbmc/xbmc.mk
> @@ -110,6 +110,12 @@ else
>  XBMC_CONF_OPT += --disable-libmp3lame
>  endif
>  
> +ifeq ($(BR2_PACKAGE_XBMC_GOOM),y)
> +XBMC_CONF_OPT += --enable-goom
> +else
> +XBMC_CONF_OPT += --disable-goom
> +endif
> +
>  ifeq ($(BR2_PACKAGE_XBMC_LIBUSB),y)
>  XBMC_DEPENDENCIES += libusb-compat
>  XBMC_CONF_OPT += --enable-libusb
> -- 
> 1.7.10.4
> 
> _______________________________________________
> buildroot mailing list
> buildroot@busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot
diff mbox

Patch

diff --git a/package/xbmc/Config.in b/package/xbmc/Config.in
index 40a1f63..4b5276c 100644
--- a/package/xbmc/Config.in
+++ b/package/xbmc/Config.in
@@ -131,6 +131,17 @@  config BR2_PACKAGE_XBMC_LIBMICROHTTPD
 	help
 	  Enable webserver feature
 
+config BR2_PACKAGE_XBMC_GOOM
+	bool "goom screensaver"
+	depends on BR2_i386 || BR2_x86_64 # it does not make sense to use goom on embedded archs
+	depends on BR2_PACKAGE_HAS_LIBGL
+	help
+	  Enable goom screensaver
+
+comment "goom needs an OpenGL backend"
+	depends on BR2_i386 || BR2_x86_64
+	depends on !BR2_PACKAGE_HAS_LIBGL
+
 config BR2_PACKAGE_XBMC_LIBNFS
 	bool "nfs"
 	depends on BR2_TOOLCHAIN_HAS_NATIVE_RPC
diff --git a/package/xbmc/xbmc.mk b/package/xbmc/xbmc.mk
index 361380d..a18b82a 100644
--- a/package/xbmc/xbmc.mk
+++ b/package/xbmc/xbmc.mk
@@ -110,6 +110,12 @@  else
 XBMC_CONF_OPT += --disable-libmp3lame
 endif
 
+ifeq ($(BR2_PACKAGE_XBMC_GOOM),y)
+XBMC_CONF_OPT += --enable-goom
+else
+XBMC_CONF_OPT += --disable-goom
+endif
+
 ifeq ($(BR2_PACKAGE_XBMC_LIBUSB),y)
 XBMC_DEPENDENCIES += libusb-compat
 XBMC_CONF_OPT += --enable-libusb