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

Use an absolute path for the global composer bin dir #310

Open
ghost opened this issue May 30, 2022 · 2 comments
Open

Use an absolute path for the global composer bin dir #310

ghost opened this issue May 30, 2022 · 2 comments
Labels
enhancement requested New feature suggested

Comments

@ghost
Copy link

ghost commented May 30, 2022

Expected Behavior

When running a binary through Symfony\Component\Process\ExecutableFinder it should be able to find the binary automatically.

Current Behavior

When using grumphp which relies on Symfony\Component\Process\ExecutableFinder the path that refers to $HOME with ~ does not work:

ENV PATH="$PATH:./vendor/bin:~/.composer/vendor/bin"

Possible Solution

You could make an argument that Symfony should be able to locate this better. But I think that modifying the PATH variable to include the absolute path for the composer binary is the easiest fix.

Steps to Reproduce (for bugs)

# Require package globally.
composer global require phpro/grumphp
# Try initializing the grumphp git hooks.
grumphp git:init
# Set an absolute path for composer binary.
export PATH=$PATH:/home/docker/.composer/vendor/bin
# Re run the initialization of grumphp git hooks.
grumphp git:init

Context

Trying to implement coding standards with the phpro/grumphp package installed globally.

Your Environment

Not relevant.

@stale
Copy link

stale bot commented Jul 29, 2022

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Please update it if any action still required.

@stale stale bot added the stale Issues inactives for a while (automatically) label Jul 29, 2022
@stale
Copy link

stale bot commented Aug 5, 2022

This issue has been automatically closed because it has not had recent activity. Please, reopen if you need.

@stale stale bot closed this as completed Aug 5, 2022
@mistraloz mistraloz added enhancement requested New feature suggested and removed stale Issues inactives for a while (automatically) labels Nov 3, 2022
@mistraloz mistraloz reopened this Nov 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement requested New feature suggested
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant