Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add options for composer and php binaries #48

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

koenpunt
Copy link
Contributor

Added :composer_bin and :composer_php options for better control of
which binaries to use.
When not explicitly set, the existence of the composer and php
executables is now validated before the tasks are run.

Also Composer is now always invoked through php. For this the path to
the composer executable is expanded by calling which composer on
the remote host.

When :composer_bin is set to :local, composer:install_executable
is automatically invoked on deployment. So no more manual hooking of
additional tasks in the deploy configuration.

This more or less supersedes #41, but I think this is more flexible.

Added `:composer_bin` and `:composer_php` options for better control of
which binaries to use.
When not explicitly set, the existence of the `composer` and `php`
executables is now validated before the tasks are run.

Also Composer is now always invoked through php. For this the path to
the `composer` executable is expanded by calling `which composer` on
the remote host.

When `:composer_bin` is set to `:local`, `composer:install_executable`
is automatically invoked on deployment. So no more manual hooking of
additional tasks in the deploy configuration.
@koenpunt
Copy link
Contributor Author

ping ✨

@ndench
Copy link

ndench commented Apr 9, 2017

This PR would be very helpful since I'm using composer installed as a .phar, so it's not in the default location. Would be great to be able to specify the location of composer.

@lucasmezencio
Copy link

Up!

@dhyegocalota
Copy link

👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants