TS-1852: fix missing AC_PROG_SED on CentOS 5.9

Project: http://git-wip-us.apache.org/repos/asf/trafficserver/repo
Commit: http://git-wip-us.apache.org/repos/asf/trafficserver/commit/16df6a2a
Tree: http://git-wip-us.apache.org/repos/asf/trafficserver/tree/16df6a2a
Diff: http://git-wip-us.apache.org/repos/asf/trafficserver/diff/16df6a2a

Branch: refs/heads/sphinx-docs
Commit: 16df6a2a0c9e9ccec2d699aa8817935559be6df1
Parents: 9c2b7ec
Author: James Peach <[email protected]>
Authored: Thu Apr 25 09:54:58 2013 -0700
Committer: James Peach <[email protected]>
Committed: Thu Apr 25 09:55:19 2013 -0700

----------------------------------------------------------------------
 CHANGES      |    2 ++
 configure.ac |   19 +++++++++++++------
 2 files changed, 15 insertions(+), 6 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/trafficserver/blob/16df6a2a/CHANGES
----------------------------------------------------------------------
diff --git a/CHANGES b/CHANGES
index b2c319b..c21c558 100644
--- a/CHANGES
+++ b/CHANGES
@@ -2,6 +2,8 @@
   Changes with Apache Traffic Server 3.3.3
 
 
+  *) [TS-1852] Fix missing AC_PROG_SED on CentOS 5.9.
+
   *) [TS-1717] Fix the static build as much as possible.
 
   *) [TS-1851] Turn HostDBInfo back into a POD type.

http://git-wip-us.apache.org/repos/asf/trafficserver/blob/16df6a2a/configure.ac
----------------------------------------------------------------------
diff --git a/configure.ac b/configure.ac
index efa003b..9f2ed72 100644
--- a/configure.ac
+++ b/configure.ac
@@ -577,13 +577,18 @@ if test "x$enable_ccache" = "xyes"; then
     fi
 fi
 
-dnl AC_PROG_SED is only avaliable in recent autoconf versions.
-dnl Use AC_CHECK_PROG instead if AC_PROG_SED is not present.
-ifdef([AC_PROG_SED],
-      [AC_PROG_SED],
-      [AC_CHECK_PROG(SED, sed, sed)])
+dnl AC_PROG_SED is only available from version 2.6 (released in 2003). CentosOS
+dnl 5.9 still has an ancient version, but we have macros that require
+dnl AC_PROG_SED. The actual AC_PROG_SED macro does functional checks, but here
+dnl we define a trivial local version for times when we are running on
+dnl obsoldete autoconf.
+ifdef([AC_PROG_SED], [], [
+  AC_DEFUN([AC_PROG_SED], [
+    AC_CHECK_PROG(SED, sed, sed)
+  ])
+])
 
-dnl Same with AM_PROG_AR, but it doesn't seem to be needed in older versions.
+dnl AM_PROG_AR is not always available, but it doesn't seem to be needed in 
older versions.
 ifdef([AM_PROG_AR],
       [AM_PROG_AR])
 
@@ -592,6 +597,7 @@ AC_PROG_CPP
 AC_PROG_CXXCPP
 AM_PROG_AS
 AC_PROG_AWK
+AC_PROG_SED
 AC_PROG_LN_S
 AC_PROG_INSTALL
 AC_PROG_LIBTOOL
@@ -599,6 +605,7 @@ AC_CHECK_PROG(RM, rm, rm)
 AC_CHECK_PROG(ASCPP, cpp, cpp)
 AC_CHECK_TOOL(AR, ar, ar)
 AC_ISC_POSIX
+
 # Do bison check by hand because we must do a version check.
 # Use YACC because it makes autotools shut up.
 BISON_MAJOR=2

Reply via email to