[Buildroot] [PATCH 10/10] libssl: add libressl as a option
Arnout Vandecappelle
arnout at mind.be
Fri Aug 4 21:50:40 UTC 2017
On 04-08-17 20:55, Adam Duskett wrote:
[snip]
> LIBRESSL_POST_INSTALL_TARGET_HOOKS += LIBRESSL_REMOVE_BIN
> endif
>
> +$(eval $(host-autotools-package))
> $(eval $(autotools-package))
Ah, here is the host package!
This should be a separate patch (the one you had before). There is still one
problem, though: there is nothing stopping both host-openssl and host-libressl
to be built. Oh, hang on, that's not true, because you never depend on
host-openssl or host-libressl but always on host-libssl. OK, that sounds like a
good solution then! Can you extend the commit message with a sentence that
explains this?
Note, I've marked the entire series as changes requested in patchwork, except
for the version bumps that can be applied independently.
Regards,
Arnout
> diff --git a/package/libssl/Config.in b/package/libssl/Config.in
> index 5f3d2fc2a..7a1ca68d4 100644
> --- a/package/libssl/Config.in
> +++ b/package/libssl/Config.in
> @@ -12,6 +12,7 @@ choice
> Select OpenSSL or LibreSSL.
>
> source "package/openssl/Config.in"
> + source "package/libressl/Config.in"
>
> endchoice
>
>
--
Arnout Vandecappelle arnout at mind be
Senior Embedded Software Architect +32-16-286500
Essensium/Mind http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint: 7493 020B C7E3 8618 8DEC 222C 82EB F404 F9AC 0DDF
More information about the buildroot
mailing list