[PATCH] Reattach trailing dirsep on existing directories too.

Ray Donnelly mingw.android@gmail.com
Tue Jan 7 18:15:00 GMT 2014


On Tue, Jan 7, 2014 at 3:12 PM, Corinna Vinschen wrote:
> On Dec 22 01:03, Ray Donnelly wrote:
>> I hope this is OK and I've done it in the best place. Please advise if
>> it needs any changes.
>
> I have no idea if this is ok.  This is a patch to a very crucial
> function in terms of path handling, and it's not clear that this isn't
> doing the wrong thing.  What is this patch trying to accomplish?  Do you
> have example user space code which is failing for this very reason?

The exact issue was that paths that do not exist would maintain their
final dirsep whereas paths that do exist would lose this dirsep:

test.exe /c/doesnt-exist/ /c/does-exist/

test.exe would see:
arg1: C:/doesnt-exist/
arg2: C:/does-exist

These paths were passed to GCC as search paths and while I could've
hacked up the GCC code to detect and correct this anomaly, but I think
this patch fixes the problem at cause.



More information about the Cygwin-patches mailing list