[Bug 16352] Unimplemented Function in Scan-Build

[Jordan Rose]

On Jul 18, 2013, at 2:52 , Anton Yartsev <anton.yartsev at gmail.com> 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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130718/9f3b6802/attachment.html>