Hi Willy,

Le 1 août 2019 à 10:07, Willy Tarreau <w...@1wt.eu> a écrit :

Hi Manu,

On Travis CI there was a fairly recent regression on BoringSSL which
happened between 03e09f3 and a7a0f99 a day ago. It breaks on definition
of EVP_PKEY_base_id() in openssl-compat.h, which was not modified, and
I guess this issue was hidden by the simultaneous breakage of libressl
by latest changes.

It looks like latest BoringSSL now defines this function and that the
definition above could be removed. Could you please have a look when
you have a moment and possibly propose a patch so that we can fix those
build reports (especially if you find that other places need to be
touched) ?

For reference, first breakage :

Last known good build:

Yep, i already built with the change.  Fix included.

I'm looking at BORINGSSL_API_VERSION for compatibility evolution, but for now
is not incremented as i would expect.

// BORINGSSL_API_VERSION is a positive integer that increments as BoringSSL
// changes over time. The value itself is not meaningful. It will be incremented
// whenever is convenient to coordinate an API change with consumers. This will
// not denote any special point in development.
// A consumer may use this symbol in the preprocessor to temporarily build
// against multiple revisions of BoringSSL at the same time. It is not
// recommended to do so for longer than is necessary.


Attachment: 0001-BUILD-ssl-BoringSSL-add-EVP_PKEY_base_id.patch
Description: Binary data

Reply via email to