[Buildroot] [PATCH 1/1] package/iputils: bump version to 20200821
Petr Vorel
petr.vorel at gmail.com
Tue Aug 25 06:38:41 UTC 2020
Hi Thomas,
> On Mon, 24 Aug 2020 18:40:42 +0200
> Petr Vorel <petr.vorel at gmail.com> wrote:
> > Remove patch from this version.
> > Remove USE_CRYPTO related config (configuration was removed in this
> > release).
> > tftpd is not built by default since this release, thus explicitly enable it.
> > Signed-off-by: Petr Vorel <petr.vorel at gmail.com>
> I have applied to next, but... I have some second thoughts.
Thank you.
> > -ifeq ($(BR2_PACKAGE_NETTLE),y)
> > -IPUTILS_CONF_OPTS += -DUSE_CRYPTO=nettle
> > -IPUTILS_DEPENDENCIES += nettle
> > -else ifeq ($(BR2_PACKAGE_LIBGCRYPT),y)
> > -IPUTILS_CONF_OPTS += -DUSE_CRYPTO=gcrypt
> > -IPUTILS_DEPENDENCIES += libgcrypt
> > -else ifeq ($(BR2_PACKAGE_OPENSSL),y)
> > -IPUTILS_CONF_OPTS += -DUSE_CRYPTO=openssl
> > -IPUTILS_DEPENDENCIES += openssl
> > -else
> > -IPUTILS_CONF_OPTS += -DUSE_CRYPTO=kernel
> > -endif
> So you no longer have any optional dependency on a crypto provider, i.e
> there is nothing that guarantees that openssl will be built before
> iputils if openssl is available, for example.
> Is that really what you want ?
Yes. MD5 implementation was added into this release (the only crypto
dependency):
https://github.com/iputils/iputils/commit/214ed83afcce7f08406b17144040891eb40113cf
Maybe my description in the commit message isn't clear enough, putting the link
of the commit would be better.
Kind regards,
Petr
More information about the buildroot
mailing list