[Buildroot] [PATCH 1/1] ntp: add patch to support for libressl
Thomas Petazzoni
thomas.petazzoni at free-electrons.com
Thu Nov 23 22:29:52 UTC 2017
Hello,
On Thu, 23 Nov 2017 23:27:18 +0100, Arnout Vandecappelle wrote:
> > Arnout, Peter, Yann, I think we discussed this topic during the
> > Buildroot meeting, and concluded we didn't want patches in Buildroot to
> > enable LibreSSL compatibility with a package. Do we stand on this
> > position, and reject Adam's contribution on ntp?
>
> I don't think the conclusion was that we would reject patches to enable
> LibreSSL compatibility outright, only:
>
> >
> >> + #ifndef OPENSSL_VERSION_NUMBER
> >> ++#ifndef LIBRESSL_VERSION_NUMBER
> >
> > In addition, this continue to use the LIBRESSL_VERSION_NUMBER approach,
> > which will fail when libressl gains support for new APIs.
>
> we would reject this approach because I believe it is not upstreamable.
>
> I think upstreamable patches are acceptable. And maybe even the
> LIBRESSL_VERSION_NUMBER approach is OK - but then I first want to see a reliable
> upstream accept it.
The issue here is that Adam has submitted the patch upstream a while
ago (see bugs.ntp.org/show_bug.cgi?id=3401#c3), and upstream has
reacted.
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
More information about the buildroot
mailing list