Re: PG_TEST_EXTRA and meson - Mailing list pgsql-hackers

From Nazir Bilal Yavuz
Subject Re: PG_TEST_EXTRA and meson
Date
Msg-id CAN55FZ2sJ4r2qtbs1xxRTrdx2sYCfVKj3=1wR7W1VQQVJADFFg@mail.gmail.com
Whole thread Raw
In response to Re: PG_TEST_EXTRA and meson  (Nazir Bilal Yavuz <byavuz81@gmail.com>)
Responses Re: PG_TEST_EXTRA and meson
List pgsql-hackers
Hi,

On Wed, 17 Jul 2024 at 13:23, Nazir Bilal Yavuz <byavuz81@gmail.com> wrote:
>
> Hi,
>
> On Wed, 17 Jul 2024 at 13:13, Ashutosh Bapat
> <ashutosh.bapat.oss@gmail.com> wrote:
> > xid_wraparound tests are run if PG_TEST_EXTRA contains xid_wraparound
> > or it is not set. Any other setting will not run xid_wraparound test.
> > That's how the patch is coded but it isn't intuitive that changing
> > whether a test is run by default would require configuring the build
> > again. Probably we should just get rid of config time PG_TEST_EXTRA
> > altogether.
> >
> > I am including +Tristan Partin who knows meson better.
> >
> > If you are willing to work on this further, please add it to the commitfest.
>
> I think I know why there is confusion. Could you try to set
> PG_TEST_EXTRA with quotes? Like PG_TEST_EXTRA="ldap mts
> xid_wraparound".

Sorry, the previous reply was wrong; I misunderstood what you said.
Yes, that is how the patch was coded and I agree that getting rid of
config time PG_TEST_EXTRA could be a better alternative.

-- 
Regards,
Nazir Bilal Yavuz
Microsoft



pgsql-hackers by date:

Previous
From: Nazir Bilal Yavuz
Date:
Subject: Re: PG_TEST_EXTRA and meson
Next
From: Joe Conway
Date:
Subject: Re: RFC: pg_stat_logmsg