]> sourceware.org Git - automake.git/commitdiff
Build elisp files all at once instead of one by one.
authorAlexandre Duret-Lutz <adl@gnu.org>
Fri, 10 Jan 2003 17:25:33 +0000 (17:25 +0000)
committerAlexandre Duret-Lutz <adl@gnu.org>
Fri, 10 Jan 2003 17:25:33 +0000 (17:25 +0000)
* automake.in (handle_emacs_lisp): Define am__ELFILES.  Add
elc-stamp to all's dependencies.
* lib/am/lisp.am (elc-stamp): New rule, build all *.elc files.
(.el.elc): Rewrite to call elc-stamp if $@ doesn't exist.
(clean-lisp): Clean elc-stamp.
* lib/elisp-comp: Reindent.  Erase the temporatry directory
from a trap.  Propagate Emacs's exit status.
* tests/lisp3.test: New file.
* tests/Makefile.am (TESTS): Add lisp3.test.
Reported by Ryan T. Sammartino.

ChangeLog
NEWS
automake.in
lib/am/lisp.am
lib/elisp-comp
tests/Makefile.am
tests/Makefile.in
tests/lisp3.test [new file with mode: 0755]

index 6e43089c2a3ed31b5b7af01a9049f5abc2cecf1d..57dacf7c6f3afe79658133a3e6e928db59a23fcd 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,17 @@
+2003-01-10  Alexandre Duret-Lutz  <adl@gnu.org>
+
+       Build elisp files all at once instead of one by one.
+       * automake.in (handle_emacs_lisp): Define am__ELFILES.  Add
+       elc-stamp to all's dependencies.
+       * lib/am/lisp.am (elc-stamp): New rule, build all *.elc files.
+       (.el.elc): Rewrite to call elc-stamp if $@ doesn't exist.
+       (clean-lisp): Clean elc-stamp.
+       * lib/elisp-comp: Reindent.  Erase the temporatry directory
+       from a trap.  Propagate Emacs's exit status.
+       * tests/lisp3.test: New file.
+       * tests/Makefile.am (TESTS): Add lisp3.test.
+       Reported by Ryan T. Sammartino.
+
 2003-01-09  Akim Demaille  <akim@epita.fr>
 
        * automake.texi: Prefer $(FOO) to @FOO@ in the documentation.
diff --git a/NEWS b/NEWS
index 43962a5253959ace3a7b459c64eb55b4a699eb06..e1c9e2d0709872f9c90e9dfe31247e79f4bb875e 100644 (file)
--- a/NEWS
+++ b/NEWS
@@ -1,4 +1,6 @@
 New in 1.7a:
+* elisp sources are compiled all at once, instead of one by one.
+  This allows interdependencies and speed up compilation.
 * AM_PROG_CC_STDC is now empty.  The content of this macro was
   merged in AC_PROG_CC.  If your code uses $am_cv_prog_cc_stdc,
   you should adjust it to use $ac_cv_prog_cc_stdc instead.
@@ -288,8 +290,8 @@ New in 0.20:
 
 -----
 
-Copyright 1995, 1996, 1997, 1998, 1999, 2000, 2001 Free Software
-Foundation, Inc.
+Copyright 1995, 1996, 1997, 1998, 1999, 2000, 2001, 2002, 2003
+Free Software Foundation, Inc.
 
 This file is part of GNU Automake.
 
index 961e887e8eee94b521640f89d5f01e8f675966aa..65294e3c4b60f327b8cb438757f06373000e007a 100755 (executable)
@@ -4969,8 +4969,14 @@ sub handle_emacs_lisp
   # Generate .elc files.
   my @elcfiles = map { $_->[1] . 'c' } @elfiles;
   define_pretty_variable ('ELCFILES', TRUE, INTERNAL, @elcfiles);
