automake-ng
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[Automake-NG] [FYI] [ng] check: deps for 'all' target listed in '$(am.al


From: Stefano Lattarini
Subject: [Automake-NG] [FYI] [ng] check: deps for 'all' target listed in '$(am.all.targets)'
Date: Mon, 30 Jul 2012 16:28:50 +0200

* automake.in (handle_all_and_check): Drop transform 'ALL-DEPS' when
processing the 'all-target.am' file.
(generate_makefile): Define '$(am.all.targets)' to the list of
dependencies for the 'all' target.
* lib/am/all-target.am: Use it instead of the transform '%ALL-DEPS%'.
* t/java.sh: Relax grepping checks to avoid spurious failures.
* t/amopts-variable-expansion.sh: Likewise.

Signed-off-by: Stefano Lattarini <address@hidden>
---
 automake.in                    | 5 +++--
 lib/am/all-target.am           | 2 +-
 t/amopts-variable-expansion.sh | 2 +-
 t/java.sh                      | 2 --
 4 files changed, 5 insertions(+), 6 deletions(-)

diff --git a/automake.in b/automake.in
index a11d90a..a25ba74 100644
--- a/automake.in
+++ b/automake.in
@@ -3887,7 +3887,7 @@ sub handle_footer
 # Generate "make all" and "make check" rules.
 sub handle_all_and_check ()
 {
-  almost_verbatim ('all-target', 'ALL-DEPS' => "@all");
+  almost_verbatim ('all-target');
   almost_verbatim ('check-target');
 }
 
@@ -6892,7 +6892,8 @@ sub generate_makefile ($$)
   define_variable 'am.all-ltlibs', INTERNAL, sort keys %known_ltlibraries;
   # Must come after invocation of several of the 'handle_*' functions
   # above, which can declare additional dependencies for the 'check'
-  # target.
+  # and 'all' target.
+  define_variable ('am.all.targets', INTERNAL, @all);
   define_variable ('am.test-suite.deps', INTERNAL, @check);
   handle_tests;
 
diff --git a/lib/am/all-target.am b/lib/am/all-target.am
index 285e902..c14ff17 100644
--- a/lib/am/all-target.am
+++ b/lib/am/all-target.am
@@ -38,5 +38,5 @@ $(foreach x,$(am.built-early),$(eval -include 
.am/built-sources/$(x)))
        @touch $@
 endif
 
-all-am: all-local %ALL-DEPS%
+all-am: all-local $(am.all.targets)
 all: $(if $(SUBDIRS),all-recursive,all-am)
diff --git a/t/amopts-variable-expansion.sh b/t/amopts-variable-expansion.sh
index 830373a..a11f576 100755
--- a/t/amopts-variable-expansion.sh
+++ b/t/amopts-variable-expansion.sh
@@ -57,6 +57,6 @@ AUTOMAKE_run
 grep '^Makefile\.am:.*sub/foo\.c.*requires.*AM_PROG_CC_C_O' stderr
 grep README stderr && exit 1
 $EGREP '(install|override)' stderr && exit 1
-$EGREP 'distdir|\.tar' Makefile.in && exit 1
+$EGREP 'distdir|\.tar[ .]' Makefile.in && exit 1
 
 :
diff --git a/t/java.sh b/t/java.sh
index 0103eea..631f98b 100755
--- a/t/java.sh
+++ b/t/java.sh
@@ -33,8 +33,6 @@ $AUTOCONF
 $AUTOMAKE
 
 $EGREP '\.stamp|class' Makefile.in # For debugging.
-grep '^all[-a-z]*:.*classjava\.stamp' Makefile.in
-test $(grep -c '^all[-a-z]*:.*classjava\.stamp' Makefile.in) -eq 1
 
 echo 'class a { }' > a.java
 echo 'class b { }' > b.java
-- 
1.7.12.rc0




reply via email to

[Prev in Thread] Current Thread [Next in Thread]