Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Commit3d37476

Browse files
committed
Fix prove_installcheck when used with PGXS
Commit153e215 added the portlock directory. This is created in$ENV{top_builddir} if it is set. Under PGXS, top_builddir points intothe installation directory, which is not necessarily writable and inany case inappropriate to use by a test suite. The cause of theproblem is that the prove_installcheck target in Makefile.globalexports top_builddir, which isn't useful (since no other Perl codeactually reads it) and breaks this use case. The reason this code isthere is probably that is has been dragged around with various otherchanges, in particulara0fc813, but without a real purpose of itsown. By just removing the exporting of top_builddir inprove_installcheck, the portlock directory then ends up undertmp_check in the build directory, which is more suitable.Reviewed-by: Andrew Dunstan <andrew@dunslane.net>Discussion:https://www.postgresql.org/message-id/78d1cfa6-0065-865d-584b-cde6d8c18aff@enterprisedb.com
1 parent825ebc9 commit3d37476

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

‎src/Makefile.global.in

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -464,7 +464,7 @@ rm -rf '$(CURDIR)'/tmp_check && \
464464
$(MKDIR_P) '$(CURDIR)'/tmp_check && \
465465
cd$(srcdir) && \
466466
TESTDIR='$(CURDIR)' PATH="$(bindir):$(CURDIR):$$PATH" \
467-
PGPORT='6$(DEF_PGPORT)'top_builddir='$(top_builddir)'\
467+
PGPORT='6$(DEF_PGPORT)' \
468468
PG_REGRESS='$(top_builddir)/src/test/regress/pg_regress' \
469469
$(PROVE)$(PG_PROVE_FLAGS)$(PROVE_FLAGS)$(if$(PROVE_TESTS),$(PROVE_TESTS),t/*.pl)
470470
endef

0 commit comments

Comments
 (0)

[8]ページ先頭

©2009-2025 Movatter.jp