-
-  push (@all, '$(ELCFILES)');
+  define_pretty_variable ('am__ELFILES', TRUE, INTERNAL,
+                         map { $_->[1] } @elfiles);
+
+  # It's important that all depends on elc-stamp so that
+  # all .elc files get recompiled whenever a .el changes.
+  # It's important that all depends on $(ELCFILES) so that
+  # we can recover if any of them is deleted.
+  push (@all, 'elc-stamp', '$(ELCFILES)');
 
   require_variables ($elfiles[0][0], "Emacs Lisp sources seen", TRUE,
                     'EMACS', 'lispdir');
index fa7ed0308c41748600ff3815bc0a1da3333f26f9..e9e6945893256f7e3283b23d120b27bfc37edf8f 100644 (file)
@@ -1,5 +1,6 @@
 ## automake - create Makefile.in from Makefile.am
-## Copyright 1996, 1998, 1999, 2000, 2001 Free Software Foundation, Inc.
+## Copyright 1996, 1998, 1999, 2000, 2001, 2002, 2003
+## Free Software Foundation, Inc.
 
 ## This program is free software; you can redistribute it and/or modify
 ## it under the terms of the GNU General Public License as published by
 ## Building.  ##
 ## ---------- ##
 
-.el.elc:
+elc-stamp: $(am__ELFILES)
        @echo 'WARNING: Warnings can be ignored. :-)'
        if test $(EMACS) != no; then \
-         EMACS=$(EMACS) $(SHELL) $(elisp_comp) $<; \
+## Make sure "$@" isn't empty initialy.
+         set x; \
+## Populate "$@" whith elisp files (found in the current directory
+## or in $srcdir).
+         list='$(am__ELFILES)'; for p in $$list; do \
+            if test -f "$$p"; then d=; else d="$(srcdir)/"; fi; \
+           set x "$$@" "$$d$$p"; shift; \
+         done; \
+## Finally call elisp-comp for all files.
+         shift; \
+         EMACS=$(EMACS) $(SHELL) $(elisp_comp) "$$@" || exit 1; \
+       else : ; fi
+       touch $@
+
+.el.elc:
+## Recover from the removal of $@
+       @if test ! -f $@; then \
+         rm -f elc-stamp; \
+         $(MAKE) $(AM_MAKEFLAGS) elc-stamp; \
        else : ; fi
 
 ## ------------ ##
@@ -87,7 +106,7 @@ endif %?INSTALL%
 
 .PHONY clean-am: clean-lisp
 clean-lisp:
-       -test -z "$(ELCFILES)" || rm -f $(ELCFILES)
+       -rm -f elc-stamp $(ELCFILES)
 
 
 ## -------------- ##
index dc7d7ca6d85a4bb4ad45a602f3078e839344ebab..031f00df9eab8852019f97a0e19d85d120549d69 100755 (executable)
@@ -1,5 +1,5 @@
 #!/bin/sh
-# Copyright 1995 Free Software Foundation, Inc.
+# Copyright (C) 1995, 2000, 2003  Free Software Foundation, Inc.
 # François Pinard <pinard@iro.umontreal.ca>, 1995.
 #
 # This program is free software; you can redistribute it and/or modify
 # they require or load-library one another.
 
 if test $# = 0; then
-   echo 1>&2 "No files given to $0"
-   exit 1
-else
-   if test -z "$EMACS" || test "$EMACS" = "t"; then
-      # Value of "t" means we are running in a shell under Emacs.
-      # Just assume Emacs is called "emacs".
-      EMACS=emacs
-   fi
-
-   tempdir=elc.$$
-   mkdir $tempdir
-   cp $* $tempdir
-   cd $tempdir
-
-   echo "(setq load-path (cons nil load-path))" > script
-   $EMACS -batch -q -l script -f batch-byte-compile *.el
-   mv *.elc ..
-
-   cd ..
-   rm -fr $tempdir
+  echo 1>&2 "No files given to $0"
+  exit 1
 fi
