Re: Something fishy about the current Makefiles - Mailing list pgsql-hackers

From Peter Eisentraut
Subject Re: Something fishy about the current Makefiles
Date
Msg-id 1294783859.17361.5.camel@vanquo.pezone.net
Whole thread Raw
In response to Something fishy about the current Makefiles  (Tom Lane <tgl@sss.pgh.pa.us>)
Responses Re: Something fishy about the current Makefiles  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-hackers
On tor, 2011-01-06 at 11:57 -0500, Tom Lane wrote:
> Whilst fooling around with GIN, I have repeatedly observed that doing
> "make" in src/backend/access/gin, followed by "make install-bin" in
> src/backend, fails to rebuild the postgres executable --- it just
> installs the existing one.  A second execution of "make install-bin"
> does notice that postgres is out of date and rebuilds it.  This
> procedure for rebuilding after changing one or two .c files has always
> worked for me before.  I can't avoid the suspicion that the recent
> changes to make things more parallel-friendly broke something.

This fixes it, but it's beyond me why.

diff --git i/src/backend/common.mk w/src/backend/common.mk
index 4e0a5da..5d599db 100644
--- i/src/backend/common.mk
+++ w/src/backend/common.mk
@@ -34,7 +34,7 @@ objfiles.txt: Makefile $(SUBDIROBJS) $(OBJS)expand_subsys = $(foreach file,$(1),$(if $(filter
%/objfiles.txt,$(file)),$(patsubst../../src/backend/%,%,$(addprefix $(top_builddir)/,$(shell cat $(file)))),$(file)))#
Parallelmake trickery
 
-$(SUBDIROBJS): | $(SUBDIRS:%=%-recursive) ;
+$(SUBDIROBJS): $(SUBDIRS:%=%-recursive) ;.PHONY: $(SUBDIRS:%=%-recursive)$(SUBDIRS:%=%-recursive):




pgsql-hackers by date:

Previous
From: Robert Haas
Date:
Subject: Re: Allowing multiple concurrent base backups
Next
From: David Fetter
Date:
Subject: Re: Allowing multiple concurrent base backups