[Top][All Lists]
[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
- [Automake-NG] [FYI] [ng] check: deps for 'all' target listed in '$(am.all.targets)',
Stefano Lattarini <=