diff options
| author | Tom Lane | 2011-08-24 19:16:23 +0000 |
|---|---|---|
| committer | Tom Lane | 2011-08-24 19:17:03 +0000 |
| commit | 5354abb8f69734ff1c07cee5e29845b74d46e9d6 (patch) | |
| tree | 41ce669b9550d8668a78f72d7bd2dac0cd2bf64f | |
| parent | ed1a4adbe38ed94cc2e3c017a9f5be4b282bccb9 (diff) | |
Fix pgxs.mk to always add --dbname=$(CONTRIB_TESTDB) to REGRESS_OPTS.
The previous coding resulted in contrib modules unintentionally overriding
the use of CONTRIB_TESTDB. There seems no particularly good reason to
allow that (after all, the makefile can set CONTRIB_TESTDB if that's really
what it intends).
In passing, document REGRESS_OPTS where the other pgxs.mk options are
documented.
Back-patch to 9.1 --- in prior versions, there were no cases of contrib
modules setting REGRESS_OPTS without including the --dbname switch, so
while the coding was fragile there was no actual bug.
| -rw-r--r-- | contrib/sepgsql/Makefile | 4 | ||||
| -rw-r--r-- | doc/src/sgml/extend.sgml | 9 | ||||
| -rw-r--r-- | src/makefiles/pgxs.mk | 7 |
3 files changed, 15 insertions, 5 deletions
diff --git a/contrib/sepgsql/Makefile b/contrib/sepgsql/Makefile index 536de2b2241..7fee218db9a 100644 --- a/contrib/sepgsql/Makefile +++ b/contrib/sepgsql/Makefile @@ -4,7 +4,10 @@ MODULE_big = sepgsql OBJS = hooks.o selinux.o label.o dml.o \ schema.o relation.o proc.o DATA_built = sepgsql.sql + REGRESS = label dml misc +REGRESS_OPTS = --launcher $(top_builddir)/contrib/sepgsql/launcher + EXTRA_CLEAN = -r tmp *.pp sepgsql-regtest.if sepgsql-regtest.fc ifdef USE_PGXS @@ -19,4 +22,3 @@ include $(top_srcdir)/contrib/contrib-global.mk endif SHLIB_LINK += -lselinux -REGRESS_OPTS += --launcher $(top_builddir)/contrib/sepgsql/launcher diff --git a/doc/src/sgml/extend.sgml b/doc/src/sgml/extend.sgml index ab538cb5008..35a5ae8fd42 100644 --- a/doc/src/sgml/extend.sgml +++ b/doc/src/sgml/extend.sgml @@ -1050,6 +1050,15 @@ include $(PGXS) </varlistentry> <varlistentry> + <term><varname>REGRESS_OPTS</varname></term> + <listitem> + <para> + additional switches to pass to <application>pg_regress</> + </para> + </listitem> + </varlistentry> + + <varlistentry> <term><varname>EXTRA_CLEAN</varname></term> <listitem> <para> diff --git a/src/makefiles/pgxs.mk b/src/makefiles/pgxs.mk index 05ed8416a9f..8b089e7a761 100644 --- a/src/makefiles/pgxs.mk +++ b/src/makefiles/pgxs.mk @@ -38,6 +38,7 @@ # SCRIPTS_built -- script files (not binaries) to install into $PREFIX/bin, # which need to be built first # REGRESS -- list of regression test cases (without suffix) +# REGRESS_OPTS -- additional switches to pass to pg_regress # EXTRA_CLEAN -- extra files to remove in 'make clean' # PG_CPPFLAGS -- will be added to CPPFLAGS # PG_LIBS -- will be added to PROGRAM link line @@ -225,10 +226,8 @@ distclean maintainer-clean: clean ifdef REGRESS -# Calling makefile can set REGRESS_OPTS, but this is the default: -ifndef REGRESS_OPTS -REGRESS_OPTS = --dbname=$(CONTRIB_TESTDB) -endif +# Select database to use for running the tests +REGRESS_OPTS += --dbname=$(CONTRIB_TESTDB) # where to find psql for running the tests PSQLDIR = $(bindir) |