Skip to content

Commit 7786fe0

Browse files
committed
Revert ed47666 and part of ef57b98
This reverts ed47666, which ended up adding a second tempoary installation for all 'make check' runs, and reverts the part of ef57b98 that changed the TAP 'prove_check' section, which appears to have been unintentional to begin with on this branch. Analysis and patch by Noah.
1 parent 48c6a1a commit 7786fe0

File tree

1 file changed

+3
-19
lines changed

1 file changed

+3
-19
lines changed

src/Makefile.global.in

Lines changed: 3 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,6 @@ endif
5959
endif
6060
else # not PGXS
6161
vpath_build = @vpath_build@
62-
abs_top_builddir = @abs_top_builddir@
6362
abs_top_srcdir = @abs_top_srcdir@
6463

6564
ifneq ($(vpath_build),yes)
@@ -317,19 +316,6 @@ BZIP2 = bzip2
317316

318317
# Testing
319318

320-
check: temp-install
321-
322-
.PHONY: temp-install
323-
temp-install:
324-
ifndef NO_TEMP_INSTALL
325-
ifeq ($(MAKELEVEL),0)
326-
rm -rf '$(abs_top_builddir)'/tmp_install
327-
$(MKDIR_P) '$(abs_top_builddir)'/tmp_install/log
328-
$(MAKE) -C '$(top_builddir)' DESTDIR='$(abs_top_builddir)'/tmp_install install >'$(abs_top_builddir)'/tmp_install/log/install.log 2>&1
329-
endif
330-
$(if $(EXTRA_INSTALL),for extra in $(EXTRA_INSTALL); do $(MAKE) -C '$(top_builddir)'/$$extra DESTDIR='$(abs_top_builddir)'/tmp_install install >>'$(abs_top_builddir)'/tmp_install/log/install.log || exit; done)
331-
endif
332-
333319
PROVE = @PROVE@
334320
PG_PROVE_FLAGS = -I $(top_srcdir)/src/test/perl/
335321
PROVE_FLAGS = --verbose
@@ -344,10 +330,6 @@ define ld_library_path_var
344330
$(if $(filter $(PORTNAME),darwin),DYLD_LIBRARY_PATH,$(if $(filter $(PORTNAME),aix),LIBPATH,LD_LIBRARY_PATH))
345331
endef
346332

347-
define with_temp_install
348-
PATH="$(abs_top_builddir)/tmp_install$(bindir):$$PATH" $(call add_to_path,$(ld_library_path_var),$(abs_top_builddir)/tmp_install$(libdir))
349-
endef
350-
351333
ifeq ($(enable_tap_tests),yes)
352334

353335
define prove_installcheck
@@ -356,7 +338,9 @@ endef
356338

357339
define prove_check
358340
rm -rf $(CURDIR)/tmp_check/log
359-
cd $(srcdir) && TESTDIR='$(CURDIR)' $(with_temp_install) PGPORT='6$(DEF_PGPORT)' top_builddir='$(CURDIR)/$(top_builddir)' $(PROVE) $(PG_PROVE_FLAGS) $(PROVE_FLAGS) t/*.pl
341+
$(MKDIR_P) tmp_check/log
342+
$(MAKE) -C $(top_builddir) DESTDIR='$(CURDIR)'/tmp_check/install install >'$(CURDIR)'/tmp_check/log/install.log 2>&1
343+
cd $(srcdir) && TESTDIR='$(CURDIR)' PATH="$(CURDIR)/tmp_check/install$(bindir):$$PATH" $(call add_to_path,$(ld_library_path_var),$(CURDIR)/tmp_check/install$(libdir)) top_builddir='$(CURDIR)/$(top_builddir)' PGPORT='6$(DEF_PGPORT)' $(PROVE) $(PG_PROVE_FLAGS) $(PROVE_FLAGS) t/*.pl
360344
endef
361345

362346
else

0 commit comments

Comments
 (0)
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy