fixes composer not using specified {{bin/php}}

[riconeitzel]

Q	A
Bug fix?	Yes
New feature?	No
BC breaks?	No
Deprecations?	No
Fixed tickets	N/A

fixes missing {{bin/php}} in vendor task for composer calls.

forces composer calls to use the specified php binary

[antonmedv]

Please, update change log

[riconeitzel]

@antonmedv done :)

[antonmedv]

Somehow travis is now working.

[riconeitzel]

"did you try to turn it off and on again?" 🤣 it's green 💚

[loevgaard]

I think this is wrong. See

deployer/recipe/common.php

Lines 111 to 122 in 8293eb9

	set('bin/composer', function () {
	if (commandExist('composer')) {
	$composer = '{{bin/php}} ' . locateBinaryPath('composer');
	}
	if (empty($composer)) {
	run("cd {{release_path}} && curl -sS https://getcomposer.org/installer | {{bin/php}}");
	$composer = '{{bin/php}} {{release_path}}/composer.phar';
	}
	return $composer;
	});

[antonmedv]

Yes, @loevgaard is right

[riconeitzel]

Then there’s another bug. In my environment composer ran into issues without my patch and was used with php instead of my set bin/php call. I can look into it again.

[loevgaard]

Because you are running v6 probably :)

[riconeitzel]

Downloaded it 5 days ago - no version constraint configured. I’ll check it!

[riconeitzel]

Hi there …

I finally looked into it again.

I'm using 6.8.0 via composer - latest stable …

deployer/recipe/common.php

Lines 114 to 125 in 4e243a6

	set('bin/composer', function () {
	if (commandExist('composer')) {
	$composer = locateBinaryPath('composer');
	}
	if (empty($composer)) {
	run("cd {{release_path}} && curl -sS https://getcomposer.org/installer | {{bin/php}}");
	$composer = '{{bin/php}} {{release_path}}/composer.phar';
	}
	return $composer;
	});

so - the code @loevgaard mentions isn't part of the stable, yet 🤷🏼‍♂️ so in that case the only way is to use the beta … Am I right?

Rico

[loevgaard]

Or copy the code and overwrite the bin/composer param in your local deploy.php file