From: Peter Eisentraut <peter@eisentraut.org>
To: Heikki Linnakangas <heikki.linnakangas@iki.fi>
To: pgsql-committers@lists.postgresql.org
Subject: Re: pgsql: Make PG_TEST_EXTRA env var override the "meson setup" option
Date: Tue, 5 Nov 2024 10:16:29 +0100
Message-ID: <5c380997-e270-425a-9542-e4ef36a285de@eisentraut.org> (raw)
In-Reply-To: <E1t7w9I-0007vD-R1@gemulon.postgresql.org>
References: <E1t7w9I-0007vD-R1@gemulon.postgresql.org>
On 04.11.24 13:25, Heikki Linnakangas wrote:
> Make PG_TEST_EXTRA env var override the "meson setup" option
I see this warning flying by:
Configuring Makefile.global using configuration
../src/meson.build:31: WARNING: The variable(s) 'PG_TEST_EXTRA' in the
input file 'src/Makefile.global.in' are not present in the given
configuration data.
> "meson test" used to ignore the PG_TEST_EXTRA environment variable,
> which meant that in order to run additional tests, you had to run
> "meson setup -DPG_TEST_EXTRA=...". That's somewhat expensive, and not
> consistent with autoconf builds. Allow PG_TEST_EXTRA environment
> variable to override the setup-time option at run time, so that you
> can do "PG_TEST_EXTRA=... meson test".
>
> To implement this, the configuration time value is passed as an extra
> "--pg-test-extra" argument to testwrap instead of adding it to the
> test environment. If the environment variable is set at the time of
> running test, testwrap uses the value from the environment variable
> and ignores the --pg-test-extra option.
>
> Now that "meson test" obeys the environment variable, we can remove it
> from the "meson setup" steps in the CI script. It will now be picked
> up from the environment variable like with "make check".
>
> Author: Nazir Bilal Yavuzk, Ashutosh Bapat
> Reviewed-by: Ashutosh Bapat with inputs from Tom Lane and Andrew Dunstan
>
> Branch
> ------
> master
>
> Details
> -------
> https://git.postgresql.org/pg/commitdiff/3d1aec225ac4a1199079150dabfd0d34c5054127
>
> Modified Files
> --------------
> .cirrus.tasks.yml | 6 +-----
> doc/src/sgml/installation.sgml | 10 +++++++---
> meson.build | 10 +++++-----
> meson_options.txt | 2 +-
> src/tools/testwrap | 10 ++++++++++
> 5 files changed, 24 insertions(+), 14 deletions(-)
>
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: pgsql-committers@postgresql.org
Cc: peter@eisentraut.org, heikki.linnakangas@iki.fi, pgsql-committers@lists.postgresql.org
Subject: Re: pgsql: Make PG_TEST_EXTRA env var override the "meson setup" option
In-Reply-To: <5c380997-e270-425a-9542-e4ef36a285de@eisentraut.org>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox