public inbox for [email protected]  

help / color / mirror / Atom feed

From: Andreas Karlsson <[email protected]>
To: Rustam ALLAKOV <[email protected]>
To: [email protected]
Subject: Re: Add support for EXTRA_REGRESS_OPTS for meson
Date: Wed, 14 Jan 2026 02:17:30 +0100
Message-ID: <[email protected]> (raw)
In-Reply-To: <176834196488.969.13193439940284055773.pgcf@coridan.postgresql.org>
References: <[email protected]>
	<[email protected]>
	<176834196488.969.13193439940284055773.pgcf@coridan.postgresql.org>

On 1/13/26 11:06 PM, Rustam ALLAKOV wrote:
>>> in addition to the above, might be worth to add some documentation like
>>>
>>> Environment Variables Supported:
>>> EXTRA_REGRESS_OPTS: Additional options to pass to regression, isolation, or ecpg tests.
>>> TEMP_CONFIG: Specify a temporary configuration file for testing purposes.
>>> Example Usage:
>>> # Use EXTRA_REGRESS_OPTS to load an extension
>>>     EXTRA_REGRESS_OPTS="--load-extension=pgcrypto" meson test
>>> # Use TEMP_CONFIG to specify a temporary configuration file
>>>     TEMP_CONFIG="path/to/test.conf" meson test
>>> # Use both EXTRA_REGRESS_OPTS and TEMP_CONFIG together
>>>     TEMP_CONFIG="path/to/test.conf" EXTRA_REGRESS_OPTS="--load-extension=pgcrypto" meson test
>>
>> Yeah, we probably should. But not sure where, maybe at
>> https://www.postgresql.org/docs/current/install-meson.html or did you
>> imagine somewhere else?
> 
> Either at docs or/and directly in file /src/tools/testwrap as comments
> Regards.

If we add it to the documentation the question is if we should also 
document environment variables for builds with make. But maybe that is 
for a separate patch.

As for documenting it in testwrap: that does not feel very useful to me 
since people who know that this is where they should look likely already 
knows about these. It is a pretty hidden place. Hmm, not sure where such 
information belongs.

Andreas

view thread (9+ messages)  latest in thread

reply

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: [email protected]
  Cc: [email protected], [email protected], [email protected]
  Subject: Re: Add support for EXTRA_REGRESS_OPTS for meson
  In-Reply-To: <[email protected]>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox