* [Buildroot] [PATCH 1/1] package/qt5enginio: needs ssl @ 2019-03-09 22:29 Fabrice Fontaine 2019-03-12 22:29 ` Thomas Petazzoni 0 siblings, 1 reply; 8+ messages in thread From: Fabrice Fontaine @ 2019-03-09 22:29 UTC (permalink / raw) To: buildroot Remove select on BR2_PACKAGE_QT5BASE and BR2_PACKAGE_OPENSSL to avoid a circular dependency Fixes: - http://autobuild.buildroot.org/results/60678cab68ec9aa17184b8417b64b3b79adf428a Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com> --- package/qt5/qt5enginio/Config.in | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/package/qt5/qt5enginio/Config.in b/package/qt5/qt5enginio/Config.in index 102058e7c5..cce0f2a1cb 100644 --- a/package/qt5/qt5enginio/Config.in +++ b/package/qt5/qt5enginio/Config.in @@ -1,7 +1,6 @@ config BR2_PACKAGE_QT5ENGINIO bool "qt5enginio" - select BR2_PACKAGE_OPENSSL - select BR2_PACKAGE_QT5BASE + depends on BR2_PACKAGE_QT5BASE_OPENSSL select BR2_PACKAGE_QT5BASE_GUI select BR2_PACKAGE_QT5BASE_NETWORK help -- 2.20.1 ^ permalink raw reply related [flat|nested] 8+ messages in thread
* [Buildroot] [PATCH 1/1] package/qt5enginio: needs ssl 2019-03-09 22:29 [Buildroot] [PATCH 1/1] package/qt5enginio: needs ssl Fabrice Fontaine @ 2019-03-12 22:29 ` Thomas Petazzoni 2019-03-12 23:06 ` Fabrice Fontaine 0 siblings, 1 reply; 8+ messages in thread From: Thomas Petazzoni @ 2019-03-12 22:29 UTC (permalink / raw) To: buildroot Hello Fabrice, On Sat, 9 Mar 2019 23:29:53 +0100 Fabrice Fontaine <fontaine.fabrice@gmail.com> wrote: > Remove select on BR2_PACKAGE_QT5BASE and BR2_PACKAGE_OPENSSL to avoid a > circular dependency I don't really understand where the "circular dependency" is. To me it seems like the problem is rather: """ Qt 5.6 SSL support is no longer available with OpenSSL since the bump of OpenSSL to 1.1.x. Therefore selecting Qt5 and OpenSSL does not guarantee that qt5base will be built with SSL support. Due to this, the build of qt5enginio currently fails on Qt 5.6. """ > diff --git a/package/qt5/qt5enginio/Config.in b/package/qt5/qt5enginio/Config.in > index 102058e7c5..cce0f2a1cb 100644 > --- a/package/qt5/qt5enginio/Config.in > +++ b/package/qt5/qt5enginio/Config.in > @@ -1,7 +1,6 @@ > config BR2_PACKAGE_QT5ENGINIO > bool "qt5enginio" > - select BR2_PACKAGE_OPENSSL > - select BR2_PACKAGE_QT5BASE > + depends on BR2_PACKAGE_QT5BASE_OPENSSL Either we do this, or: select BR2_PACKAGE_OPENSSL if BR2_PACKAGE_QT5_VERSION_LATEST select BR2_PACKAGE_LIBRESSL if BR2_PACKAGE_QT5_VERSION_5_6 Best regards, Thomas -- Thomas Petazzoni, CTO, Bootlin Embedded Linux and Kernel engineering https://bootlin.com ^ permalink raw reply [flat|nested] 8+ messages in thread
* [Buildroot] [PATCH 1/1] package/qt5enginio: needs ssl 2019-03-12 22:29 ` Thomas Petazzoni @ 2019-03-12 23:06 ` Fabrice Fontaine 2019-03-26 22:52 ` Arnout Vandecappelle 0 siblings, 1 reply; 8+ messages in thread From: Fabrice Fontaine @ 2019-03-12 23:06 UTC (permalink / raw) To: buildroot Hello Thomas, Le mar. 12 mars 2019 ? 23:29, Thomas Petazzoni <thomas.petazzoni@bootlin.com> a ?crit : > > Hello Fabrice, > > On Sat, 9 Mar 2019 23:29:53 +0100 > Fabrice Fontaine <fontaine.fabrice@gmail.com> wrote: > > > Remove select on BR2_PACKAGE_QT5BASE and BR2_PACKAGE_OPENSSL to avoid a > > circular dependency > > I don't really understand where the "circular dependency" is. > > To me it seems like the problem is rather: > > """ > Qt 5.6 SSL support is no longer available with OpenSSL since the bump > of OpenSSL to 1.1.x. Therefore selecting Qt5 and OpenSSL does not > guarantee that qt5base will be built with SSL support. Due to this, the > build of qt5enginio currently fails on Qt 5.6. > """ > > > diff --git a/package/qt5/qt5enginio/Config.in b/package/qt5/qt5enginio/Config.in > > index 102058e7c5..cce0f2a1cb 100644 > > --- a/package/qt5/qt5enginio/Config.in > > +++ b/package/qt5/qt5enginio/Config.in > > @@ -1,7 +1,6 @@ > > config BR2_PACKAGE_QT5ENGINIO > > bool "qt5enginio" > > - select BR2_PACKAGE_OPENSSL > > - select BR2_PACKAGE_QT5BASE > > + depends on BR2_PACKAGE_QT5BASE_OPENSSL > > Either we do this, or: > > select BR2_PACKAGE_OPENSSL if BR2_PACKAGE_QT5_VERSION_LATEST > select BR2_PACKAGE_LIBRESSL if BR2_PACKAGE_QT5_VERSION_5_6 I tried this, but I assume that I must add the dependencies of libressl if QT 5.6 is true, for example like this: depends on !BR2_PACKAGE_QT5_VERSION_5_6 || \ (BR2_PACKAGE_QT5_VERSION_5_6 && \ (!BR2_PACKAGE_OPENSSL_FORCE_LIBOPENSSL && \ !(BR2_TOOLCHAIN_USES_UCLIBC && !BR2_USE_MMU))) But then, this creates the following circular dependency: package/sqlcipher/Config.in:1:error: recursive dependency detected! package/sqlcipher/Config.in:1: symbol BR2_PACKAGE_SQLCIPHER depends on BR2_PACKAGE_SQLITE package/sqlite/Config.in:1: symbol BR2_PACKAGE_SQLITE is selected by BR2_PACKAGE_QT5BASE_SQLITE_SYSTEM package/qt5/qt5base/Config.in:104: symbol BR2_PACKAGE_QT5BASE_SQLITE_SYSTEM is part of choice <choice> package/qt5/qt5base/Config.in:88: choice <choice> contains symbol <choice> package/qt5/qt5base/Config.in:88: choice <choice> contains symbol BR2_PACKAGE_QT5BASE package/qt5/qt5base/Config.in:1: symbol BR2_PACKAGE_QT5BASE is selected by BR2_PACKAGE_QT5ENGINIO package/qt5/qt5enginio/Config.in:1: symbol BR2_PACKAGE_QT5ENGINIO depends on BR2_PACKAGE_OPENSSL_FORCE_LIBOPENSSL package/openssl/Config.in:1: symbol BR2_PACKAGE_OPENSSL_FORCE_LIBOPENSSL is selected by BR2_PACKAGE_SQLCIPHER Did I made a mistake? Shouldn't we add the libressl dependencies? > > Best regards, > > Thomas > -- > Thomas Petazzoni, CTO, Bootlin > Embedded Linux and Kernel engineering > https://bootlin.com Best Regards, Fabrice ^ permalink raw reply [flat|nested] 8+ messages in thread
* [Buildroot] [PATCH 1/1] package/qt5enginio: needs ssl 2019-03-12 23:06 ` Fabrice Fontaine @ 2019-03-26 22:52 ` Arnout Vandecappelle 2019-03-27 18:08 ` Fabrice Fontaine 0 siblings, 1 reply; 8+ messages in thread From: Arnout Vandecappelle @ 2019-03-26 22:52 UTC (permalink / raw) To: buildroot On 13/03/2019 00:06, Fabrice Fontaine wrote: > Hello Thomas, > Le mar. 12 mars 2019 ? 23:29, Thomas Petazzoni > <thomas.petazzoni@bootlin.com> a ?crit : >> >> Hello Fabrice, >> >> On Sat, 9 Mar 2019 23:29:53 +0100 >> Fabrice Fontaine <fontaine.fabrice@gmail.com> wrote: >> >>> Remove select on BR2_PACKAGE_QT5BASE and BR2_PACKAGE_OPENSSL to avoid a >>> circular dependency >> >> I don't really understand where the "circular dependency" is. >> >> To me it seems like the problem is rather: >> >> """ >> Qt 5.6 SSL support is no longer available with OpenSSL since the bump >> of OpenSSL to 1.1.x. Therefore selecting Qt5 and OpenSSL does not >> guarantee that qt5base will be built with SSL support. Due to this, the >> build of qt5enginio currently fails on Qt 5.6. >> """ >> >>> diff --git a/package/qt5/qt5enginio/Config.in b/package/qt5/qt5enginio/Config.in >>> index 102058e7c5..cce0f2a1cb 100644 >>> --- a/package/qt5/qt5enginio/Config.in >>> +++ b/package/qt5/qt5enginio/Config.in >>> @@ -1,7 +1,6 @@ >>> config BR2_PACKAGE_QT5ENGINIO >>> bool "qt5enginio" >>> - select BR2_PACKAGE_OPENSSL >>> - select BR2_PACKAGE_QT5BASE >>> + depends on BR2_PACKAGE_QT5BASE_OPENSSL >> >> Either we do this, or: >> >> select BR2_PACKAGE_OPENSSL if BR2_PACKAGE_QT5_VERSION_LATEST >> select BR2_PACKAGE_LIBRESSL if BR2_PACKAGE_QT5_VERSION_5_6 > I tried this, but I assume that I must add the dependencies of > libressl if QT 5.6 is true, for example like this: > > depends on !BR2_PACKAGE_QT5_VERSION_5_6 || \ > (BR2_PACKAGE_QT5_VERSION_5_6 && \ > (!BR2_PACKAGE_OPENSSL_FORCE_LIBOPENSSL && \ > !(BR2_TOOLCHAIN_USES_UCLIBC && !BR2_USE_MMU))) > > But then, this creates the following circular dependency: > > package/sqlcipher/Config.in:1:error: recursive dependency detected! > package/sqlcipher/Config.in:1: symbol BR2_PACKAGE_SQLCIPHER depends on > BR2_PACKAGE_SQLITE > package/sqlite/Config.in:1: symbol BR2_PACKAGE_SQLITE is selected by > BR2_PACKAGE_QT5BASE_SQLITE_SYSTEM > package/qt5/qt5base/Config.in:104: symbol > BR2_PACKAGE_QT5BASE_SQLITE_SYSTEM is part of choice <choice> > package/qt5/qt5base/Config.in:88: choice <choice> contains symbol <choice> > package/qt5/qt5base/Config.in:88: choice <choice> contains symbol > BR2_PACKAGE_QT5BASE > package/qt5/qt5base/Config.in:1: symbol BR2_PACKAGE_QT5BASE is > selected by BR2_PACKAGE_QT5ENGINIO > package/qt5/qt5enginio/Config.in:1: symbol BR2_PACKAGE_QT5ENGINIO > depends on BR2_PACKAGE_OPENSSL_FORCE_LIBOPENSSL > package/openssl/Config.in:1: symbol > BR2_PACKAGE_OPENSSL_FORCE_LIBOPENSSL is selected by > BR2_PACKAGE_SQLCIPHER > > Did I made a mistake? Shouldn't we add the libressl dependencies? Whaa, kconfig... How about just depending on QT5_LATEST || BR2_PACKAGE_QT5BASE_OPENSSL? Or does that also give a circular dependency? Regards, Arnout >> >> Best regards, >> >> Thomas >> -- >> Thomas Petazzoni, CTO, Bootlin >> Embedded Linux and Kernel engineering >> https://bootlin.com > Best Regards, > > Fabrice > _______________________________________________ > buildroot mailing list > buildroot at busybox.net > http://lists.busybox.net/mailman/listinfo/buildroot > ^ permalink raw reply [flat|nested] 8+ messages in thread
* [Buildroot] [PATCH 1/1] package/qt5enginio: needs ssl 2019-03-26 22:52 ` Arnout Vandecappelle @ 2019-03-27 18:08 ` Fabrice Fontaine 2019-04-27 22:33 ` Arnout Vandecappelle 0 siblings, 1 reply; 8+ messages in thread From: Fabrice Fontaine @ 2019-03-27 18:08 UTC (permalink / raw) To: buildroot Dear Arnout, Le mar. 26 mars 2019 ? 23:52, Arnout Vandecappelle <arnout@mind.be> a ?crit : > > > > On 13/03/2019 00:06, Fabrice Fontaine wrote: > > Hello Thomas, > > Le mar. 12 mars 2019 ? 23:29, Thomas Petazzoni > > <thomas.petazzoni@bootlin.com> a ?crit : > >> > >> Hello Fabrice, > >> > >> On Sat, 9 Mar 2019 23:29:53 +0100 > >> Fabrice Fontaine <fontaine.fabrice@gmail.com> wrote: > >> > >>> Remove select on BR2_PACKAGE_QT5BASE and BR2_PACKAGE_OPENSSL to avoid a > >>> circular dependency > >> > >> I don't really understand where the "circular dependency" is. > >> > >> To me it seems like the problem is rather: > >> > >> """ > >> Qt 5.6 SSL support is no longer available with OpenSSL since the bump > >> of OpenSSL to 1.1.x. Therefore selecting Qt5 and OpenSSL does not > >> guarantee that qt5base will be built with SSL support. Due to this, the > >> build of qt5enginio currently fails on Qt 5.6. > >> """ > >> > >>> diff --git a/package/qt5/qt5enginio/Config.in b/package/qt5/qt5enginio/Config.in > >>> index 102058e7c5..cce0f2a1cb 100644 > >>> --- a/package/qt5/qt5enginio/Config.in > >>> +++ b/package/qt5/qt5enginio/Config.in > >>> @@ -1,7 +1,6 @@ > >>> config BR2_PACKAGE_QT5ENGINIO > >>> bool "qt5enginio" > >>> - select BR2_PACKAGE_OPENSSL > >>> - select BR2_PACKAGE_QT5BASE > >>> + depends on BR2_PACKAGE_QT5BASE_OPENSSL > >> > >> Either we do this, or: > >> > >> select BR2_PACKAGE_OPENSSL if BR2_PACKAGE_QT5_VERSION_LATEST > >> select BR2_PACKAGE_LIBRESSL if BR2_PACKAGE_QT5_VERSION_5_6 > > I tried this, but I assume that I must add the dependencies of > > libressl if QT 5.6 is true, for example like this: > > > > depends on !BR2_PACKAGE_QT5_VERSION_5_6 || \ > > (BR2_PACKAGE_QT5_VERSION_5_6 && \ > > (!BR2_PACKAGE_OPENSSL_FORCE_LIBOPENSSL && \ > > !(BR2_TOOLCHAIN_USES_UCLIBC && !BR2_USE_MMU))) > > > > But then, this creates the following circular dependency: > > > > package/sqlcipher/Config.in:1:error: recursive dependency detected! > > package/sqlcipher/Config.in:1: symbol BR2_PACKAGE_SQLCIPHER depends on > > BR2_PACKAGE_SQLITE > > package/sqlite/Config.in:1: symbol BR2_PACKAGE_SQLITE is selected by > > BR2_PACKAGE_QT5BASE_SQLITE_SYSTEM > > package/qt5/qt5base/Config.in:104: symbol > > BR2_PACKAGE_QT5BASE_SQLITE_SYSTEM is part of choice <choice> > > package/qt5/qt5base/Config.in:88: choice <choice> contains symbol <choice> > > package/qt5/qt5base/Config.in:88: choice <choice> contains symbol > > BR2_PACKAGE_QT5BASE > > package/qt5/qt5base/Config.in:1: symbol BR2_PACKAGE_QT5BASE is > > selected by BR2_PACKAGE_QT5ENGINIO > > package/qt5/qt5enginio/Config.in:1: symbol BR2_PACKAGE_QT5ENGINIO > > depends on BR2_PACKAGE_OPENSSL_FORCE_LIBOPENSSL > > package/openssl/Config.in:1: symbol > > BR2_PACKAGE_OPENSSL_FORCE_LIBOPENSSL is selected by > > BR2_PACKAGE_SQLCIPHER > > > > Did I made a mistake? Shouldn't we add the libressl dependencies? > > Whaa, kconfig... > > How about just depending on QT5_LATEST || BR2_PACKAGE_QT5BASE_OPENSSL? Or does > that also give a circular dependency? Yes, adding the select suggested by Thomas and the above dependencies also give a circular dependency. The issue is that we can't select BR2_PACKAGE_LIBRESSL without adding a circular dependency because of the BR2_PACKAGE_OPENSSL_FORCE_LIBOPENSSL option that can be selected by any package that can't work with libressl. Currently, this option is selected by 4 packages: sqlcipher, wpa_supplicant, tpm2-tss and softether. So until this option is removed (if the above packages can work with libressl), I think my current patch is the only solution. > > Regards, > Arnout > > > >> > >> Best regards, > >> > >> Thomas > >> -- > >> Thomas Petazzoni, CTO, Bootlin > >> Embedded Linux and Kernel engineering > >> https://bootlin.com > > Best Regards, > > > > Fabrice > > _______________________________________________ > > buildroot mailing list > > buildroot at busybox.net > > http://lists.busybox.net/mailman/listinfo/buildroot > > Best Regards, Fabrice ^ permalink raw reply [flat|nested] 8+ messages in thread
* [Buildroot] [PATCH 1/1] package/qt5enginio: needs ssl 2019-03-27 18:08 ` Fabrice Fontaine @ 2019-04-27 22:33 ` Arnout Vandecappelle 2019-04-28 9:21 ` Fabrice Fontaine 0 siblings, 1 reply; 8+ messages in thread From: Arnout Vandecappelle @ 2019-04-27 22:33 UTC (permalink / raw) To: buildroot On 27/03/2019 19:08, Fabrice Fontaine wrote: > Dear Arnout, > > Le mar. 26 mars 2019 ? 23:52, Arnout Vandecappelle <arnout@mind.be> a ?crit : >> >> >> >> On 13/03/2019 00:06, Fabrice Fontaine wrote: >>> Hello Thomas, >>> Le mar. 12 mars 2019 ? 23:29, Thomas Petazzoni >>> <thomas.petazzoni@bootlin.com> a ?crit : >>>> >>>> Hello Fabrice, >>>> >>>> On Sat, 9 Mar 2019 23:29:53 +0100 >>>> Fabrice Fontaine <fontaine.fabrice@gmail.com> wrote: >>>> >>>>> Remove select on BR2_PACKAGE_QT5BASE and BR2_PACKAGE_OPENSSL to avoid a >>>>> circular dependency >>>> >>>> I don't really understand where the "circular dependency" is. >>>> >>>> To me it seems like the problem is rather: >>>> >>>> """ >>>> Qt 5.6 SSL support is no longer available with OpenSSL since the bump >>>> of OpenSSL to 1.1.x. Therefore selecting Qt5 and OpenSSL does not >>>> guarantee that qt5base will be built with SSL support. Due to this, the >>>> build of qt5enginio currently fails on Qt 5.6. >>>> """ >>>> >>>>> diff --git a/package/qt5/qt5enginio/Config.in b/package/qt5/qt5enginio/Config.in >>>>> index 102058e7c5..cce0f2a1cb 100644 >>>>> --- a/package/qt5/qt5enginio/Config.in >>>>> +++ b/package/qt5/qt5enginio/Config.in >>>>> @@ -1,7 +1,6 @@ >>>>> config BR2_PACKAGE_QT5ENGINIO >>>>> bool "qt5enginio" >>>>> - select BR2_PACKAGE_OPENSSL >>>>> - select BR2_PACKAGE_QT5BASE >>>>> + depends on BR2_PACKAGE_QT5BASE_OPENSSL >>>> >>>> Either we do this, or: >>>> >>>> select BR2_PACKAGE_OPENSSL if BR2_PACKAGE_QT5_VERSION_LATEST >>>> select BR2_PACKAGE_LIBRESSL if BR2_PACKAGE_QT5_VERSION_5_6 >>> I tried this, but I assume that I must add the dependencies of >>> libressl if QT 5.6 is true, for example like this: >>> >>> depends on !BR2_PACKAGE_QT5_VERSION_5_6 || \ >>> (BR2_PACKAGE_QT5_VERSION_5_6 && \ >>> (!BR2_PACKAGE_OPENSSL_FORCE_LIBOPENSSL && \ >>> !(BR2_TOOLCHAIN_USES_UCLIBC && !BR2_USE_MMU))) >>> >>> But then, this creates the following circular dependency: >>> >>> package/sqlcipher/Config.in:1:error: recursive dependency detected! >>> package/sqlcipher/Config.in:1: symbol BR2_PACKAGE_SQLCIPHER depends on >>> BR2_PACKAGE_SQLITE >>> package/sqlite/Config.in:1: symbol BR2_PACKAGE_SQLITE is selected by >>> BR2_PACKAGE_QT5BASE_SQLITE_SYSTEM >>> package/qt5/qt5base/Config.in:104: symbol >>> BR2_PACKAGE_QT5BASE_SQLITE_SYSTEM is part of choice <choice> >>> package/qt5/qt5base/Config.in:88: choice <choice> contains symbol <choice> >>> package/qt5/qt5base/Config.in:88: choice <choice> contains symbol >>> BR2_PACKAGE_QT5BASE >>> package/qt5/qt5base/Config.in:1: symbol BR2_PACKAGE_QT5BASE is >>> selected by BR2_PACKAGE_QT5ENGINIO >>> package/qt5/qt5enginio/Config.in:1: symbol BR2_PACKAGE_QT5ENGINIO >>> depends on BR2_PACKAGE_OPENSSL_FORCE_LIBOPENSSL >>> package/openssl/Config.in:1: symbol >>> BR2_PACKAGE_OPENSSL_FORCE_LIBOPENSSL is selected by >>> BR2_PACKAGE_SQLCIPHER >>> >>> Did I made a mistake? Shouldn't we add the libressl dependencies? >> >> Whaa, kconfig... >> >> How about just depending on QT5_LATEST || BR2_PACKAGE_QT5BASE_OPENSSL? Or does >> that also give a circular dependency? > Yes, adding the select suggested by Thomas and the above dependencies > also give a circular dependency. > The issue is that we can't select BR2_PACKAGE_LIBRESSL without adding > a circular dependency because of the > BR2_PACKAGE_OPENSSL_FORCE_LIBOPENSSL option that can be selected by > any package that can't work with libressl. > Currently, this option is selected by 4 packages: sqlcipher, > wpa_supplicant, tpm2-tss and softether. > So until this option is removed (if the above packages can work with > libressl), I think my current patch is the only solution. I think I got it to work like this: # This is a hack to avoid circular dependencies when selecting OPENSSL config BR2_PACKAGE_QT5ENGINIO_SELECT_OPENSSL bool default y if BR2_PACKAGE_QT5ENGINIO select BR2_PACKAGE_OPENSSL config BR2_PACKAGE_QT5ENGINIO bool "qt5enginio" depends on !BR2_PACKAGE_QT5_VERSION_5_6 || \ (BR2_PACKAGE_QT5_VERSION_5_6 && \ (!BR2_PACKAGE_OPENSSL_FORCE_LIBOPENSSL && \ !(BR2_TOOLCHAIN_USES_UCLIBC && !BR2_USE_MMU))) select BR2_PACKAGE_QT5BASE_GUI select BR2_PACKAGE_QT5BASE_NETWORK Could you give that a go? Regards, Arnout ^ permalink raw reply [flat|nested] 8+ messages in thread
* [Buildroot] [PATCH 1/1] package/qt5enginio: needs ssl 2019-04-27 22:33 ` Arnout Vandecappelle @ 2019-04-28 9:21 ` Fabrice Fontaine 2019-04-29 9:01 ` Arnout Vandecappelle 0 siblings, 1 reply; 8+ messages in thread From: Fabrice Fontaine @ 2019-04-28 9:21 UTC (permalink / raw) To: buildroot Hello Arnout, Le dim. 28 avr. 2019 ? 00:33, Arnout Vandecappelle <arnout@mind.be> a ?crit : > > > > On 27/03/2019 19:08, Fabrice Fontaine wrote: > > Dear Arnout, > > > > Le mar. 26 mars 2019 ? 23:52, Arnout Vandecappelle <arnout@mind.be> a ?crit : > >> > >> > >> > >> On 13/03/2019 00:06, Fabrice Fontaine wrote: > >>> Hello Thomas, > >>> Le mar. 12 mars 2019 ? 23:29, Thomas Petazzoni > >>> <thomas.petazzoni@bootlin.com> a ?crit : > >>>> > >>>> Hello Fabrice, > >>>> > >>>> On Sat, 9 Mar 2019 23:29:53 +0100 > >>>> Fabrice Fontaine <fontaine.fabrice@gmail.com> wrote: > >>>> > >>>>> Remove select on BR2_PACKAGE_QT5BASE and BR2_PACKAGE_OPENSSL to avoid a > >>>>> circular dependency > >>>> > >>>> I don't really understand where the "circular dependency" is. > >>>> > >>>> To me it seems like the problem is rather: > >>>> > >>>> """ > >>>> Qt 5.6 SSL support is no longer available with OpenSSL since the bump > >>>> of OpenSSL to 1.1.x. Therefore selecting Qt5 and OpenSSL does not > >>>> guarantee that qt5base will be built with SSL support. Due to this, the > >>>> build of qt5enginio currently fails on Qt 5.6. > >>>> """ > >>>> > >>>>> diff --git a/package/qt5/qt5enginio/Config.in b/package/qt5/qt5enginio/Config.in > >>>>> index 102058e7c5..cce0f2a1cb 100644 > >>>>> --- a/package/qt5/qt5enginio/Config.in > >>>>> +++ b/package/qt5/qt5enginio/Config.in > >>>>> @@ -1,7 +1,6 @@ > >>>>> config BR2_PACKAGE_QT5ENGINIO > >>>>> bool "qt5enginio" > >>>>> - select BR2_PACKAGE_OPENSSL > >>>>> - select BR2_PACKAGE_QT5BASE > >>>>> + depends on BR2_PACKAGE_QT5BASE_OPENSSL > >>>> > >>>> Either we do this, or: > >>>> > >>>> select BR2_PACKAGE_OPENSSL if BR2_PACKAGE_QT5_VERSION_LATEST > >>>> select BR2_PACKAGE_LIBRESSL if BR2_PACKAGE_QT5_VERSION_5_6 > >>> I tried this, but I assume that I must add the dependencies of > >>> libressl if QT 5.6 is true, for example like this: > >>> > >>> depends on !BR2_PACKAGE_QT5_VERSION_5_6 || \ > >>> (BR2_PACKAGE_QT5_VERSION_5_6 && \ > >>> (!BR2_PACKAGE_OPENSSL_FORCE_LIBOPENSSL && \ > >>> !(BR2_TOOLCHAIN_USES_UCLIBC && !BR2_USE_MMU))) > >>> > >>> But then, this creates the following circular dependency: > >>> > >>> package/sqlcipher/Config.in:1:error: recursive dependency detected! > >>> package/sqlcipher/Config.in:1: symbol BR2_PACKAGE_SQLCIPHER depends on > >>> BR2_PACKAGE_SQLITE > >>> package/sqlite/Config.in:1: symbol BR2_PACKAGE_SQLITE is selected by > >>> BR2_PACKAGE_QT5BASE_SQLITE_SYSTEM > >>> package/qt5/qt5base/Config.in:104: symbol > >>> BR2_PACKAGE_QT5BASE_SQLITE_SYSTEM is part of choice <choice> > >>> package/qt5/qt5base/Config.in:88: choice <choice> contains symbol <choice> > >>> package/qt5/qt5base/Config.in:88: choice <choice> contains symbol > >>> BR2_PACKAGE_QT5BASE > >>> package/qt5/qt5base/Config.in:1: symbol BR2_PACKAGE_QT5BASE is > >>> selected by BR2_PACKAGE_QT5ENGINIO > >>> package/qt5/qt5enginio/Config.in:1: symbol BR2_PACKAGE_QT5ENGINIO > >>> depends on BR2_PACKAGE_OPENSSL_FORCE_LIBOPENSSL > >>> package/openssl/Config.in:1: symbol > >>> BR2_PACKAGE_OPENSSL_FORCE_LIBOPENSSL is selected by > >>> BR2_PACKAGE_SQLCIPHER > >>> > >>> Did I made a mistake? Shouldn't we add the libressl dependencies? > >> > >> Whaa, kconfig... > >> > >> How about just depending on QT5_LATEST || BR2_PACKAGE_QT5BASE_OPENSSL? Or does > >> that also give a circular dependency? > > Yes, adding the select suggested by Thomas and the above dependencies > > also give a circular dependency. > > The issue is that we can't select BR2_PACKAGE_LIBRESSL without adding > > a circular dependency because of the > > BR2_PACKAGE_OPENSSL_FORCE_LIBOPENSSL option that can be selected by > > any package that can't work with libressl. > > Currently, this option is selected by 4 packages: sqlcipher, > > wpa_supplicant, tpm2-tss and softether. > > So until this option is removed (if the above packages can work with > > libressl), I think my current patch is the only solution. > > I think I got it to work like this: > > # This is a hack to avoid circular dependencies when selecting OPENSSL > config BR2_PACKAGE_QT5ENGINIO_SELECT_OPENSSL > bool > default y if BR2_PACKAGE_QT5ENGINIO > select BR2_PACKAGE_OPENSSL > > config BR2_PACKAGE_QT5ENGINIO > bool "qt5enginio" > depends on !BR2_PACKAGE_QT5_VERSION_5_6 || \ > (BR2_PACKAGE_QT5_VERSION_5_6 && \ > (!BR2_PACKAGE_OPENSSL_FORCE_LIBOPENSSL && \ > !(BR2_TOOLCHAIN_USES_UCLIBC && !BR2_USE_MMU))) > select BR2_PACKAGE_QT5BASE_GUI > select BR2_PACKAGE_QT5BASE_NETWORK > > > Could you give that a go? It works :-). Should I sent a new version of my patch or do you prefer to send it as this your solution? > > Regards, > Arnout Best Regards, Fabrice ^ permalink raw reply [flat|nested] 8+ messages in thread
* [Buildroot] [PATCH 1/1] package/qt5enginio: needs ssl 2019-04-28 9:21 ` Fabrice Fontaine @ 2019-04-29 9:01 ` Arnout Vandecappelle 0 siblings, 0 replies; 8+ messages in thread From: Arnout Vandecappelle @ 2019-04-29 9:01 UTC (permalink / raw) To: buildroot On 28/04/2019 11:21, Fabrice Fontaine wrote: > It works :-). Should I sent a new version of my patch or do you prefer > to send it as this your solution? Better if you send it. The comment update is still to do, and that is complicated as well, so I will not get around to it. Regards, Arnout ^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2019-04-29 9:01 UTC | newest] Thread overview: 8+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2019-03-09 22:29 [Buildroot] [PATCH 1/1] package/qt5enginio: needs ssl Fabrice Fontaine 2019-03-12 22:29 ` Thomas Petazzoni 2019-03-12 23:06 ` Fabrice Fontaine 2019-03-26 22:52 ` Arnout Vandecappelle 2019-03-27 18:08 ` Fabrice Fontaine 2019-04-27 22:33 ` Arnout Vandecappelle 2019-04-28 9:21 ` Fabrice Fontaine 2019-04-29 9:01 ` Arnout Vandecappelle
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox