Bug Fix on Windows changing /USER~1/ to /User Name/ #542
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
🌈
This is to fix a bug that I (and others, see rspec#1555, rspec#1577, rspec#1747 and rails#27129) have experienced when running
rails generate rspec:install
on a windows machine.Ruby creates a tmpdir with
/USER~1/
and then thor uses another method,files(lookup)
, that converts it to/User Name/
and tries to compare the two and gsub it out when it assigns it tofile_destination
. My fix, though a little hacky (I'm new to Ruby and don't really know of a better way in all honesty) works and shouldn't create problems elsewhere as far as I can see.