From: | Jacob Champion <jacob(dot)champion(at)enterprisedb(dot)com> |
---|---|
To: | Ashutosh Bapat <ashutosh(dot)bapat(dot)oss(at)gmail(dot)com> |
Cc: | Nazir Bilal Yavuz <byavuz81(at)gmail(dot)com>, Tristan Partin <tristan(at)partin(dot)io>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Andrew Dunstan <andrew(at)dunslane(dot)net> |
Subject: | Re: PG_TEST_EXTRA and meson |
Date: | 2024-08-23 16:25:12 |
Message-ID: | CAOYmi+=jx=MWP2KjhjvM4viw2p3yPmsHMCQAXa6YwD29SBhRiQ@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Fri, Aug 23, 2024 at 5:59 AM Ashutosh Bapat
<ashutosh(dot)bapat(dot)oss(at)gmail(dot)com> wrote:
> If I run
> export PG_TEST_EXTRA=xid_wraparound; ./configure --prefix=$BuildDir
> --enable-tap-tests && make -j4 && make -j4 install; unset
> PG_TEST_EXTRA
> followed by
> make -C $XID_MODULE_DIR check where
> XID_MODULE_DIR=src/test/modules/xid_wraparound - it skips the tests.
Right.
> I thought this was working before.
No, that goes back to my note in [1] -- as of c3382a3c3cc, the
variable was exported at only the src/test level, and I wanted to get
input on that so we could decide on the next approach if needed.
> Anyway, now I have written a script to test all the scenarios. You may
> want to test your patch using the script. It just needs PGDir to set
> to root directory of code.
Thanks! I see failures in 110, 120, and 130, as expected. Note that
constructions like
PG_TEST_EXTRA="" cd $XID_MODULE_DIR && make check && cd $PGDir
will not override the environment variable for the make invocation,
just for the `cd`. Also, rather than
export PG_TEST_EXTRA; ./configure ...; unset PG_TEST_EXTRA
it's probably easier to just pass PG_TEST_EXTRA=<setting> as a command
line option to configure.
> If there's some other way to setting PG_TEST_EXTRA when running
> configure, I think it needs to be documented.
./configure --help shows the new variable, with the same wording as
Meson. Or do you mean that it's significant enough to deserve a spot
in installation.sgml?
--Jacob
From | Date | Subject | |
---|---|---|---|
Next Message | Alexander Korotkov | 2024-08-23 16:38:36 | Re: POC, WIP: OR-clause support for indexes |
Previous Message | Robert Haas | 2024-08-23 15:59:17 | Re: Eager aggregation, take 3 |