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.

Sign up for GitHub

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

Jump to bottom

Add /opt/drupal/vendor/bin to PATH #177

Closed
davidbarratt opened this issue Aug 8, 2020 · 3 comments · Fixed by #178
Closed

Add /opt/drupal/vendor/bin to PATH #177

davidbarratt opened this issue Aug 8, 2020 · 3 comments · Fixed by #178
Labels
Request Request for image modification or feature

Comments

@davidbarratt
Copy link
Contributor

davidbarratt commented Aug 8, 2020 •
edited
Loading

It would be really helpful if the /opt/drupal/vendor/bin folder were added to the PATH. This would make it possible to install Drush with composer and it would be usable without additional commands or configuration.
https://www.drush.org/install/
@davidbarratt davidbarratt changed the title Add COMPOSER_BIN_DIR to PATH Add /opt/drupal/vendor/bin to PATH Aug 8, 2020
@wglambert wglambert added the Request Request for image modification or feature label Aug 10, 2020
@yosifkit
Copy link
Member

Seems sane to me. What do you think @skyred?

@tianon
Copy link
Member

tianon commented Aug 11, 2020

I wasn't sure on COMPOSER_BIN_DIR, but adding /opt/drupal/vendor/bin to PATH seems 💯% reasonable/sane to me! 😄

@davidbarratt davidbarratt mentioned this issue Aug 12, 2020
Merged
@skyred
Copy link
Contributor

skyred commented Aug 12, 2020

oh yeah, this is super useful!

@WengerK WengerK mentioned this issue Aug 16, 2020
Merged
mstenta added a commit to mstenta/farmOS that referenced this issue Aug 17, 2020
@mstenta
The /opt/drupal/vendor/bin is now in PATH in the upstream drupal:9, s…
786ac6c 
…o vendor/bin prefix is no longer necessary.

See docker-library/drupal#177
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Request Request for image modification or feature
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add the composer bin folder to the PATH davidbarratt/docker-drupal
5 participants
@tianon @skyred @davidbarratt @yosifkit @wglambert