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

Composer path with spaces bugfix #126

Closed
wants to merge 2 commits into from
Closed

Conversation

dnquinn
Copy link

@dnquinn dnquinn commented Jan 13, 2022

Probably it works on linux environment, but on windows, if composer's executable file located by path that includes the spaces, command can't be executed. So this small fix should will prevent this error.

Probably it works on linux environment, but on windows, if composer's executable file located by path that includes the spaces, command can't be executed. So this small fix should will prevent this error.
@dnquinn
Copy link
Author

dnquinn commented Jan 13, 2022

In my case its stops with this type of error.

'C:\Program' is not recognized as an internal or external command, operable program or batch file.

Explict bug fix in 871bc77 commit
@clue
Copy link
Owner

clue commented Jan 25, 2022

@dnquinn Thank you for looking into this and filing this PR!

Your changes look very reasonable and I've indeed encountered a similar issue a few weeks ago. As a starting point, I've just filed #127 to address the build error, I'll look into this again once this addressed.

@dnquinn
Copy link
Author

dnquinn commented Jan 25, 2022

Thank you for nice lib. I've tried to solve the problem quickly as i can and only for my environment. So i am not suprised about failing the tests. Maybe you find more elegant way to fix it.

@clue
Copy link
Owner

clue commented Feb 14, 2022

@dnquinn Thank you, this has now been fixed via #128 (and #129) and I'll make sure to get this shipped asap! :shipit:

I hope this helps! 👍 If so, consider supporting this project, for example by becoming a sponsor ❤️

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

Successfully merging this pull request may close these issues.

2 participants