From: Andres Freund Date: Mon, 14 Nov 2016 22:53:07 +0000 (-0800) Subject: Provide NO_INSTALLCHECK option for pgxs. X-Git-Tag: REL_10_BETA1~1406 X-Git-Url: http://git.postgresql.org/gitweb/?a=commitdiff_plain;h=ffa8c3d8521bbd7c6da497264f1a12572a1ff713;p=postgresql.git Provide NO_INSTALLCHECK option for pgxs. This allows us to avoid running the regression tests in contrib modules like pg_stat_statement in a less ugly manner. Discussion: <22432.1478968242@sss.pgh.pa.us> --- diff --git a/contrib/pg_stat_statements/Makefile b/contrib/pg_stat_statements/Makefile index f1a45eb17a8..298951a5f50 100644 --- a/contrib/pg_stat_statements/Makefile +++ b/contrib/pg_stat_statements/Makefile @@ -13,6 +13,9 @@ LDFLAGS_SL += $(filter -lm, $(LIBS)) REGRESS_OPTS = --temp-config $(top_srcdir)/contrib/pg_stat_statements/pg_stat_statements.conf REGRESS = pg_stat_statements +# Disabled because these tests require "shared_preload_libraries=pg_stat_statements", +# which typical installcheck users do not have (e.g. buildfarm clients). +NO_INSTALLCHECK = 1 ifdef USE_PGXS PG_CONFIG = pg_config @@ -24,7 +27,3 @@ top_builddir = ../.. include $(top_builddir)/src/Makefile.global include $(top_srcdir)/contrib/contrib-global.mk endif - -# Disabled because these tests require "shared_preload_libraries=pg_stat_statements", -# which typical installcheck users do not have (e.g. buildfarm clients). -installcheck: REGRESS= diff --git a/doc/src/sgml/extend.sgml b/doc/src/sgml/extend.sgml index e19c657d8fe..f9d91a39235 100644 --- a/doc/src/sgml/extend.sgml +++ b/doc/src/sgml/extend.sgml @@ -1193,6 +1193,15 @@ include $(PGXS) + + NO_INSTALLCHECK + + + don't define an installcheck target, useful e.g. if tests require special configuration, or don't use pg_regress + + + + EXTRA_CLEAN diff --git a/src/makefiles/pgxs.mk b/src/makefiles/pgxs.mk index 2b4d684e8e7..c27004ecfbe 100644 --- a/src/makefiles/pgxs.mk +++ b/src/makefiles/pgxs.mk @@ -40,6 +40,8 @@ # which need to be built first # REGRESS -- list of regression test cases (without suffix) # REGRESS_OPTS -- additional switches to pass to pg_regress +# NO_INSTALLCHECK -- don't define an installcheck target, useful e.g. if +# tests require special configuration, or don't use 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 @@ -268,8 +270,10 @@ ifndef PGXS endif # against installed postmaster +ifndef NO_INSTALLCHECK installcheck: submake $(REGRESS_PREP) $(pg_regress_installcheck) $(REGRESS_OPTS) $(REGRESS) +endif ifdef PGXS check: