This is the mail archive of the cygwin-patches mailing list for the Cygwin project.

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH] Reattach trailing dirsep on existing directories too.

On Jan  7 18:15, Ray Donnelly wrote:
> 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.

And that is a problem, because...?


Corinna Vinschen                  Please, send mails regarding Cygwin to
Cygwin Maintainer                 cygwin AT cygwin DOT com
Red Hat

Attachment: pgp4GZ5RsRUx1.pgp
Description: PGP signature

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]