* [Buildroot] [PATCH v2 1/2] toolchain/buildroot: Fix symbol for uClibc
@ 2018-05-09 16:33 Petr Vorel
2018-05-09 16:33 ` [Buildroot] [PATCH v2 2/2] package/libmediaart: Fix config prefix Petr Vorel
` (2 more replies)
0 siblings, 3 replies; 5+ messages in thread
From: Petr Vorel @ 2018-05-09 16:33 UTC (permalink / raw)
To: buildroot
Symbol BR2_TOOLCHAIN_UCLIBC doesn't exist, it was meant to be
BR2_TOOLCHAIN_BUILDROOT_UCLIBC.
Fixes: 493b1177b9 toolchain/buildroot: default to uClibc
Signed-off-by: Petr Vorel <petr.vorel@gmail.com>
---
toolchain/toolchain-buildroot/Config.in | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/toolchain/toolchain-buildroot/Config.in b/toolchain/toolchain-buildroot/Config.in
index 2f6624d217..75e8191f46 100644
--- a/toolchain/toolchain-buildroot/Config.in
+++ b/toolchain/toolchain-buildroot/Config.in
@@ -22,7 +22,7 @@ config BR2_TOOLCHAIN_BUILDROOT_VENDOR
choice
prompt "C library"
- default BR2_TOOLCHAIN_UCLIBC
+ default BR2_TOOLCHAIN_BUILDROOT_UCLIBC
default BR2_TOOLCHAIN_BUILDROOT_GLIBC if BR2_powerpc64
config BR2_TOOLCHAIN_BUILDROOT_UCLIBC
--
2.16.3
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [Buildroot] [PATCH v2 2/2] package/libmediaart: Fix config prefix
2018-05-09 16:33 [Buildroot] [PATCH v2 1/2] toolchain/buildroot: Fix symbol for uClibc Petr Vorel
@ 2018-05-09 16:33 ` Petr Vorel
2018-05-13 19:19 ` [Buildroot] [PATCH v2 1/2] toolchain/buildroot: Fix symbol for uClibc Thomas Petazzoni
2018-05-13 20:10 ` Thomas Petazzoni
2 siblings, 0 replies; 5+ messages in thread
From: Petr Vorel @ 2018-05-09 16:33 UTC (permalink / raw)
To: buildroot
Options should be prefixed by BR2_PACKAGE_LIBMEDIAART and not
BR2_PACKAGE_MEDIAART, but package was using both prefixes.
We cannot put old options into Config.in.legacy, because those
options are part of a Kconfig choice, and it's not possible to
select an item of a choice.
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).
Fixes: c443830a57 libmediaart: new package
Signed-off-by: Petr Vorel <petr.vorel@gmail.com>
---
Changes v1->v2:
* Change package prefix to BR2_PACKAGE_LIBMEDIAART as
BR2_PACKAGE_MEDIAART is wrong (asked by Thomas).
Kind regards,
Petr
---
package/libmediaart/Config.in | 8 ++++----
package/libmediaart/libmediaart.mk | 6 +++---
2 files changed, 7 insertions(+), 7 deletions(-)
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.16.3
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [Buildroot] [PATCH v2 1/2] toolchain/buildroot: Fix symbol for uClibc
2018-05-09 16:33 [Buildroot] [PATCH v2 1/2] toolchain/buildroot: Fix symbol for uClibc Petr Vorel
2018-05-09 16:33 ` [Buildroot] [PATCH v2 2/2] package/libmediaart: Fix config prefix Petr Vorel
@ 2018-05-13 19:19 ` Thomas Petazzoni
2018-05-14 4:09 ` Ricardo Martincoski
2018-05-13 20:10 ` Thomas Petazzoni
2 siblings, 1 reply; 5+ messages in thread
From: Thomas Petazzoni @ 2018-05-13 19:19 UTC (permalink / raw)
To: buildroot
Ricardo,
On Wed, 9 May 2018 18:33:22 +0200, Petr Vorel wrote:
> Symbol BR2_TOOLCHAIN_UCLIBC doesn't exist, it was meant to be
> BR2_TOOLCHAIN_BUILDROOT_UCLIBC.
>
> Fixes: 493b1177b9 toolchain/buildroot: default to uClibc
>
> Signed-off-by: Petr Vorel <petr.vorel@gmail.com>
> ---
> toolchain/toolchain-buildroot/Config.in | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/toolchain/toolchain-buildroot/Config.in b/toolchain/toolchain-buildroot/Config.in
> index 2f6624d217..75e8191f46 100644
> --- a/toolchain/toolchain-buildroot/Config.in
> +++ b/toolchain/toolchain-buildroot/Config.in
> @@ -22,7 +22,7 @@ config BR2_TOOLCHAIN_BUILDROOT_VENDOR
>
> choice
> prompt "C library"
> - default BR2_TOOLCHAIN_UCLIBC
> + default BR2_TOOLCHAIN_BUILDROOT_UCLIBC
Just like I added a checker in check-package to verify automatically
the issue raised by PATCH 2/2 of Petr's series [1], I wanted to add a
checker to detect the problem raised by this patch.
My idea was to have something that:
(1) Collects options defined by "config BR2_<something>"
(2) Collects references to option, in "select", "depends on",
"default" clauses, etc.
(3) Ensures that all references to options have a matching option
definition.
However, this requires a global pass on all Config.in files to do (1)
and (2) before doing (3), and I'm not sure how this fits in the current
check-package logic, which seems very per-file oriented.
What do you think about this ? Should this be done as part of
check-package, or a separate checker ?
Best regards,
Thomas
[1] http://patchwork.ozlabs.org/patch/910991/
--
Thomas Petazzoni, CTO, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
https://bootlin.com
^ permalink raw reply [flat|nested] 5+ messages in thread
* [Buildroot] [PATCH v2 1/2] toolchain/buildroot: Fix symbol for uClibc
2018-05-09 16:33 [Buildroot] [PATCH v2 1/2] toolchain/buildroot: Fix symbol for uClibc Petr Vorel
2018-05-09 16:33 ` [Buildroot] [PATCH v2 2/2] package/libmediaart: Fix config prefix Petr Vorel
2018-05-13 19:19 ` [Buildroot] [PATCH v2 1/2] toolchain/buildroot: Fix symbol for uClibc Thomas Petazzoni
@ 2018-05-13 20:10 ` Thomas Petazzoni
2 siblings, 0 replies; 5+ messages in thread
From: Thomas Petazzoni @ 2018-05-13 20:10 UTC (permalink / raw)
To: buildroot
Hello,
On Wed, 9 May 2018 18:33:22 +0200, Petr Vorel wrote:
> Symbol BR2_TOOLCHAIN_UCLIBC doesn't exist, it was meant to be
> BR2_TOOLCHAIN_BUILDROOT_UCLIBC.
>
> Fixes: 493b1177b9 toolchain/buildroot: default to uClibc
>
> Signed-off-by: Petr Vorel <petr.vorel@gmail.com>
> ---
> toolchain/toolchain-buildroot/Config.in | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
Applied to master, thanks.
Thomas
--
Thomas Petazzoni, CTO, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
https://bootlin.com
^ permalink raw reply [flat|nested] 5+ messages in thread
* [Buildroot] [PATCH v2 1/2] toolchain/buildroot: Fix symbol for uClibc
2018-05-13 19:19 ` [Buildroot] [PATCH v2 1/2] toolchain/buildroot: Fix symbol for uClibc Thomas Petazzoni
@ 2018-05-14 4:09 ` Ricardo Martincoski
0 siblings, 0 replies; 5+ messages in thread
From: Ricardo Martincoski @ 2018-05-14 4:09 UTC (permalink / raw)
To: buildroot
Hello,
On Sun, May 13, 2018 at 04:19 PM, Thomas Petazzoni wrote:
> On Wed, 9 May 2018 18:33:22 +0200, Petr Vorel wrote:
>> - default BR2_TOOLCHAIN_UCLIBC
>> + default BR2_TOOLCHAIN_BUILDROOT_UCLIBC
>
> Just like I added a checker in check-package to verify automatically
> the issue raised by PATCH 2/2 of Petr's series [1], I wanted to add a
> checker to detect the problem raised by this patch.
>
> My idea was to have something that:
>
> (1) Collects options defined by "config BR2_<something>"
>
> (2) Collects references to option, in "select", "depends on",
> "default" clauses, etc.
>
> (3) Ensures that all references to options have a matching option
> definition.
>
> However, this requires a global pass on all Config.in files to do (1)
> and (2) before doing (3), and I'm not sure how this fits in the current
> check-package logic, which seems very per-file oriented.
>
> What do you think about this ? Should this be done as part of
> check-package, or a separate checker ?
A separate checker seems more suitable in this case.
Since it will find global inconsistencies it creates two large databases to be
compared against each other, so running this when creating or changing any
package may be too much.
But it seems nice to have it as another job on Gitlab to warn us as soon an
inconsistency is detected.
And when people change a package with lots of dependencies it can be run
locally.
Regards,
Ricardo
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2018-05-14 4:09 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-05-09 16:33 [Buildroot] [PATCH v2 1/2] toolchain/buildroot: Fix symbol for uClibc Petr Vorel
2018-05-09 16:33 ` [Buildroot] [PATCH v2 2/2] package/libmediaart: Fix config prefix Petr Vorel
2018-05-13 19:19 ` [Buildroot] [PATCH v2 1/2] toolchain/buildroot: Fix symbol for uClibc Thomas Petazzoni
2018-05-14 4:09 ` Ricardo Martincoski
2018-05-13 20:10 ` Thomas Petazzoni
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.