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] "strace ./app.exe" probably runs application from /bin


Christopher Faylor wrote:
...
Thanks for the problem report and test case but this is pretty clearly
not the right way to deal with the issue. Putting a special case catch
of "./" around a function call which is intended to deal with paths is
pretty clearly a band-aid.

Yes.



Let me rephrase the problem:

"cygpath does not properly deal with the current directory"


It depends. Removing "./" from a path is usually OK. Except when application search should be enabled if a path is missing.


I don't know whether this current behaviour of path.cc:cygpath() is required somewhere else
(cygcheck.cc, dump_setup.cc ...).


Thats the reason why I didn't modify cygpath() itself and used this (workaround|hack) instead.

Christian


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