hmans / rbfu

Minimal Ruby version management is minimal.
MIT License
118 stars 10 forks source link

Honor whitespace in the 'path_remove' function #26

Closed kellyredding closed 12 years ago

kellyredding commented 12 years ago

The Problem

I have /Applications/Sublime Text 2.app/Contents/SharedSupport/bin in my $PATH. When rbfu runs its path_remove function, my path gets parsed and applied incorrectly (due to the whitespace in there).

It seems that path_remove function isn't honoring whitespace.

The Fix

I just had path_remove honor the whitespace when setting the new path variable.

The Test

I added a test bash script in there that demonstrates the problem and fix. I doubt you'll want/need to keep that around so I put it in its own commit so you can just cherry pick the fix commit.

Thanks.