Re: Fix use of openssl.path() if openssl isn't found

2023-11-08 Thread Tristan Partin
On Wed Nov 8, 2023 at 2:31 AM CST, Michael Paquier wrote: On Wed, Nov 08, 2023 at 12:07:49AM -0600, Tristan Partin wrote: >'with_ssl': ssl_library, > - 'OPENSSL': openssl.path(), > + 'OPENSSL': openssl.found() ? openssl.path : '', Except that this was incorrect. I've fixed

Re: Fix use of openssl.path() if openssl isn't found

2023-11-08 Thread Michael Paquier
On Wed, Nov 08, 2023 at 12:07:49AM -0600, Tristan Partin wrote: >'with_ssl': ssl_library, > - 'OPENSSL': openssl.path(), > + 'OPENSSL': openssl.found() ? openssl.path : '', Except that this was incorrect. I've fixed the grammar and applied that down to 16. -- Michael

Re: Fix use of openssl.path() if openssl isn't found

2023-11-07 Thread Tristan Partin
2c0f6ec650c40df833 Mon Sep 17 00:00:00 2001 From: Tristan Partin Date: Tue, 7 Nov 2023 15:59:04 -0600 Subject: [PATCH v2] Fix use of openssl.path() if openssl isn't found openssl(1) is an optional dependency of the Postgres Meson build, but was inadvertantly required when defining some SSL tests

Re: Fix use of openssl.path() if openssl isn't found

2023-11-07 Thread Michael Paquier
On Tue, Nov 07, 2023 at 04:06:56PM -0600, Tristan Partin wrote: > Found this issue during my Fedora 39 upgrade. Tested that uninstalling > openssl still allows the various ssl tests to run and succeed. Good catch. You are right that this is inconsistent with what we expect in the test. >

Fix use of openssl.path() if openssl isn't found

2023-11-07 Thread Tristan Partin
:59:04 -0600 Subject: [PATCH v1] Fix use of openssl.path() if openssl isn't found openssl(1) is an optional dependency of the Postgres Meson build, but was inadvertantly required when defining some SSL tests. --- src/test/ssl/meson.build | 7 ++- 1 file changed, 6 insertions(+), 1 deletion