[Buildroot] [PATCH 7/8] libmediaart: rename options to have proper prefix
Yann E. MORIN
yann.morin.1998 at free.fr
Sun May 13 19:32:14 UTC 2018
Petr, Thomas, All,
On 2018-05-13 21:07 +0200, Thomas Petazzoni spake thusly:
> From: Petr Vorel <petr.vorel at gmail.com>
>
> Options should be prefixed by BR2_PACKAGE_LIBMEDIAART and not
> BR2_PACKAGE_MEDIAART, but package was using both prefixes.
>
> This was found as default symbol was defined as
> BR2_PACKAGE_LIBMEDIAART_BACKEND_NONE (correct prefix), but symbol
> was actually BR2_PACKAGE_MEDIAART_BACKEND_NONE).
>
> This commit therefore renames the incorrectly named options, and adds
> Config.in.legacy handling. Since the options are part of a choice, the
> legacy handling cannot select the new options, and is only here to
> inform the user.
>
> Fixes: c443830a57 libmediaart: new package
>
> Signed-off-by: Petr Vorel <petr.vorel at gmail.com>
> [Thomas: improve commit log, add Config.in.legacy handling]
> Signed-off-by: Thomas Petazzoni <thomas.petazzoni at bootlin.com>
Acked-by: "Yann E. MORIN" <yann.morin.1998 at free.fr>
Regards,
Yann E. MORIN.
> ---
> Config.in.legacy | 24 ++++++++++++++++++++++++
> package/libmediaart/Config.in | 8 ++++----
> package/libmediaart/libmediaart.mk | 6 +++---
> 3 files changed, 31 insertions(+), 7 deletions(-)
>
> diff --git a/Config.in.legacy b/Config.in.legacy
> index 5cf25463ad..8d946b6bed 100644
> --- a/Config.in.legacy
> +++ b/Config.in.legacy
> @@ -145,6 +145,30 @@ endif
> ###############################################################################
> comment "Legacy options removed in 2018.05"
>
> +config BR2_PACKAGE_MEDIAART_BACKEND_NONE
> + bool "libmediaart none backend option renamed"
> + select BR2_LEGACY
> + help
> + For consistency reasons, the option
> + BR2_PACKAGE_MEDIAART_BACKEND_NONE has been renamed to
> + BR2_PACKAGE_LIBMEDIAART_BACKEND_NONE
> +
> +config BR2_PACKAGE_MEDIAART_BACKEND_GDK_PIXBUF
> + bool "libmediaart gdk-pixbuf backend option renamed"
> + select BR2_LEGACY
> + help
> + For consistency reasons, the option
> + BR2_PACKAGE_MEDIAART_BACKEND_GDK_PIXBUF has been renamed to
> + BR2_PACKAGE_LIBMEDIAART_BACKEND_GDK_PIXBUF
> +
> +config BR2_PACKAGE_MEDIAART_BACKEND_GDK_PIXBUF
> + bool "libmediaart qt backend option renamed"
> + select BR2_LEGACY
> + help
> + For consistency reasons, the option
> + BR2_PACKAGE_MEDIAART_BACKEND_QT has been renamed to
> + BR2_PACKAGE_LIBMEDIAART_BACKEND_QT
> +
> config BR2_PACKAGE_TI_SGX_AM335X
> bool "ti-sgx-km AM335X option renamed"
> select BR2_LEGACY
> diff --git a/package/libmediaart/Config.in b/package/libmediaart/Config.in
> index e5e770d4fc..e073677395 100644
> --- a/package/libmediaart/Config.in
> +++ b/package/libmediaart/Config.in
> @@ -20,17 +20,17 @@ choice
> prompt "media art backend"
> default BR2_PACKAGE_LIBMEDIAART_BACKEND_NONE
>
> -config BR2_PACKAGE_MEDIAART_BACKEND_NONE
> +config BR2_PACKAGE_LIBMEDIAART_BACKEND_NONE
> bool "none"
> help
> With no backend, libmediaart will not be able to process MP3
> album art.
>
> -config BR2_PACKAGE_MEDIAART_BACKEND_GDK_PIXBUF
> +config BR2_PACKAGE_LIBMEDIAART_BACKEND_GDK_PIXBUF
> bool "gdk-pixbuf"
> select BR2_PACKAGE_GDK_PIXBUF
>
> -config BR2_PACKAGE_MEDIAART_BACKEND_QT
> +config BR2_PACKAGE_LIBMEDIAART_BACKEND_QT
> bool "Qt"
> depends on BR2_PACKAGE_QT || BR2_PACKAGE_QT5
> select BR2_PACKAGE_QT_GUI_MODULE if BR2_PACKAGE_QT
> @@ -41,4 +41,4 @@ comment "Qt backend depends on Qt or Qt5"
>
> endchoice
>
> -endif # BR2_PACKAGE_MEDIAART
> +endif # BR2_PACKAGE_LIBMEDIAART
> diff --git a/package/libmediaart/libmediaart.mk b/package/libmediaart/libmediaart.mk
> index db4f1fa2a1..f64210c266 100644
> --- a/package/libmediaart/libmediaart.mk
> +++ b/package/libmediaart/libmediaart.mk
> @@ -15,12 +15,12 @@ LIBMEDIAART_INSTALL_STAGING = YES
> LIBMEDIAART_DEPENDENCIES = libglib2
> LIBMEDIAART_CONF_OPTS = --disable-unit-tests
>
> -ifeq ($(BR2_PACKAGE_MEDIAART_BACKEND_GDK_PIXBUF),y)
> +ifeq ($(BR2_PACKAGE_LIBMEDIAART_BACKEND_GDK_PIXBUF),y)
> LIBMEDIAART_DEPENDENCIES += gdk-pixbuf
> LIBMEDIAART_CONF_OPTS += \
> --enable-gdkpixbuf \
> --disable-qt
> -else ifeq ($(BR2_PACKAGE_MEDIAART_BACKEND_QT),y)
> +else ifeq ($(BR2_PACKAGE_LIBMEDIAART_BACKEND_QT),y)
> # qt5 needs c++11 (since qt-5.7)
> ifeq ($(BR2_PACKAGE_QT5_VERSION_LATEST),y)
> LIBMEDIAART_CONF_ENV += CXXFLAGS="$(TARGET_CXXFLAGS) -std=c++11"
> @@ -31,7 +31,7 @@ LIBMEDIAART_DEPENDENCIES += \
> LIBMEDIAART_CONF_OPTS += \
> --disable-gdkpixbuf \
> --enable-qt
> -else ifeq ($(BR2_PACKAGE_MEDIAART_BACKEND_NONE),y)
> +else ifeq ($(BR2_PACKAGE_LIBMEDIAART_BACKEND_NONE),y)
> LIBMEDIAART_CONF_OPTS += \
> --disable-gdkpixbuf \
> --disable-qt
> --
> 2.14.3
>
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot
--
.-----------------.--------------------.------------------.--------------------.
| Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ |
| +33 223 225 172 `------------.-------: X AGAINST | \e/ There is no |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. |
'------------------------------^-------^------------------^--------------------'
More information about the buildroot
mailing list