automake-patches
[Top][All Lists]
Advanced

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

[FYI] {minor} TAP driver: no need to invoke AC_PROG_AWK directly


From: Stefano Lattarini
Subject: [FYI] {minor} TAP driver: no need to invoke AC_PROG_AWK directly
Date: Tue, 24 Dec 2013 22:06:00 +0100

It is already required by AM_INIT_AUTOMAKE anyway.

* doc/automake.texi: Adjust examples.
* t/tap-doc2.sh: Adjust documentation-tracking test.
* m4/init.m4 (AM_INIT_AUTOMAKE): Explicitly tell that the AC_PROG_AWK
requirement is also needed whenever the TAP driver is used.

Signed-off-by: Stefano Lattarini <address@hidden>
---
 doc/automake.texi | 8 +++-----
 m4/init.m4        | 4 ++--
 t/tap-doc2.sh     | 1 -
 3 files changed, 5 insertions(+), 8 deletions(-)

diff --git a/doc/automake.texi b/doc/automake.texi
index cd33ad7..645e729 100644
--- a/doc/automake.texi
+++ b/doc/automake.texi
@@ -9650,10 +9650,9 @@ Currently, the TAP driver that comes with Automake 
requires some by-hand
 steps on the developer's part (this situation should hopefully be improved
 in future Automake versions).  You'll have to grab the @file{tap-driver.sh}
 script from the Automake distribution by hand, copy it in your source tree,
-add a call to @code{AC_PROG_AWK} in @file{configure.ac} to search for a
-proper awk program, and use the Automake support for third-party test
-drivers to instruct the harness to use the @file{tap-driver.sh} script
-and that awk program to run your TAP-producing tests.  See the example
+and use the Automake support for third-party test drivers to instruct the
+harness to use the @file{tap-driver.sh} script and the awk program found
+by @code{AM_INIT_AUTOMAKE} to run your TAP-producing tests.  See the example
 below for clarification.
 
 Apart from the options common to all the Automake test drivers
@@ -9709,7 +9708,6 @@ AC_CONFIG_AUX_DIR([build-aux])
 AM_INIT_AUTOMAKE([foreign -Wall -Werror])
 AC_CONFIG_FILES([Makefile])
 AC_REQUIRE_AUX_FILE([tap-driver.sh])
-AC_PROG_AWK
 AC_OUTPUT
 
 % @kbd{cat Makefile.am}
diff --git a/m4/init.m4 b/m4/init.m4
index 432ff20..05ad294 100644
--- a/m4/init.m4
+++ b/m4/init.m4
@@ -90,8 +90,8 @@ AC_REQUIRE([AC_PROG_MKDIR_P])dnl
 # <http://lists.gnu.org/archive/html/automake/2012-07/msg00001.html>
 # <http://lists.gnu.org/archive/html/automake/2012-07/msg00014.html>
 AC_SUBST([mkdir_p], ['$(MKDIR_P)'])
-# We need awk for the "check" target.  The system "awk" is bad on
-# some platforms.
+# We need awk for the "check" target (and possibly the TAP driver).  The
+# system "awk" is bad on some platforms.
 AC_REQUIRE([AC_PROG_AWK])dnl
 AC_REQUIRE([AC_PROG_MAKE_SET])dnl
 AC_REQUIRE([AM_SET_LEADING_DOT])dnl
diff --git a/t/tap-doc2.sh b/t/tap-doc2.sh
index ddac739..d260d9a 100644
--- a/t/tap-doc2.sh
+++ b/t/tap-doc2.sh
@@ -34,7 +34,6 @@ AC_CONFIG_AUX_DIR([build-aux])
 AM_INIT_AUTOMAKE([foreign -Wall -Werror])
 AC_CONFIG_FILES([Makefile])
 AC_REQUIRE_AUX_FILE([tap-driver.sh])
-AC_PROG_AWK
 AC_OUTPUT
 END
 
-- 
1.8.5.rc0.23.gaa27064




reply via email to

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