We had a hard time overriding php.ini settings with periods in them (e.g. xdebug.remote_enable = On). We had to change the sed a bit. Assigning to Greg for review and commit.

--- /usr/local/bin/drush5    2012-11-12 20:54:46.000000000 +0000
+++ /usr/local/bin/drushnbc    2013-01-04 16:17:43.000000000 +0000
@@ -80,7 +80,7 @@
   php_options="--php-ini $drush_php_ini"
 fi
 if [ "x$drush_php_override" != "x" ] ; then
-  php_options=`grep '^[a-z_A-Z0-9]\+ *=' $drush_php_override | sed -e 's|\([^ =]*\) *= *\(.*\)|\1="\2"|' -e 's| ||g' -e 's|^|-d |' | tr '\n\r' '  '`
+  php_options=`grep '^[a-z_A-Z0-9.]\+ *=' $drush_php_override | sed -e 's|\([^ =]*\) *= *\(.*\)|\1="\2"|' -e 's| ||g' -e 's|^|-d |' | tr '\n\r' '  '`
 fi
 
 # Pass in the path to php so that drush knows which one

Comments

moshe weitzman’s picture

Status: Active » Needs review
greg.1.anderson’s picture

Yes, that's good; committed #0.

moshe weitzman’s picture

Version: 8.x-6.x-dev » 7.x-5.x-dev
Status: Needs review » Patch (to be ported)

For cherry pick: af3abf1

greg.1.anderson’s picture

Status: Patch (to be ported) » Fixed

Committed to 7.x-5.x

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.