From: Tom Lane Date: Mon, 18 May 2015 00:04:42 +0000 (-0400) Subject: Use += not = to set makefile variables after including base makefiles. X-Git-Tag: REL9_5_ALPHA1~210 X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=b14cf229f4bd7238be2e31d873dc5dd241d3871e;p=postgresql Use += not = to set makefile variables after including base makefiles. The previous coding in hstore_plpython and ltree_plpython wiped out any values set by the base makefiles. This at least had the effect of running the tests in "regression" not "contrib_regression" as expected. These being pretty new modules, there might be other bad effects we'd not noticed yet. --- diff --git a/contrib/hstore_plpython/Makefile b/contrib/hstore_plpython/Makefile index 6125ea4d2c..6ee434bafa 100644 --- a/contrib/hstore_plpython/Makefile +++ b/contrib/hstore_plpython/Makefile @@ -28,10 +28,10 @@ ifeq ($(PORTNAME), win32) SHLIB_LINK += ../hstore/libhstore.a $(wildcard ../../src/pl/plpython/libpython*.a) $(wildcard ../../src/pl/plpython/libplpython*.a) endif -REGRESS_OPTS = --load-extension=hstore +REGRESS_OPTS += --load-extension=hstore ifeq ($(python_majorversion),2) REGRESS_OPTS += --load-extension=plpythonu --load-extension=hstore_plpythonu endif -EXTRA_INSTALL = contrib/hstore +EXTRA_INSTALL += contrib/hstore include $(top_srcdir)/src/pl/plpython/regress-python3-mangle.mk diff --git a/contrib/ltree_plpython/Makefile b/contrib/ltree_plpython/Makefile index a0c107c2c3..64ca1275f1 100644 --- a/contrib/ltree_plpython/Makefile +++ b/contrib/ltree_plpython/Makefile @@ -28,10 +28,10 @@ ifeq ($(PORTNAME), win32) SHLIB_LINK += $(wildcard ../../src/pl/plpython/libpython*.a) endif -REGRESS_OPTS = --load-extension=ltree +REGRESS_OPTS += --load-extension=ltree ifeq ($(python_majorversion),2) REGRESS_OPTS += --load-extension=plpythonu --load-extension=ltree_plpythonu endif -EXTRA_INSTALL = contrib/ltree +EXTRA_INSTALL += contrib/ltree include $(top_srcdir)/src/pl/plpython/regress-python3-mangle.mk