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

Fix bundle gem on path with spaces #4816

Merged
merged 3 commits into from Aug 4, 2021

Conversation

deivid-rodriguez
Copy link
Member

What was the end-user or developer problem that led to this PR?

bundler would fail to initialize the fresh git repository of new gems when bundle gem is run inside a path with spaces.

What is your fix for the problem, implemented in this PR?

Properly shellescape target folder.

Closes #4622.

Make sure the following tasks are checked

@deivid-rodriguez deivid-rodriguez force-pushed the fix_bundle_gem_on_path_with_spaces branch from 971bac5 to c31b8cd Compare August 4, 2021 07:54
Copy link
Member

@olleolleolle olleolleolle left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💯

The tests were improved, too. Much easier to follow, and the context used is now for "the exceptional", whereas the regular cases are ordinary its. Super!

@deivid-rodriguez
Copy link
Member Author

Thanks for your review @olleolleolle!

@deivid-rodriguez deivid-rodriguez merged commit e56c785 into master Aug 4, 2021
@deivid-rodriguez deivid-rodriguez deleted the fix_bundle_gem_on_path_with_spaces branch August 4, 2021 13:32
deivid-rodriguez added a commit that referenced this pull request Aug 17, 2021
…aces

Fix `bundle gem` on path with spaces

(cherry picked from commit e56c785)
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.

Unable to create gem using Bundler
3 participants