Makefile improvements.

* Makefile.in (lib): Depend on am--refresh, to avoid a race.
(src): Remove duplicate dependency on FRC.
Invoke just one submake, not two.  Avoid the need for 'pwd'.
* lwlib/Makefile.in (all): Put this first.  Don't use double-colon
rules, as they are not portable according to POSIX.  Mark as phony.
* oldXMenu/Makefile.in (all, tags): Don't use double-colon rules, as they
are not portable according to POSIX.  Mark as phony.
* src/Makefile.in (config.status): Don't use double-colon rules, as
they are not portable according to POSIX.  Fix shell typo with `;
I guess this rule has never been tested?
(VCSWITNESS): New macro, to override any environment var.

diff --git a/ChangeLog b/ChangeLog
index ba6c22c13b6ac5272c003fa293d4baa2aafda486..b25d30f42ca0fd9f215e0dea0eb457728de4c451 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,10 @@
+2013-09-04  Paul Eggert  <eggert@cs.ucla.edu>
+
+       Makefile improvements.
+       * Makefile.in (lib): Depend on am--refresh, to avoid a race.
+       (src): Remove duplicate dependency on FRC.
+       Invoke just one submake, not two.  Avoid the need for 'pwd'.
+
 2013-09-02  Jan Djärv  <jan.h.d@swipnet.se>
 
        * configure.ac: Add ns_check_file.

diff --git a/Makefile.in b/Makefile.in
index 8e3a846c8dfe187ec4fa38b31f726438d46a7467..204099cc202814614c53d5dcdac014df4b31befc 100644 (file)
--- a/Makefile.in
+++ b/Makefile.in
@@ -352,9 +352,13 @@ epaths-force-w32: FRC
          -e "/^.*#/s|@SRC@|$${w32srcdir}|g") &&                \
        ${srcdir}/build-aux/move-if-change epaths.h.$$$$ src/epaths.h
 
+# If lib/Makefile would build files in '.', then build them before
+# building 'lib', to avoid races with parallel makes.
+lib: am--refresh
+
 lib-src src: $(NTDIR) lib
 
-src:   lib-src FRC
+src: lib-src
 
 # We need to build `emacs' in `src' to compile the *.elc files in `lisp'
 # and `leim'.
@@ -376,18 +380,15 @@ lib lib-src lisp leim nt: Makefile FRC
 # all preloaded elisp files, and only then dump the actual src/emacs, which
 # is not wrong, but is overkill in 99.99% of the cases.
 src: Makefile FRC
-       boot=bootstrap-emacs$(EXEEXT);                         \
-       if [ ! -x "src/$$boot" ]; then                                     \
-           cd $@; $(MAKE) all $(MFLAGS)                                   \
-             CC='${CC}' CFLAGS='${CFLAGS}' CPPFLAGS='${CPPFLAGS}'         \
-             LDFLAGS='${LDFLAGS}' MAKE='${MAKE}' BOOTSTRAPEMACS="$$boot"; \
-       fi;
-       if [ -r .bzr/checkout/dirstate ]; then                  \
-           vcswitness="`pwd`/.bzr/checkout/dirstate";  \
-       fi;                                                     \
-       cd $@; $(MAKE) all $(MFLAGS)                           \
-         CC='${CC}' CFLAGS='${CFLAGS}' CPPFLAGS='${CPPFLAGS}' \
-         LDFLAGS='${LDFLAGS}' MAKE='${MAKE}' BOOTSTRAPEMACS="" \
+       dirstate='.bzr/checkout/dirstate';                              \
+       vcswitness='$$(srcdir)/../'$$dirstate;                          \
+       [ -r "$(srcdir)/$$dirstate" ] || vcswitness='';                 \
+       cd $@ || exit;                                                  \
+       boot=bootstrap-emacs$(EXEEXT);                                  \
+       [ ! -x "$$boot" ] || boot='';                                   \
+       $(MAKE) all $(MFLAGS)                                           \
+         CC='${CC}' CFLAGS='${CFLAGS}' CPPFLAGS='${CPPFLAGS}'          \
+         LDFLAGS='${LDFLAGS}' MAKE='${MAKE}' BOOTSTRAPEMACS="$$boot"   \
          VCSWITNESS="$$vcswitness"
 
 blessmail: Makefile src FRC

diff --git a/lwlib/ChangeLog b/lwlib/ChangeLog
index a9e424ac4b312b58f66175c6e95aeff7b5cb48cf..5a1018f85e5f9c7cad8417a2a564a783e83b33f6 100644 (file)
--- a/lwlib/ChangeLog
+++ b/lwlib/ChangeLog
@@ -1,3 +1,9 @@
+2013-09-04  Paul Eggert  <eggert@cs.ucla.edu>
+
+       Makefile improvements.
+       * Makefile.in (all): Put this first.  Don't use double-colon
+       rules, as they are not portable according to POSIX.  Mark as phony.
+
 2013-06-30  Paul Eggert  <eggert@cs.ucla.edu>
 
        Fix minor problems found by static checking.

