[Buildroot] [PATCH 1/1] package/qt5enginio: needs ssl
arnout at mind.be
Sat Apr 27 22:33:32 UTC 2019
On 27/03/2019 19:08, Fabrice Fontaine wrote:
> Dear Arnout,
> Le mar. 26 mars 2019 à 23:52, Arnout Vandecappelle <arnout at 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 at bootlin.com> a écrit :
>>>> Hello Fabrice,
>>>> On Sat, 9 Mar 2019 23:29:53 +0100
>>>> Fabrice Fontaine <fontaine.fabrice at 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
>>> package/sqlite/Config.in:1: symbol BR2_PACKAGE_SQLITE is selected by
>>> 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
>>> 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
>>> 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
default y if BR2_PACKAGE_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)))
Could you give that a go?
More information about the buildroot