diff mbox series

[2/2] package/libmediaart: Fix default symbol for media art backend

Message ID 20180509155146.25410-2-petr.vorel@gmail.com
State Superseded
Headers show
Series [1/2] toolchain/buildroot: Fix symbol for uClibc | expand

Commit Message

Petr Vorel May 9, 2018, 3:51 p.m. UTC
Symbol BR2_PACKAGE_LIBMEDIAART_BACKEND_NONE does not exist it was meant
to be BR2_PACKAGE_MEDIAART_BACKEND_NONE.

Fixes: c443830a57 libmediaart: new package

Signed-off-by: Petr Vorel <petr.vorel@gmail.com>
---
Error found while working with updated kconfig, which offers this
feature.
---
 package/libmediaart/Config.in | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Thomas Petazzoni May 9, 2018, 3:55 p.m. UTC | #1
Hello,

On Wed,  9 May 2018 17:51:46 +0200, Petr Vorel wrote:
> Symbol BR2_PACKAGE_LIBMEDIAART_BACKEND_NONE does not exist it was meant
> to be BR2_PACKAGE_MEDIAART_BACKEND_NONE.
> 
> Fixes: c443830a57 libmediaart: new package
> 
> Signed-off-by: Petr Vorel <petr.vorel@gmail.com>
> ---
> Error found while working with updated kconfig, which offers this
> feature.

Meh, in fact the real bug is that all those options should be prefixed
by BR2_PACKAGE_LIBMEDIAART and not BR2_PACKAGE_MEDIAART.

Could you fix that instead ? Config.in.legacy handling will however not
be possible, because those options are part of a Kconfig choice, and
you can't select an item of a choice.

Thomas
diff mbox series

Patch

diff --git a/package/libmediaart/Config.in b/package/libmediaart/Config.in
index e5e770d4fc..d3fea05ac4 100644
--- a/package/libmediaart/Config.in
+++ b/package/libmediaart/Config.in
@@ -18,7 +18,7 @@  if BR2_PACKAGE_LIBMEDIAART
 
 choice
 	prompt "media art backend"
-	default BR2_PACKAGE_LIBMEDIAART_BACKEND_NONE
+	default BR2_PACKAGE_MEDIAART_BACKEND_NONE
 
 config BR2_PACKAGE_MEDIAART_BACKEND_NONE
 	bool "none"