+
+if test -z "$EMACS" || test "$EMACS" = "t"; then
+  # Value of "t" means we are running in a shell under Emacs.
+  # Just assume Emacs is called "emacs".
+  EMACS=emacs
+fi
+
+tempdir=elc.$$
+
+# Cleanup the temporary directory on exit.
+trap 'status=$?; rm -rf "$tempdir" && exit $status' 0
+trap '(exit $?); exit' 1 2 13 15
+
+mkdir $tempdir
+cp "$@" $tempdir
+
+(
+  cd $tempdir
+  echo "(setq load-path (cons nil load-path))" > script
+  $EMACS -batch -q -l script -f batch-byte-compile *.el || exit $?
+  mv *.elc ..
+) || exit $?
+
+(exit 0); exit
index 7199dfcc6e4e0a9a4dbfe8fa2627ce527c8265cd..065d1c370c961b8b86cd94310f4b36883d49cd77 100644 (file)
@@ -247,6 +247,7 @@ link_f_cxx.test     \
 link_f_only.test \
 lisp.test \
 lisp2.test \
+lisp3.test \
 listval.test \
 location.test \
 ltdeps.test \
index 6a3823fefd6280b2a7da7f55ce14679aa17645e9..15620a67fb3cbfcd2fc4954b261847ba53e42834 100644 (file)
@@ -340,6 +340,7 @@ link_f_cxx.test     \
 link_f_only.test \
 lisp.test \
 lisp2.test \
+lisp3.test \
 listval.test \
 location.test \
 ltdeps.test \
diff --git a/tests/lisp3.test b/tests/lisp3.test
new file mode 100755 (executable)
index 0000000..46e698d
--- /dev/null
@@ -0,0 +1,73 @@
+#! /bin/sh
+# Copyright (C) 2003  Free Software Foundation, Inc.
+#
+# This file is part of GNU Automake.
+#
+# GNU Automake is free software; you can redistribute it and/or modify
+# it under the terms of the GNU General Public License as published by
+# the Free Software Foundation; either version 2, or (at your option)
+# any later version.
+#
+# GNU Automake is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+# GNU General Public License for more details.
+#
+# You should have received a copy of the GNU General Public License
+# along with GNU Automake; see the file COPYING.  If not, write to
+# the Free Software Foundation, Inc., 59 Temple Place - Suite 330,
+# Boston, MA 02111-1307, USA.
+
+# Test that compilings interdependent elisp files works.
+
+required=emacs
+. ./defs || exit 1
+
+set -e
+
+cat > Makefile.am << 'EOF'
+lisp_LISP = am-one.el am-two.el am-three.el
+EXTRA_DIST = am-one.el am-two.el
+am-three.el:
+       echo "(provide 'am-three)" > $@
+CLEANFILES = am-three.el
+EOF
+
+cat >> configure.in << 'EOF'
+AM_PATH_LISPDIR
+AC_OUTPUT
+EOF
+
+echo "(require 'am-two)" > am-one.el
+echo "(require 'am-three) (provide 'am-two)" > am-two.el
+# am-tree.el is a built source
+
+
+$ACLOCAL
+$AUTOCONF
+$AUTOMAKE --add-missing
+./configure
+
+$MAKE
+
+test -f am-one.elc
+test -f am-two.elc
+test -f am-three.elc
+test -f elc-stamp
+
+# Make sure we can recover from a deletion.
+rm -f am-one.elc
+$MAKE
+test -f am-one.elc
+
+# Make sure we build all files when any of them change.
+# (We grep a message to make sure the compilation happens.)
+unique=0a3346e2af8a689b85002b53df09142a
+sleep 2
+echo "(message \"$unique\")(provide 'am-three)" > am-three.el
+$MAKE >output 2>&1
+cat output
+grep $unique output
+
+# It should also work for VPATH-builds.
+$MAKE distcheck
This page took 0.058115 seconds and 5 git commands to generate.