[Buildroot] [PATCH] axel: bump to version 2.16
Thomas Petazzoni
thomas.petazzoni at free-electrons.com
Sat Nov 25 12:23:57 UTC 2017
Hello,
On Sat, 25 Nov 2017 01:44:47 -0300, Ismael Luceno wrote:
> - Change upstream URL.
> - Add optional dependency on libressl/openssl.
> - Switch to autotools-package.
>
> Signed-off-by: Ismael Luceno <ismael at iodev.co.uk>
I've applied to next, but now that I review the patch again, I see a
problem, see below.
> +ifeq ($(BR2_PACKAGE_LIBRESSL),y)
> +AXEL_CONF_OPTS += --with-ssl
> +AXEL_DEPENDENCIES += libressl
This is not needed, because the below case with BR2_PACKAGE_OPENSSL is
sufficient. Indeed openssl is not a virtual package, that depends
either on libopenssl or libressl depending on which implementation is
selected.
> +else ifeq ($(BR2_PACKAGE_OPENSSL),y)
> +AXEL_CONF_OPTS += --with-ssl
> +AXEL_DEPENDENCIES += openssl
Could you send a follow-up patch (based on the next branch) that fixes
this?
Thanks!
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
More information about the buildroot
mailing list