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