On 05.04.24 18:59, Daniel Gustafsson wrote:
> Attached is a WIP patch to get more eyes on it, the Meson test for 1.1.1 fails
> on Windows in CI which I will investigate next.
I'm not a fan of the new PGAC_CHECK_OPENSSL. It creates a second place
where the OpenSSL version number has to be updated. We had this
carefully constructed so that there is only one place that
OPENSSL_API_COMPAT is defined and that is the only place that needs to
be updated. We put the setting of OPENSSL_API_COMPAT into configure so
that the subsequent OpenSSL tests would use it, and if the API number
higher than what the library supports, the tests should fail. So I
don't understand why the configure changes have to be so expansive.