From 02b40f6cdb7ec56f34c6b773b1d7768b4b135fd9 Mon Sep 17 00:00:00 2001 From: Ike Mulder Date: Fri, 26 Apr 2019 16:16:47 +0200 Subject: Fixed some escaping problems with arguments containing spaces in IDE's Compile buffer, and with building from a path containing spaces. Updated CHANGES.md Now using Filename.quote instead of enclosing in single quotes. Fixed rebasing problems. --- test-suite/Makefile | 14 +++++++------- 1 file changed, 7 insertions(+), 7 deletions(-) (limited to 'test-suite') diff --git a/test-suite/Makefile b/test-suite/Makefile index 265c2eafa7..6a6b729578 100644 --- a/test-suite/Makefile +++ b/test-suite/Makefile @@ -33,10 +33,10 @@ include ../Makefile.common ####################################################################### ifneq ($(wildcard ../_build),) -BIN:=$(shell cd ..; pwd)/_build/install/default/bin/ -COQLIB:=$(shell cd ..; pwd)/_build/install/default/lib/coq +BIN:='$(shell cd ..; pwd)'/_build/install/default/bin/ +COQLIB:='$(shell cd ..; pwd)'/_build/install/default/lib/coq else -BIN := $(shell cd ..; pwd)/bin/ +BIN := '$(shell cd ..; pwd)'/bin/ COQLIB?= ifeq ($(COQLIB),) @@ -602,10 +602,10 @@ $(patsubst %.sh,%.log,$(wildcard misc/*.sh)): %.log: %.sh $(PREREQUISITELOG) @echo "TEST $<" $(HIDE){ \ echo $(call log_intro,$<); \ - export BIN="$(BIN)"; \ - export coqc="$(coqc)"; \ - export coqtop="$(coqc)"; \ - export coqdep="$(coqdep)"; \ + export BIN=$(BIN); \ + export coqc="eval $(coqc)"; \ + export coqtop="eval $(coqc)"; \ + export coqdep="eval $(coqdep)"; \ "$<" 2>&1; R=$$?; times; \ if [ $$R = 0 ]; then \ echo $(log_success); \ -- cgit v1.2.3 From a4d8c9414e65271fc97347ae04b0b431206d33f8 Mon Sep 17 00:00:00 2001 From: Hugo Herbelin Date: Tue, 25 Feb 2020 13:53:03 +0100 Subject: Makefile in test-suite: More separation of concerns as suggested by Enrico. See "https://github.com/coq/coq/pull/10008#discussion_r382899607". --- test-suite/Makefile | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) (limited to 'test-suite') diff --git a/test-suite/Makefile b/test-suite/Makefile index 6a6b729578..1681150f7b 100644 --- a/test-suite/Makefile +++ b/test-suite/Makefile @@ -32,11 +32,15 @@ include ../Makefile.common # Variables ####################################################################### +# Using quotes to anticipate the possibility of spaces in the directory name +# Note that this will later need an eval in shell to interpret the quotes +ROOT='$(shell cd ..; pwd)' + ifneq ($(wildcard ../_build),) -BIN:='$(shell cd ..; pwd)'/_build/install/default/bin/ -COQLIB:='$(shell cd ..; pwd)'/_build/install/default/lib/coq +BIN:=$(ROOT)/_build/install/default/bin/ +COQLIB:=$(ROOT)/_build/install/default/lib/coq else -BIN := '$(shell cd ..; pwd)'/bin/ +BIN := $(ROOT)/bin/ COQLIB?= ifeq ($(COQLIB),) -- cgit v1.2.3