Horton

Re: meson missing test dependencies

Nazir Bilal Yavuz <byavuz81@gmail.com>

View thread View on postgr.es
From: Nazir Bilal Yavuz <byavuz81@gmail.com>
To: Peter Eisentraut <peter@eisentraut.org>
Cc: pgsql-hackers <pgsql-hackers@postgresql.org>
Date: 2024-12-03T08:18:13Z
Lists: pgsql-hackers 
Hi,

On Mon, 2 Dec 2024 at 22:27, Peter Eisentraut <peter@eisentraut.org> wrote:
>
> On 02.12.24 11:50, Nazir Bilal Yavuz wrote:
> > I applied your patches and the cube example worked. But when I edited
> > 'test_json_parser_perf.c' and 'test_json_parser_incremental.c', meson
> > did not rebuild. I used the 'meson test -C build --suite setup --suite
> > test_json_parser' command to test test_json_parser. Did I do something
> > wrong?
>
> Seems to work for me. I edited test_json_parser_incremental.c and then ran
>
> $ meson test -C build --suite setup --suite test_json_parser -v
> ninja: Entering directory `/Users/peter/devel/postgresql/postgresql/build'
> [2/2] Linking target
> src/test/modules/test_json_parser/test_json_parser_incremental
> 1/7 postgresql:setup / tmp_install
>             RUNNING
> ...
>
> Without my patch, you don't get the "Linking target ..." output.

It is working fine now, I must have done something wrong before. Sorry
for the noise.


--
Regards,
Nazir Bilal Yavuz
Microsoft

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