-
Notifications
You must be signed in to change notification settings - Fork 2.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
Fix #374 shellescape problem #380
Conversation
@starcraftman just add .patch to the url Thanks for checking it out! |
@gmarik Thanks. That's pretty easy. I'll try it after lunch. |
I just tested this on Windows 7 64bit with Vim. Works as expected. Without the patch and with shellslash set I get an error. With the patch applied to the latest vundle it works as expected. Only warning I got is that there is a trailing whitespace on line 96 of the patch, otherwise works as advertised. |
Tested on OSX no issues found… |
Thank you! |
I fixed shellescape problem and tests.
I tested it, but it is not under Windows. Can you test this fix before including?