Thread

  1. meson/testwrap: support EXTRA_REGRESS_OPTS environment variable

    Bertrand Drouvot <bertranddrouvot.pg@gmail.com> — 2026-04-23T13:05:52Z

    Hi hackers,
    
    While looking at [1], I did use the pg_regress --use-existing flag that way:
    
    TESTS="test_setup create_index memoize --use-existing" meson test -C meson_build --setup running --suite regress-running
    
    and that produced the desired outcome (i.e running those 3 tests from the regress
    suite on a running instance and existing regression database).
    
    That was kind of a trick and not surprisingly when I wanted to run the entire
    regress suite that way:
    
    TESTS="--use-existing" meson test -C meson_build --setup running --suite regress-running
    
    things did not work as expected and produced:
    
    1/1 regress-running - postgresql:regress-running/regress        SKIP              0.04s
    
    Ok:                0
    Fail:              0
    Skipped:           1
    
    means it skipped the tests.
    
    OTOH, with autoconf, one could run:
    
    EXTRA_REGRESS_OPTS="--use-existing" make installcheck
    
    and that would work as expected (means running the entire regress suite on an
    existing regression database).
    
    So it looks like that currently (unless I missed it), with meson, it is not
    possible to run an entire test suite on a running instance and existing regression
    database.
    
    Indeed, setting the TESTS environment variable replaces the schedule and test
    list entirely. This means that passing only pg_regress flags (e.g.,
    TESTS="--use-existing") results in no tests being run, since no schedule or test
    names are included.
    
    The attached patch adds support for EXTRA_REGRESS_OPTS in testwrap. When set,
    its contents are appended to the test command for regress, isolation, and ecpg
    test types, matching autoconf behavior.
    
    This enables running the full schedule with extra pg_regress options, like:
    
    EXTRA_REGRESS_OPTS="--use-existing" meson test --setup running --suite regress-running
    
    Once we agree on a fix and that fix is pushed, I think it would make sense
    to add an example in [2].
    
    [1]: https://postgr.es/m/aekGK/SUIeW5n5fY%40bdtpg
    [2]: https://wiki.postgresql.org/wiki/Meson
    
    Regards,
    
    -- 
    Bertrand Drouvot
    PostgreSQL Contributors Team
    RDS Open Source Databases
    Amazon Web Services: https://aws.amazon.com
    
  2. Re: meson/testwrap: support EXTRA_REGRESS_OPTS environment variable

    Andreas Karlsson <andreas@proxel.se> — 2026-05-01T06:41:55Z

    On 4/23/26 15:05, Bertrand Drouvot wrote:
    > This enables running the full schedule with extra pg_regress options, like:
    > 
    > EXTRA_REGRESS_OPTS="--use-existing" meson test --setup running --suite regress-running
    > 
    > Once we agree on a fix and that fix is pushed, I think it would make sense
    > to add an example in [2].
    Nice to see more people wanting this. :)
    
    I already have a patch[1] for this in the current commitfest which 
    people think looks good other than that it lacks documentation, which I 
    think is a fair complaint. It is basically just waiting for me (or 
    someone else!) to find a bit of time to write documentation for it. Feel 
    free to help out, but otherwise I plan to write documentation after 
    PGConf.dev when my life is a bit less hectic.
    
    1. https://commitfest.postgresql.org/patch/6361/
    
    --
    Andreas Karlsson
    Percona
    
    
    
    
    
  3. Re: meson/testwrap: support EXTRA_REGRESS_OPTS environment variable

    Bertrand Drouvot <bertranddrouvot.pg@gmail.com> — 2026-05-04T05:57:45Z

    Hi,
    
    On Fri, May 01, 2026 at 08:41:55AM +0200, Andreas Karlsson wrote:
    > On 4/23/26 15:05, Bertrand Drouvot wrote:
    > > This enables running the full schedule with extra pg_regress options, like:
    > > 
    > > EXTRA_REGRESS_OPTS="--use-existing" meson test --setup running --suite regress-running
    > > 
    > > Once we agree on a fix and that fix is pushed, I think it would make sense
    > > to add an example in [2].
    > Nice to see more people wanting this. :)
    > 
    > I already have a patch[1] for this in the current commitfest
    
    Thanks for the reach out, I just withdrawn mine then.
    
    Regards,
    
    -- 
    Bertrand Drouvot
    PostgreSQL Contributors Team
    RDS Open Source Databases
    Amazon Web Services: https://aws.amazon.com