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 for #20: Handle Windows file paths from WSL #27

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

henning-erlandsen
Copy link

Running vagrant under WSL can confuse ruby as to how it should treat file paths. In this case it will treat the entire path as a filename, for example "C:\path\to\a\file.ext" will be parsed as

File.dirname(src) => "./"
File.basename(src) => "C:\path\to\a\file"

Replacing the back-slashes makes ruby recognize the windows path.

@rockdreamer
Copy link

Thank you for this, it has fixed the plugin for me on WSL.

@nathan-fiscaletti
Copy link

It's been nearly three months, @sprotheroe any chance that this will get merged?

miikkij added a commit to miikkij/vagrant-disksize that referenced this pull request May 28, 2019
@samuliasmala
Copy link

I can confirm it worked nicely so would be great to get this merged!

@cdausmus
Copy link

I've hit this issue too, can we get this merged please?

@maica1
Copy link

maica1 commented Dec 28, 2021

I've needed this too, tried this fix and worked.
Thank you.

@mathix420
Copy link

Needed here too! Would be nice to merge that PR

@rajadain
Copy link

I really need this merged as well. Any ideas how to use this variation until this is merged?

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

Successfully merging this pull request may close these issues.

8 participants