On Jul 18, 2013, at 2:52 , Anton Yartsev <[email protected]> wrote:

>> +  $PortablePath =~ s/^\/cygdrive\/(\w)/$1:/o if ($^O =~ /cygwin/);
>> 
>> The user is free to modify the mount path for /cygdrive/, but I guess that's 
>> okay. What I'm more confused about is how programs running inside Cygwin are 
>> supposed to use Windows-ish paths (still with forward slashes) to run 
>> executables. But I don't have a Windows machine available, so I'll trust you 
>> know what's right here. Please do try to hand-test all the code paths 
>> affected by this if you haven't already.
> Tested with pure cygwin, windows paths are unwillingly accepted, with 
> warnings; cygwin docs says "Using native Win32 paths in Cygwin, while 
> possible, is generally inadvisable."
> Decided to remove path conversion for now. This enhancement was added for 
> particular cases when cygwin is used together with other ports (MinGW + perl 
> from Cygwin + separate Python in my case).

I think committing the patch without this is the right thing to do right now, 
but for the record I'm perfectly happy to make this work in the long run...just 
want to make sure we do the right thing.

Jordan
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to