From: Peter Eisentraut Date: Fri, 5 May 2023 04:29:49 +0000 (+0200) Subject: Fix prove_installcheck when used with PGXS X-Git-Tag: REL_16_BETA1~103 X-Git-Url: http://git.postgresql.org/gitweb/?a=commitdiff_plain;h=5d42a97519581a31bdc1d3d4cf10c07ae18b8296;p=postgresql.git Fix prove_installcheck when used with PGXS Commit 153e215677 added the portlock directory. This is created in $ENV{top_builddir} if it is set. Under PGXS, top_builddir points into the installation directory, which is not necessarily writable and in any case inappropriate to use by a test suite. The cause of the problem is that the prove_installcheck target in Makefile.global exports top_builddir, which isn't useful (since no other Perl code actually reads it) and breaks this use case. The reason this code is there is probably that is has been dragged around with various other changes, in particular a0fc813266, but without a real purpose of its own. By just removing the exporting of top_builddir in prove_installcheck, the portlock directory then ends up under tmp_check in the build directory, which is more suitable. Reviewed-by: Andrew Dunstan Discussion: https://www.postgresql.org/message-id/78d1cfa6-0065-865d-584b-cde6d8c18aff@enterprisedb.com --- diff --git a/src/Makefile.global.in b/src/Makefile.global.in index 772b91261d8..974b1dfef9e 100644 --- a/src/Makefile.global.in +++ b/src/Makefile.global.in @@ -464,7 +464,7 @@ cd $(srcdir) && \ TESTLOGDIR='$(CURDIR)/tmp_check/log' \ TESTDATADIR='$(CURDIR)/tmp_check' \ PATH="$(bindir):$(CURDIR):$$PATH" \ - PGPORT='6$(DEF_PGPORT)' top_builddir='$(top_builddir)' \ + PGPORT='6$(DEF_PGPORT)' \ PG_REGRESS='$(top_builddir)/src/test/regress/pg_regress' \ $(PROVE) $(PG_PROVE_FLAGS) $(PROVE_FLAGS) $(if $(PROVE_TESTS),$(PROVE_TESTS),t/*.pl) endef