pg.ddx.io  pgsql-committers@postgresql.org mailing list archive  
help / color / mirror / Atom feed
From: Andres Freund <andres@anarazel.de>
To: pgsql-committers@lists.postgresql.org
Subject: pgsql: Make test portlock logic work with meson
Date: Fri, 21 Feb 2025 16:48:31 +0000
Message-ID: <E1tlWCW-000UUn-2K@gemulon.postgresql.org> (raw)

Make test portlock logic work with meson

Previously the portlock logic, added in 9b4eafcaf41, didn't actually work
properly when the tests were run via meson. 9b4eafcaf41 used the
MESON_BUILD_ROOT environment variable to determine the directory for the port
lock directory, but that's never set for running the tests.  That meant that
each test used its own portlock dir, unless the PG_TEST_PORT_DIR environment
variable was set.

Fix the problem by setting top_builddir for the environment. That's also used
for the autoconf/make build.

Backpatch back to 16, where meson support was added.

Reported-by: Zharkov Roman <r.zharkov@postgrespro.ru>
Reviewed-by: Andrew Dunstan <andrew@dunslane.net>
Backpatch-through: 16

Branch
------
REL_17_STABLE

Details
-------
https://git.postgresql.org/pg/commitdiff/0f7c0fb59cb71e1ecd45c6892ab184fefbfde8be

Modified Files
--------------
meson.build                              | 2 ++
src/test/perl/PostgreSQL/Test/Cluster.pm | 4 +---
2 files changed, 3 insertions(+), 3 deletions(-)



view thread (3+ messages)  latest in thread

Message-ID: <E1tlWCW-000UUn-2K@gemulon.postgresql.org>
Permalink:  ../E1tlWCW-000UUn-2K@gemulon.postgresql.org/
Also on:    postgresql.org/message-id/E1tlWCW-000UUn-2K@gemulon.postgresql.org

 · 

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: pgsql-committers@postgresql.org
  Cc: andres@anarazel.de, pgsql-committers@lists.postgresql.org
  Subject: Re: pgsql: Make test portlock logic work with meson
  In-Reply-To: <E1tlWCW-000UUn-2K@gemulon.postgresql.org>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox