* [Buildroot] [PATCH] mysql: propagate common dependencies to toplevel config
@ 2017-01-09 10:56 Peter Korsgaard
2017-01-09 12:16 ` Baruch Siach
0 siblings, 1 reply; 3+ messages in thread
From: Peter Korsgaard @ 2017-01-09 10:56 UTC (permalink / raw)
To: buildroot
Commit 3d707d2b (mysql: rename package to oracle-mysql, make a virtual
package) introduced a user selectable virtual BR2_PACKAGE_MYSQL package, but
didn't propagate the (common) dependencies of the two variants to it, so the
virtual package can now be selected even though neither of the variants are
available.
As several packages enable mysql support when BR2_PACKAGE_MYSQL is selected,
this causes a number of autobuilder issues:
http://autobuild.buildroot.net/results/7fe/7fe0d0a3e7ed0430852dc42b718dd037557207e8/
http://autobuild.buildroot.net/results/cc4/cc4c2d936f3e1ba6c0a9782b2218de54a4ff75d2/
Fix it by propagating the common dependencies of the two variants to the
virtual package to ensure it cannot be enabled unless at least one of them
are available.
Also move the toolchain comment outside the conditional so it is visible
when mysql isn't available.
Signed-off-by: Peter Korsgaard <peter@korsgaard.com>
---
package/mysql/Config.in | 22 +++++++---------------
1 file changed, 7 insertions(+), 15 deletions(-)
diff --git a/package/mysql/Config.in b/package/mysql/Config.in
index e485d6b99..69ad58654 100644
--- a/package/mysql/Config.in
+++ b/package/mysql/Config.in
@@ -1,5 +1,8 @@
config BR2_PACKAGE_MYSQL
bool "mysql support"
+ depends on BR2_INSTALL_LIBSTDCPP
+ depends on BR2_USE_MMU # fork()
+ depends on BR2_TOOLCHAIN_HAS_THREADS
help
Select the desired mysql provider.
@@ -13,9 +16,6 @@ choice
config BR2_PACKAGE_MARIADB
bool "mariadb"
- depends on BR2_INSTALL_LIBSTDCPP
- depends on BR2_USE_MMU # fork()
- depends on BR2_TOOLCHAIN_HAS_THREADS
depends on BR2_PACKAGE_LIBAIO_ARCH_SUPPORTS
select BR2_PACKAGE_LIBAIO
select BR2_PACKAGE_LIBXML2
@@ -32,9 +32,6 @@ config BR2_PACKAGE_MARIADB
config BR2_PACKAGE_ORACLE_MYSQL
bool "oracle mysql"
- depends on BR2_INSTALL_LIBSTDCPP
- depends on BR2_USE_MMU # fork()
- depends on BR2_TOOLCHAIN_HAS_THREADS
select BR2_PACKAGE_NCURSES
select BR2_PACKAGE_READLINE
select BR2_PACKAGE_HAS_MYSQL
@@ -45,15 +42,6 @@ config BR2_PACKAGE_ORACLE_MYSQL
endchoice
-comment "mariadb needs a toolchain w/ C++, threads"
- depends on BR2_USE_MMU
- depends on BR2_PACKAGE_LIBAIO_ARCH_SUPPORTS
- depends on !BR2_INSTALL_LIBSTDCPP || !BR2_TOOLCHAIN_HAS_THREADS
-
-comment "oracle mysql needs a toolchain w/ C++, threads"
- depends on BR2_USE_MMU
- depends on !BR2_INSTALL_LIBSTDCPP || !BR2_TOOLCHAIN_HAS_THREADS
-
if BR2_PACKAGE_MARIADB
config BR2_PACKAGE_MARIADB_SERVER
@@ -81,3 +69,7 @@ config BR2_PACKAGE_PROVIDES_MYSQL
default "oracle-mysql" if BR2_PACKAGE_ORACLE_MYSQL
endif
+
+comment "mysql needs a toolchain w/ C++, threads"
+ depends on BR2_USE_MMU
+ depends on !BR2_INSTALL_LIBSTDCPP || !BR2_TOOLCHAIN_HAS_THREADS
--
2.11.0
^ permalink raw reply related [flat|nested] 3+ messages in thread
* [Buildroot] [PATCH] mysql: propagate common dependencies to toplevel config
2017-01-09 10:56 [Buildroot] [PATCH] mysql: propagate common dependencies to toplevel config Peter Korsgaard
@ 2017-01-09 12:16 ` Baruch Siach
2017-01-09 12:43 ` Peter Korsgaard
0 siblings, 1 reply; 3+ messages in thread
From: Baruch Siach @ 2017-01-09 12:16 UTC (permalink / raw)
To: buildroot
Hi Peter,
On Mon, Jan 09, 2017 at 11:56:54AM +0100, Peter Korsgaard wrote:
> Commit 3d707d2b (mysql: rename package to oracle-mysql, make a virtual
> package) introduced a user selectable virtual BR2_PACKAGE_MYSQL package, but
> didn't propagate the (common) dependencies of the two variants to it, so the
> virtual package can now be selected even though neither of the variants are
> available.
>
> As several packages enable mysql support when BR2_PACKAGE_MYSQL is selected,
> this causes a number of autobuilder issues:
>
> http://autobuild.buildroot.net/results/7fe/7fe0d0a3e7ed0430852dc42b718dd037557207e8/
> http://autobuild.buildroot.net/results/cc4/cc4c2d936f3e1ba6c0a9782b2218de54a4ff75d2/
>
> Fix it by propagating the common dependencies of the two variants to the
> virtual package to ensure it cannot be enabled unless at least one of them
> are available.
>
> Also move the toolchain comment outside the conditional so it is visible
> when mysql isn't available.
>
> Signed-off-by: Peter Korsgaard <peter@korsgaard.com>
This is similar to http://patchwork.ozlabs.org/patch/709670/, but more
comprehensive. Looks good to me.
baruch
> ---
> package/mysql/Config.in | 22 +++++++---------------
> 1 file changed, 7 insertions(+), 15 deletions(-)
>
> diff --git a/package/mysql/Config.in b/package/mysql/Config.in
> index e485d6b99..69ad58654 100644
> --- a/package/mysql/Config.in
> +++ b/package/mysql/Config.in
> @@ -1,5 +1,8 @@
> config BR2_PACKAGE_MYSQL
> bool "mysql support"
> + depends on BR2_INSTALL_LIBSTDCPP
> + depends on BR2_USE_MMU # fork()
> + depends on BR2_TOOLCHAIN_HAS_THREADS
> help
> Select the desired mysql provider.
>
> @@ -13,9 +16,6 @@ choice
>
> config BR2_PACKAGE_MARIADB
> bool "mariadb"
> - depends on BR2_INSTALL_LIBSTDCPP
> - depends on BR2_USE_MMU # fork()
> - depends on BR2_TOOLCHAIN_HAS_THREADS
> depends on BR2_PACKAGE_LIBAIO_ARCH_SUPPORTS
> select BR2_PACKAGE_LIBAIO
> select BR2_PACKAGE_LIBXML2
> @@ -32,9 +32,6 @@ config BR2_PACKAGE_MARIADB
>
> config BR2_PACKAGE_ORACLE_MYSQL
> bool "oracle mysql"
> - depends on BR2_INSTALL_LIBSTDCPP
> - depends on BR2_USE_MMU # fork()
> - depends on BR2_TOOLCHAIN_HAS_THREADS
> select BR2_PACKAGE_NCURSES
> select BR2_PACKAGE_READLINE
> select BR2_PACKAGE_HAS_MYSQL
> @@ -45,15 +42,6 @@ config BR2_PACKAGE_ORACLE_MYSQL
>
> endchoice
>
> -comment "mariadb needs a toolchain w/ C++, threads"
> - depends on BR2_USE_MMU
> - depends on BR2_PACKAGE_LIBAIO_ARCH_SUPPORTS
> - depends on !BR2_INSTALL_LIBSTDCPP || !BR2_TOOLCHAIN_HAS_THREADS
> -
> -comment "oracle mysql needs a toolchain w/ C++, threads"
> - depends on BR2_USE_MMU
> - depends on !BR2_INSTALL_LIBSTDCPP || !BR2_TOOLCHAIN_HAS_THREADS
> -
> if BR2_PACKAGE_MARIADB
>
> config BR2_PACKAGE_MARIADB_SERVER
> @@ -81,3 +69,7 @@ config BR2_PACKAGE_PROVIDES_MYSQL
> default "oracle-mysql" if BR2_PACKAGE_ORACLE_MYSQL
>
> endif
> +
> +comment "mysql needs a toolchain w/ C++, threads"
> + depends on BR2_USE_MMU
> + depends on !BR2_INSTALL_LIBSTDCPP || !BR2_TOOLCHAIN_HAS_THREADS
> --
> 2.11.0
>
> _______________________________________________
> buildroot mailing list
> buildroot at busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot
--
http://baruch.siach.name/blog/ ~. .~ Tk Open Systems
=}------------------------------------------------ooO--U--Ooo------------{=
- baruch at tkos.co.il - tel: +972.52.368.4656, http://www.tkos.co.il -
^ permalink raw reply [flat|nested] 3+ messages in thread
* [Buildroot] [PATCH] mysql: propagate common dependencies to toplevel config
2017-01-09 12:16 ` Baruch Siach
@ 2017-01-09 12:43 ` Peter Korsgaard
0 siblings, 0 replies; 3+ messages in thread
From: Peter Korsgaard @ 2017-01-09 12:43 UTC (permalink / raw)
To: buildroot
>>>>> "Baruch" == Baruch Siach <baruch@tkos.co.il> writes:
> Hi Peter,
> On Mon, Jan 09, 2017 at 11:56:54AM +0100, Peter Korsgaard wrote:
>> Commit 3d707d2b (mysql: rename package to oracle-mysql, make a virtual
>> package) introduced a user selectable virtual BR2_PACKAGE_MYSQL package, but
>> didn't propagate the (common) dependencies of the two variants to it, so the
>> virtual package can now be selected even though neither of the variants are
>> available.
>>
>> As several packages enable mysql support when BR2_PACKAGE_MYSQL is selected,
>> this causes a number of autobuilder issues:
>>
>> http://autobuild.buildroot.net/results/7fe/7fe0d0a3e7ed0430852dc42b718dd037557207e8/
>> http://autobuild.buildroot.net/results/cc4/cc4c2d936f3e1ba6c0a9782b2218de54a4ff75d2/
>>
>> Fix it by propagating the common dependencies of the two variants to the
>> virtual package to ensure it cannot be enabled unless at least one of them
>> are available.
>>
>> Also move the toolchain comment outside the conditional so it is visible
>> when mysql isn't available.
>>
>> Signed-off-by: Peter Korsgaard <peter@korsgaard.com>
> This is similar to http://patchwork.ozlabs.org/patch/709670/, but more
> comprehensive. Looks good to me.
Ups, I somehow missed your patch - Sorry about that!
Committed, thanks.
--
Bye, Peter Korsgaard
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2017-01-09 12:43 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-01-09 10:56 [Buildroot] [PATCH] mysql: propagate common dependencies to toplevel config Peter Korsgaard
2017-01-09 12:16 ` Baruch Siach
2017-01-09 12:43 ` Peter Korsgaard
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.