automake-patches
[Top][All Lists]
Advanced

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

[PATCH] {maint} configure: remove extraneous 'eval's from AM_RUN_LOG inv


From: Stefano Lattarini
Subject: [PATCH] {maint} configure: remove extraneous 'eval's from AM_RUN_LOG invocations
Date: Wed, 21 Dec 2011 20:00:27 +0100

* configure.ac: Remove extra 'eval's from AM_RUN_LOG invocations;
for example, instead of "AM_RUN_LOG([eval $PERL --version])",
simply use "AM_RUN_LOG([$PERL --version])"
---
 ChangeLog    |    7 +++++++
 configure    |   16 ++++++++--------
 configure.ac |    8 ++++----
 3 files changed, 19 insertions(+), 12 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index 49b6e8b..6c43b27 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,10 @@
+2011-12-21  Stefano Lattarini  <address@hidden>
+
+       configure: remove extraneous 'eval's from AM_RUN_LOG invocations
+       * configure.ac: Remove extra 'eval's from AM_RUN_LOG invocations;
+       for example, instead of "AM_RUN_LOG([eval $PERL --version])",
+       simply use "AM_RUN_LOG([$PERL --version])"
+
 2011-12-20  Peter Rosin  <address@hidden>
 
        tests: fix spurious failure on systems lacking unistd.h
diff --git a/configure b/configure
index 7f3a0db..87c9eee 100755
--- a/configure
+++ b/configure
@@ -2336,8 +2336,8 @@ if test -z "$PERL"; then
    as_fn_error $? "perl not found" "$LINENO" 5
 fi
 # Save details about the selected perl interpreter in config.log.
-{ echo "$as_me:$LINENO: eval $PERL --version" >&5
-   (eval $PERL --version) >&5 2>&5
+{ echo "$as_me:$LINENO: $PERL --version" >&5
+   ($PERL --version) >&5 2>&5
    ac_status=$?
    echo "$as_me:$LINENO: \$? = $ac_status" >&5
    (exit $ac_status); }
@@ -2437,8 +2437,8 @@ $as_echo_n "checking whether autoconf is installed... " 
>&6; }
 if ${am_cv_autoconf_installed+:} false; then :
   $as_echo_n "(cached) " >&6
 else
-  if { echo "$as_me:$LINENO: eval $am_AUTOCONF --version" >&5
-   (eval $am_AUTOCONF --version) >&5 2>&5
+  if { echo "$as_me:$LINENO: $am_AUTOCONF --version" >&5
+   ($am_AUTOCONF --version) >&5 2>&5
    ac_status=$?
    echo "$as_me:$LINENO: \$? = $ac_status" >&5
    (exit $ac_status); };
@@ -2462,8 +2462,8 @@ if ${am_cv_autoconf_works+:} false; then :
 else
   mkdir conftest
 echo 'AC''_INIT' > conftest/conftest.ac
-if { echo "$as_me:$LINENO: cd conftest && eval $am_AUTOCONF -o /dev/null 
conftest.ac" >&5
-   (cd conftest && eval $am_AUTOCONF -o /dev/null conftest.ac) >&5 2>&5
+if { echo "$as_me:$LINENO: cd conftest && $am_AUTOCONF -o /dev/null 
conftest.ac" >&5
+   (cd conftest && $am_AUTOCONF -o /dev/null conftest.ac) >&5 2>&5
    ac_status=$?
    echo "$as_me:$LINENO: \$? = $ac_status" >&5
    (exit $ac_status); };
@@ -2488,8 +2488,8 @@ if ${am_cv_autoconf_version+:} false; then :
 else
   mkdir conftest
 echo 'AC'"_PREREQ([$required_autoconf_version])" > conftest/conftest.ac
-if { echo "$as_me:$LINENO: cd conftest && eval $am_AUTOCONF -o /dev/null 
conftest.ac" >&5
-   (cd conftest && eval $am_AUTOCONF -o /dev/null conftest.ac) >&5 2>&5
+if { echo "$as_me:$LINENO: cd conftest && $am_AUTOCONF -o /dev/null 
conftest.ac" >&5
+   (cd conftest && $am_AUTOCONF -o /dev/null conftest.ac) >&5 2>&5
    ac_status=$?
    echo "$as_me:$LINENO: \$? = $ac_status" >&5
    (exit $ac_status); };
diff --git a/configure.ac b/configure.ac
index b01026d..9004b61 100644
--- a/configure.ac
+++ b/configure.ac
@@ -61,7 +61,7 @@ if test -z "$PERL"; then
    AC_MSG_ERROR([perl not found])
 fi
 # Save details about the selected perl interpreter in config.log.
-AM_RUN_LOG([eval $PERL --version])
+AM_RUN_LOG([$PERL --version])
 $PERL -e 'require 5.006;' || {
    AC_MSG_ERROR(
 [perl 5.6 or better is required; perl 5.8.2 or better
@@ -110,7 +110,7 @@ AM_MISSING_PROG([HELP2MAN], [help2man])
 
 required_autoconf_version=2.62
 AC_CACHE_CHECK([whether autoconf is installed], [am_cv_autoconf_installed],
-[if AM_RUN_LOG([eval $am_AUTOCONF --version]);
+[if AM_RUN_LOG([$am_AUTOCONF --version]);
 then
   am_cv_autoconf_installed=yes
 else
@@ -124,7 +124,7 @@ fi
 AC_CACHE_CHECK([whether autoconf works], [am_cv_autoconf_works],
 [mkdir conftest
 echo 'AC''_INIT' > conftest/conftest.ac
-if AM_RUN_LOG([cd conftest && eval $am_AUTOCONF -o /dev/null conftest.ac]);
+if AM_RUN_LOG([cd conftest && $am_AUTOCONF -o /dev/null conftest.ac]);
 then
   am_cv_autoconf_works=yes
 else
@@ -140,7 +140,7 @@ AC_CACHE_CHECK([whether autoconf is recent enough], 
[am_cv_autoconf_version],
 [mkdir conftest
 dnl Creative quoting required to avoid spurious expansion of AC_PREREQ macro
 echo 'AC'"_PREREQ([[$required_autoconf_version]])" > conftest/conftest.ac
-if AM_RUN_LOG([cd conftest && eval $am_AUTOCONF -o /dev/null conftest.ac]);
+if AM_RUN_LOG([cd conftest && $am_AUTOCONF -o /dev/null conftest.ac]);
 then
   am_cv_autoconf_version=yes
 else
-- 
1.7.7.3




reply via email to

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