This is the mail archive of the mailing list for the GDB project.

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [patch] Fix parallel gdb/ make check//% site.exp

On Mon, 09 Nov 2009 16:37:42 +0100, Tom Tromey wrote:
> >>>>> "Jan" == Jan Kratochvil <> writes:
> Jan> -check-single: all site.exp
> Jan> +check-single: all ./site.exp
> I think make will still search vpath for "./site.exp".
> So, won't that cause the same problem?

A nice catch, verified now VPATH does work even for relative paths.  The only
safe way is to use absolute path there.
	make-3.81/ vpath_search(): 
	  /* If there are no VPATH entries or FILENAME starts at the root,
	     there is nothing we can do.  */

> What am I missing?

It was a testing mistake as when I edited ./Makefile I changed its timestamp
which did falsely "fix" it by:
	  Considering target file `site.exp'.
	    Pruning file `config.status'.
	    Pruning file `Makefile'.
	   Finished prerequisites of target file `site.exp'.
	   Prerequisite `config.status' is older than target `site.exp'.
	   Prerequisite `Makefile' is newer than target `site.exp'.
	  Must remake target `site.exp'.
	  Ignoring VPATH name `/home/jkratoch/redhat/gdb-clean/gdb/testsuite/site.exp'.

On Mon, 09 Nov 2009 17:38:04 +0100, Doug Evans wrote:
> but if the patch does go in, I'd like to see comments added to the
> code explaining why things are the way they are.
> site vs ./site.exp seems fairly subtle.



2009-11-09  Jan Kratochvil  <>

	* (abs_builddir): New.
	(site.exp): New target `$(abs_builddir)/site.exp'.  New comment.
	(check-single, $(TEST_TARGETS), check-gdb.base%): Change `site.exp' to

--- a/gdb/testsuite/
+++ b/gdb/testsuite/
@@ -22,6 +22,7 @@ VPATH = @srcdir@
 srcdir = @srcdir@
 prefix = @prefix@
 exec_prefix = @exec_prefix@
+abs_builddir = @abs_builddir@
 target_alias = @target_noncanonical@
 program_transform_name = @program_transform_name@
@@ -102,7 +103,11 @@ install:
 uninstall: force
-site.exp: ./config.status Makefile
+# Use absolute `site.exp' path everywhere to suppress VPATH lookups for it.
+# $(RUNTEST) is looking up `site.exp' only in the current directory.
+# Bare `site.exp' is present as a target here if user requests it explicitly.
+$(abs_builddir)/site.exp site.exp: ./config.status Makefile
 	@echo "Making a new config file..."
 	-@rm -f ./tmp?
 	@touch site.exp
@@ -147,7 +151,7 @@ DO_RUNTEST = \
 	  export TCL_LIBRARY ; fi ; \
-check-single: all site.exp
+check-single: all $(abs_builddir)/site.exp
 # A list of all directories named "gdb.*" which also hold a .exp file.
@@ -170,7 +174,7 @@ check-parallel:
 	$(SHELL) $(srcdir)/ -L \
 	  $(addsuffix /gdb.log,$(TEST_DIRS)) > gdb.log
-$(filter-out check-gdb.base%,$(TEST_TARGETS)): check-gdb.%: all site.exp
+$(filter-out check-gdb.base%,$(TEST_TARGETS)): check-gdb.%: all $(abs_builddir)/site.exp
 	@if test ! -d gdb.$*; then mkdir gdb.$*; fi
 	$(DO_RUNTEST) --directory=gdb.$* --outdir=gdb.$* $(RUNTESTFLAGS)
@@ -179,7 +183,7 @@ BASE1_FILES = $(patsubst $(srcdir)/%,%,$(wildcard $(srcdir)/gdb.base/[a-m]*.exp)
 BASE2_FILES = $(patsubst $(srcdir)/%,%,$(wildcard $(srcdir)/gdb.base/[n-z]*.exp))
 # Handle each half of gdb.base.
-check-gdb.base%: all site.exp
+check-gdb.base%: all $(abs_builddir)/site.exp
 	@if test ! -d gdb.base$*; then mkdir gdb.base$*; fi
 	$(DO_RUNTEST) $(BASE$*_FILES) --outdir gdb.base$* $(RUNTESTFLAGS)

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]