diff options
author | Alvaro Herrera <alvherre@alvh.no-ip.org> | 2016-09-08 14:39:05 -0300 |
---|---|---|
committer | Alvaro Herrera <alvherre@alvh.no-ip.org> | 2016-09-08 14:39:05 -0300 |
commit | 19acee8c5adb68b96222e41c084efbc9b31d397a (patch) | |
tree | 90420ec7cf87e24720d0f37f195713bd9572ede3 /src/test/modules/commit_ts | |
parent | df5d9bb8d5074138e6fea63ac8acd9b95a0eb859 (diff) | |
download | postgresql-19acee8c5adb68b96222e41c084efbc9b31d397a.tar.gz postgresql-19acee8c5adb68b96222e41c084efbc9b31d397a.zip |
Fix two src/test/modules Makefiles
commit_ts and test_pg_dump were declaring targets before including the
PGXS stanza, which meant that the "all" target customarily defined as
the first (and therefore default target) was not the default anymore.
Fix that by moving those target definitions to after PGXS.
commit_ts was initially good, but I broke it in commit 9def031bd2;
test_pg_dump was born broken, probably copying from commit_ts' mistake.
In passing, fix a comment mistake in test_pg_dump/Makefile.
Backpatch to 9.6.
Noted by Tom Lane.
Diffstat (limited to 'src/test/modules/commit_ts')
-rw-r--r-- | src/test/modules/commit_ts/Makefile | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/src/test/modules/commit_ts/Makefile b/src/test/modules/commit_ts/Makefile index f814f767c44..86b93b5e762 100644 --- a/src/test/modules/commit_ts/Makefile +++ b/src/test/modules/commit_ts/Makefile @@ -3,11 +3,6 @@ REGRESS = commit_timestamp REGRESS_OPTS = --temp-config=$(top_srcdir)/src/test/modules/commit_ts/commit_ts.conf -check: prove-check - -prove-check: - $(prove_check) - ifdef USE_PGXS PG_CONFIG = pg_config PGXS := $(shell $(PG_CONFIG) --pgxs) @@ -18,3 +13,8 @@ top_builddir = ../../../.. include $(top_builddir)/src/Makefile.global include $(top_srcdir)/contrib/contrib-global.mk endif + +check: prove-check + +prove-check: + $(prove_check) |