diff --git a/lwlib/Makefile.in b/lwlib/Makefile.in
index f228221c45dad202983f212f9ca3966e475c3471..40f4a405e97027cd16faf4989f050588c66239ea 100644 (file)
--- a/lwlib/Makefile.in
+++ b/lwlib/Makefile.in
@@ -59,11 +59,12 @@ ALL_CFLAGS= $(C_SWITCH_SYSTEM) $(C_SWITCH_X_SITE) \
   -Demacs -I../src \
   -I$(srcdir) -I$(srcdir)/../src -I../lib -I$(srcdir)/../lib
 
+all: liblw.a
+.PHONY: all
+
 .c.o:
        $(CC) -c $(CPPFLAGS) $(ALL_CFLAGS) $<
 
-all:: liblw.a
-
 liblw.a: $(OBJS)
        rm -f $@
        $(AR) $(ARFLAGS) $@ $(OBJS)

diff --git a/oldXMenu/ChangeLog b/oldXMenu/ChangeLog
index 8f2951700e8836be695054f251adf3638ae223eb..21acbe830530a010c448988398208b42335324d0 100644 (file)
--- a/oldXMenu/ChangeLog
+++ b/oldXMenu/ChangeLog
@@ -1,3 +1,9 @@
+2013-09-04  Paul Eggert  <eggert@cs.ucla.edu>
+
+       Makefile improvements.
+       * Makefile.in (all, tags): Don't use double-colon rules, as they
+       are not portable according to POSIX.  Mark as phony.
+
 2012-12-04  Paul Eggert  <eggert@cs.ucla.edu>
 
        Include <config.h> uniformly in oldXMenu sources.

diff --git a/oldXMenu/Makefile.in b/oldXMenu/Makefile.in
index b08e6649cbd201cc7433a2e36321519b45623762..394bfb3ab500498f547a5b5d0a80653f0bb9c511 100644 (file)
--- a/oldXMenu/Makefile.in
+++ b/oldXMenu/Makefile.in
@@ -85,7 +85,8 @@ OBJS =  Activate.o \
        SetSel.o \
       XDelAssoc.o XLookAssoc.o XCrAssoc.o XDestAssoc.o XMakeAssoc.o
 
-all:: libXMenu11.a
+all: libXMenu11.a
+.PHONY: all
 
 ALL_CFLAGS=$(C_SWITCH_SYSTEM) $(C_SWITCH_MACHINE) \
   $(C_SWITCH_X_SITE) \
@@ -139,5 +140,6 @@ distclean: clean
        rm -f Makefile
 maintainer-clean: distclean
 
-tags::
+tags:
        $(TAGS) -t *.[ch]
+.PHONY: tags

diff --git a/src/ChangeLog b/src/ChangeLog
index c355232482f1f2b5d101ae929e1cf2765422dccb..39d2f2f412a1f2106536877e445b822b475df93c 100644 (file)
--- a/src/ChangeLog
+++ b/src/ChangeLog
@@ -1,3 +1,11 @@
+2013-09-04  Paul Eggert  <eggert@cs.ucla.edu>
+
+       Makefile improvements.
+       * Makefile.in (config.status): Don't use double-colon rules, as
+       they are not portable according to POSIX.  Fix shell typo with `;
+       I guess this rule has never been tested?
+       (VCSWITNESS): New macro, to override any environment var.
+
 2013-09-04  Dmitry Antipov  <dmantipov@yandex.ru>
 
        * xterm.h (struct x_display_info): Do not track X connection

diff --git a/src/Makefile.in b/src/Makefile.in
index 0e7ff9cbd33f76570260c8d91f2e3d140e3fcba1..fe8d2d13ce074a8d777eeda86bd94aaf87549da6 100644 (file)
--- a/src/Makefile.in
+++ b/src/Makefile.in
@@ -538,14 +538,9 @@ stamp-oldxmenu: $(OLDXMENU_DEPS)
 
 $(OLDXMENU): $(OLDXMENU_TARGET)
 
-../config.status:: epaths.in
-       @echo "The file epaths.h needs to be set up from epaths.in."
-       @echo "Please run the `configure' script again."
-       exit 1
-
-../config.status:: config.in
-       @echo "The file config.h needs to be set up from config.in."
-       @echo "Please run the `configure' script again."
+../config.status: config.in epaths.in
+       @echo "The file ${?:.in=.h} needs to be set up from $?."
+       @echo "Please run the 'configure' script again."
        exit 1
 
 doc.o: buildobj.h
@@ -658,6 +653,9 @@ $(lisp): $(BOOTSTRAPEMACS)
 
 ## VCSWITNESS points to the file that holds info about the current checkout.
 ## We use it as a heuristic to decide when to rebuild loaddefs.el.
+## If empty it is ignored; the parent makefile can set it to some other value.
+VCSWITNESS =
+
 $(lispsource)/loaddefs.el: $(BOOTSTRAPEMACS) $(VCSWITNESS)
        cd ../lisp; $(MAKE) $(MFLAGS) autoloads EMACS=$(bootstrap_exe)