OpenAFS Master Repository branch, master, updated. openafs-devel-1_5_76-3951-gdbf6822
Gerrit Code Review
gerrit@openafs.org
Mon, 30 Sep 2013 06:31:48 -0700 (PDT)
The following commit has been merged in the master branch:
commit dbf68229c6c90d65781fdc16565dd1e9b56248c0
Author: Michael Meffie <mmeffie@sinenomine.net>
Date: Mon Sep 9 22:25:50 2013 -0400
build: compile_et rules for parallel make
Change all makefile rules which run compile_et in order support parallel
make. The compile_et generates two outputs, so special care must be
taken in rules which run compile_et.
All the rules for compile_et have been changed to the form:
foo.c foo.h: foo.et
compile_et foo.et -h foo
foo.h: foo.c
The above rules are equivalent to:
foo.c: foo.et
compile_et foo.et -h foo
foo.h: foo.et foo.c
compile_et foo.et -h foo
therefore a parallel make will serialize the builds of foo.c and foo.h,
and should detect that the second is no longer needed once the first is
over. This form works since foo.et is not a phony target, and does not
depend on a phony target.
Previously, the rules for compile_et were of the one of the two forms:
a) foo.c foo.h: foo.et
compile_et foo.et -h foo
or
b) foo.h: foo.c
foo.c: foo.et
compile_et foo.et -h foo
Form a) is problematic for parallel makes, since it is equivalent to:
foo.c:
compile_et foo.et -h foo
foo.h:
compile_et foo.et -h foo
In a parallel make, compile_et will be run concurrently, clobbering
each other's output files.
Form b) is better, but is problematic when foo.h is removed, since foo.h
will not be updated.
Thanks to Russ Allbery for pointing out the automake documentation which
describes issues with commands that produce multiple outputs, and
portable solutions.
http://www.gnu.org/software/automake/manual/automake.html#Multiple-Outputs
Change-Id: I14c056606084f80270e05592d3d09a600f804e24
Reviewed-on: http://gerrit.openafs.org/10237
Reviewed-by: Benjamin Kaduk <kaduk@mit.edu>
Reviewed-by: Derrick Brashear <shadow@your-file-system.com>
Tested-by: BuildBot <buildbot@rampaginggeek.com>
src/afs/Makefile.in | 4 ++-
src/auth/Makefile.in | 2 +
src/bozo/Makefile.in | 5 +--
src/bubasics/Makefile.in | 15 +++++--------
src/bucoord/Makefile.in | 5 +--
src/budb/Makefile.in | 5 +--
src/cmd/Makefile.in | 4 +-
src/kauth/Makefile.in | 5 +--
src/libadmin/adminutil/Makefile.in | 37 +++++++++++++++++------------------
src/ptserver/Makefile.in | 3 +-
src/rxkad/Makefile.in | 5 +--
src/rxkad/test/Makefile.in | 4 ++-
src/tbudb/Makefile.in | 7 ++---
src/tools/dumpscan/Makefile.in | 4 +++
src/tptserver/Makefile.in | 4 +-
src/tvlserver/Makefile.in | 6 +++-
src/tvolser/Makefile.in | 6 +++-
src/ubik/Makefile.in | 5 +--
src/vlserver/Makefile.in | 7 ++++-
src/volser/Makefile.in | 5 +--
20 files changed, 72 insertions(+), 66 deletions(-)
--
OpenAFS Master Repository