Horton

Re: meson missing test dependencies

Peter Eisentraut <peter@eisentraut.org>

View thread View on postgr.es
From: Peter Eisentraut <peter@eisentraut.org>
To: Andres Freund <andres@anarazel.de>
Cc: pgsql-hackers <pgsql-hackers@postgresql.org>, Nazir Bilal Yavuz <byavuz81@gmail.com>
Date: 2024-12-06T17:54:25Z
Lists: pgsql-hackers 
On 05.12.24 21:10, Andres Freund wrote:
> Hi,
> 
> On 2024-12-05 20:08:24 +0100, Peter Eisentraut wrote:
>> On 03.12.24 17:01, Andres Freund wrote:
>>> On 2024-12-02 11:10:56 +0100, Peter Eisentraut wrote:
>>> That's unfortunately a very partial fix - because we insist on tests being run
>>> against a temporary install, we don't just need to rebuild the code, we also
>>> need to re-install it.  Without that you don't, e.g., see server changes.
>>
>> Right, I was willing to accept that as a compromise, but see below.
>>
>>> However, it looks like the tmp_install test *does* miss dependencies too and I
>>> see no reason to not fix that.
>>
>>> diff --git i/meson.build w/meson.build
>>> index ff3848b1d85..55b751a0c6b 100644
>>> --- i/meson.build
>>> +++ w/meson.build
>>> @@ -3263,6 +3263,7 @@ test('tmp_install',
>>>        priority: setup_tests_priority,
>>>        timeout: 300,
>>>        is_parallel: false,
>>> +    depends: all_built,
>>>        suite: ['setup'])
>>>
>>>    test('install_test_files',
>>
>> Yes, that addresses my cube example.
> 
> Let's do that. I'm inclined to only do so on master, but I can also see
> backpatching it. Arguments?

master is enough for me.

Commits

  1. oauth: Correct test dependency on oauth_hook_client

  2. meson: Add missing dependencies for libpq tests

  3. meson: Add missing dependencies to libpq_pipeline test

  4. meson: Add pg_regress_ecpg to ecpg test dependencies

  5. meson: Add test dependencies for test_json_parser

  6. meson: Improve dependencies for tmp_install test target

  7. meson: Narrow dependencies for 'install-quiet' target