[Patch] "strace ./app.exe" probably runs application from /bin

[Christian Franke]

Brian Dessent wrote:
> Christopher Faylor wrote:
>
>   
>> Let me rephrase the problem:
>>
>> "cygpath does not properly deal with the current directory"
>>
>> Thanks for the patch but we won't be applying it in this form.
>>     
>
> I've been meaning to take a look at fixing this myself, because I'm
> tired of:
>
> $ cd /usr/bin
>
> $ cygcheck ./ls
> .\.\.\.\ - Cannot open
>
> $ cygcheck ls
>  - Cannot open
> Error: could not find ls
>
> $ cygcheck ls.exe
>  - Cannot open
> Error: could not find ls.exe
>
> $ cygcheck ./ls.exe
> .\ls.exe
>   .\cygwin1.dll
>     C:\WINXP\system32\ADVAPI32.DLL
>       C:\WINXP\system32\ntdll.dll
>       C:\WINXP\system32\KERNEL32.dll
>       C:\WINXP\system32\RPCRT4.dll
>   .\cygintl-8.dll
>     .\cygiconv-2.dll
>
>   

This is a different and subtle issue in cygcheck itself:

The init_path() routine adds cwd first.
add_path() does not add duplicate path names later.
If cwd is C:\cygwin\bin, it only appears in the sysdirs part of the path.

find_on_path(.,.,.,0) does never check a PATH component equal to cwd.

The attached patch is a quick hack (again, sorry ;-) to fix this.

Christian

-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: cygwin-1.5.24-2-cygcheck.patch.txt
URL: <http://cygwin.com/pipermail/cygwin-patches/attachments/20070603/23d0c25d/attachment.txt>