]> granicus.if.org Git - php/commitdiff
Fix bug #65088 (Generated configure script is malformed on OpenBSD).
authorAdam Harvey <aharvey@php.net>
Sun, 23 Jun 2013 18:01:36 +0000 (11:01 -0700)
committerAdam Harvey <aharvey@php.net>
Sun, 23 Jun 2013 18:02:00 +0000 (11:02 -0700)
We fell foul of a "common urban legend"[0], which resulted in us backslash
escaping double quotes that we didn't need to in acinclude.m4. This worked fine
on most shells, but OpenBSD's ksh implementation really didn't like it.

[0] http://www.gnu.org/savannah-checkouts/gnu/autoconf/manual/autoconf-2.69/html_node/Shell-Substitutions.html

NEWS
acinclude.m4

diff --git a/NEWS b/NEWS
index 9612928523d2b9278a0484311f8e3db98b0e7272..be774f84f75f42d3bc32cd91a046b65c3747219e 100644 (file)
--- a/NEWS
+++ b/NEWS
@@ -2,6 +2,10 @@ PHP                                                                        NEWS
 |||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
 ?? ??? 2013, PHP 5.4.18
 
+- Core:
+  . Fixed bug #65088 (Generated configure script is malformed on OpenBSD).
+    (Adam)
+
 - CLI server:
   . Fixed bug #65066 (Cli server not responsive when responding with 422 http
     status code). (Adam)
index 350e4989bb6b258709bbb7ba6b707e539882ca16..d5912ad10112dce8d2d248b4b4b09a73dad90fbb 100644 (file)
@@ -66,10 +66,10 @@ AC_DEFUN([PHP_EXPAND_PATH],[
     $2=$1
   else
     changequote({,})
-    ep_dir="`echo $1|$SED 's%/*[^/][^/]*/*$%%'`"
+    ep_dir=`echo $1|$SED 's%/*[^/][^/]*/*$%%'`
     changequote([,])
-    ep_realdir="`(cd \"$ep_dir\" && pwd)`"
-    $2="$ep_realdir/`basename \"$1\"`"
+    ep_realdir=`(cd "$ep_dir" && pwd)`
+    $2="$ep_realdir"/`basename "$1"`
   fi
 ])