You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@httpd.apache.org by tr...@apache.org on 2012/09/30 23:55:13 UTC

svn commit: r1392120 - /httpd/httpd/trunk/modules/config5.m4

Author: trawick
Date: Sun Sep 30 21:55:13 2012
New Revision: 1392120

URL: http://svn.apache.org/viewvc?rev=1392120&view=rev
Log:
Fix shell errors when trying to AC_MSG_RESULT() for --with-module.
(The change to IFS hosed any AC_MSG_foo() invocations.)

It is easy enough to reorder the reset of IFS and the AC_MSG_RESULT()
invocation, but there's a big stretch of shell code where AC_MSG_foo()
might be useful someday, so get rid of the commas at the start and
leave IFS alone and AC_MSG_foo() at will.

Modified:
    httpd/httpd/trunk/modules/config5.m4

Modified: httpd/httpd/trunk/modules/config5.m4
URL: http://svn.apache.org/viewvc/httpd/httpd/trunk/modules/config5.m4?rev=1392120&r1=1392119&r2=1392120&view=diff
==============================================================================
--- httpd/httpd/trunk/modules/config5.m4 (original)
+++ httpd/httpd/trunk/modules/config5.m4 Sun Sep 30 21:55:13 2012
@@ -3,7 +3,7 @@ AC_ARG_WITH(module,
   APACHE_HELP_STRING(--with-module=module-type:module-file,
                      Enable module-file in the modules/<module-type> directory.),
   [
-    as_save_IFS="$IFS"; IFS=","
+    withval=`echo $withval | sed -e 's/,/ /'`
     for mod in $withval
     do
       modtype=`echo $mod | sed -e's/\(.*\):.*/\1/'`
@@ -51,7 +51,6 @@ EOF
     if test ! -z "$EXTRA_MODLIST"; then
       AC_MSG_RESULT(added:$EXTRA_MODLIST)
     fi
-    IFS="$as_save_IFS"
   ],
   [ AC_MSG_RESULT(none) 
   ])