Skip to content

add options for composer and php binaries#48

Open
koenpunt wants to merge 3 commits into
capistrano:masterfrom
koenpunt:php_composer_options
Open

add options for composer and php binaries#48
koenpunt wants to merge 3 commits into
capistrano:masterfrom
koenpunt:php_composer_options

Conversation

@koenpunt

Copy link
Copy Markdown
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
Copy Markdown
Contributor Author

ping ✨

@ndench

ndench commented Apr 9, 2017

Copy link
Copy Markdown

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
Copy Markdown

Up!

@dhyegocalota

Copy link
Copy Markdown

👍

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