-
Notifications
You must be signed in to change notification settings - Fork 24.6k
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
Added quotes to allow spaces in installation path #8428
Conversation
Thanks for the PR. Please could I ask you to sign the CLA so that we can merge it in? @gmarz please could you review this? |
@clintongormley Jumped through the hoop: signed :) |
LGTM. Thanks @vidarkongsli 👍 |
LOL |
Merged. Thanks again @vidarkongsli |
@gmarz can you make sure this issue doesn't exist in our |
@s1monw In 1.3.x this worked fine for me in the past, but maybe you have a similar problem in later 1.3 branches. |
@s1monw Confirmed, this is not an issue in |
thx @gmarz |
Closes #8441