in former days we said in our sitealias

'%drush-script' => '/esoteric/pathto/php /custom/drush/drush.php',

to tell our custom php path.

this does not work anymore current dev drush uses /usr/bin/php

CommentFileSizeAuthor
#4 1163234-set-php-options-in-alias-4.patch3.54 KBgreg.1.anderson
#1 0001-fix-1163234-remoteremote-custom-php-path-breaks-drus.patch2.52 KBgeek-merlin

Comments

geek-merlin’s picture

geek-merlin
Primary language German
Location Freiburg, Germany
commented
Status: Active » Needs review
StatusFileSize
new 0001-fix-1163234-remoteremote-custom-php-path-breaks-drus.patch2.52 KB

reworked this a bit so that we can have 'php' and 'php-options' keys in our sitealias file.

docs included.

greg.1.anderson’s picture

greg.1.anderson commented
Status: Needs review » Needs work

This does not work for me.

drush @local status

Does not call drush_redispatch_get_options, so patch has no effect.

drush @remote status

Calls 'drush' script, so php-options have no effect.

#1 gets us closer to supporting this feature, but it does not yet work. Could have some issues in common with #1171722: path-aliases for %drush %drush-script not working as documented?.

greg.1.anderson’s picture

greg.1.anderson commented

Correction to #2: drush @remote status works with this patch; I had a typo in my test. Local aliases should work too, though, but ignore 'php-options' as mentioned in #2.

greg.1.anderson’s picture

greg.1.anderson commented
Status: Needs work » Needs review
Issue tags: +Windows
StatusFileSize
new 1163234-set-php-options-in-alias-4.patch3.54 KB

Here is an adjustment to #1 that allows php-options to be set for local aliases as well.

Tagging as 'Windows' since this is a continuation of the spaces-in-paths fix recently done for Windows compatibility.

greg.1.anderson’s picture

greg.1.anderson commented

The patch from #4 has been rolled in to #1058874: drush_backend_invoke buffers output till end of command. Show progress.. If that issue is committed, this one may be closed; if this issue is committed, that one will need to be re-rolled.

greg.1.anderson’s picture

greg.1.anderson commented
Status: Needs review » Fixed

Committed a while ago.

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