This fixes another redirect-directly-to-target nit:

diff --git a/ChangeLog b/ChangeLog
index 79c6cc1..86cccd2 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,10 @@
+2007-11-13  Jim Meyering  <[EMAIL PROTECTED]>
+
+       Clean up the rule to create "expr".
+       * tests/Makefile.am (expr): Don't redirect directly to target.
+       Redirect just once, not for each echo statement.
+       Use $@, not literal "expr".
+
 2007-11-13  Paul Eggert  <[EMAIL PROTECTED]>

        Don't worry about preprocessor when testing long long.
diff --git a/tests/Makefile.am b/tests/Makefile.am
index 4cf7fd1..fd15563 100644
--- a/tests/Makefile.am
+++ b/tests/Makefile.am
@@ -195,14 +195,17 @@ maintainer-check: maintainer-check-posix 
maintainer-check-c++
 # The hairy heredoc is more robust than using echo.
 CLEANFILES += expr
 expr:
-       echo '#! $(SHELL)'            >expr
-       echo 'result=`$(EXPR) "$$@"`'>>expr
-       echo 'estatus=$$?'           >>expr
-       echo 'cat <<EOF'             >>expr
-       echo '$${result:-0}'         >>expr
-       echo 'EOF'                   >>expr
-       echo 'exit $$estatus'        >>expr
-       chmod +x expr
+       :;{                                     \
+         echo '#! $(SHELL)';                   \
+         echo 'result=`$(EXPR) "$$@"`';        \
+         echo 'estatus=$$?';                   \
+         echo 'cat <<EOF';                     \
+         echo '$${result:-0}';                 \
+         echo 'EOF';                           \
+         echo 'exit $$estatus';                \
+       } > [EMAIL PROTECTED]
+       chmod +x [EMAIL PROTECTED]
+       mv [EMAIL PROTECTED] $@

 # Try the test suite with more severe environments.
 maintainer-check-posix: expr
--
1.5.3.5.643.g40e25


Reply via email to