From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | Andres Freund <andres(at)anarazel(dot)de> |
Cc: | Michael Paquier <michael(at)paquier(dot)xyz>, Postgres hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
Subject: | Re: make -C src/test/isolation failure in index-killtuples due to btree_gist |
Date: | 2025-08-18 15:38:02 |
Message-ID: | 414029.1755531482@sss.pgh.pa.us |
Views: | Whole Thread | Raw Message | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Andres Freund <andres(at)anarazel(dot)de> writes:
> On 2025-08-18 08:57:13 +0900, Michael Paquier wrote:
>> The following command fails, because btree_gist is not installed in
>> the context of the isolation tests:
>> make -C src/test/isolation/
>> ...
>> and it does not strike me as a good idea to begin doing that in the main
>> isolation test suite, so perhaps the best thing to do here is just move this
>> test to contrib/btree_gist/?
> No, it makes absolutely no sense to test e.g. hash killtuples support in
> btree_gist.
I think the complaint is that nothing has been done to ensure that
these modules have been installed. You created a new dependency that
developers have to work around, rather than teaching the build system
to handle it. As a comparison point, all of the tests in
src/test/recovery, src/test/authentication, etc take care to install
required modules when you say "make check" in those directories.
You broke that for src/test/isolation, and you should fix it.
It shouldn't be much harder than setting EXTRA_INSTALL in the
Makefile case; I dunno about meson.
regards, tom lane
From | Date | Subject | |
---|---|---|---|
Next Message | Jacob Champion | 2025-08-18 15:38:25 | Re: Support getrandom() for pg_strong_random() source |
Previous Message | Aleksander Alekseev | 2025-08-18 15:33:16 | Re: Proposal: Extending the PostgreSQL Protocol with Command Metadata |