From: | Nazir Bilal Yavuz <byavuz81(at)gmail(dot)com> |
---|---|
To: | Peter Eisentraut <peter(dot)eisentraut(at)enterprisedb(dot)com>, Andres Freund <andres(at)anarazel(dot)de> |
Cc: | pgsql-hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: meson: Optionally disable installation of test modules |
Date: | 2023-02-01 12:41:21 |
Message-ID: | 6e805027-19db-f8c3-8acd-962fd1ffe96e@gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Hi,
On 1/31/23 11:44, Peter Eisentraut wrote:
> On 30.01.23 18:42, Andres Freund wrote:
>> Bilal, with a bit of help by me, worked on an alternative approach to
>> this. It's a lot more verbose in the initial change, but wouldn't
>> increase the
>> amount of work/lines for new test modules. The main advantage is that we
>> wouldn't have disable the modules by default, which I think would be
>> quite
>> likely to result in plenty people not running the tests.
>>
>> Sending a link instead of attaching, in case you already registered a
>> cfbot entry:
>> https://github.com/anarazel/postgres/commit/d1d192a860da39af9aa63d7edf643eed0eeee7c4
>>
>>
>> Probably worth adding an install-test-modules target for manual use.
>
> Looks like a good idea. I'm happy to proceed along that line.
I am adding a patch of an alternative approach. Also, I updated the
commit at the link by adding regress_module, autoinc_regress and
refint_regress to the test_install_libs.
Regards,
Nazir Bilal Yavuz
Microsoft
Attachment | Content-Type | Size |
---|---|---|
v2-0001-meson-prevent-installation-of-test-files-during-m.patch | text/x-diff | 26.9 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Yugo NAGATA | 2023-02-01 12:55:54 | Re: transition tables and UPDATE |
Previous Message | Dean Rasheed | 2023-02-01 12:38:15 | Re: transition tables and UPDATE |