This is the mail archive of the cygwin-patches@cygwin.com 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]: ./.. changed during execution of find


On Sat, May 15, 2004 at 11:56:08AM -0400, Christopher Faylor wrote:
>On Fri, May 14, 2004 at 10:38:18PM -0400, Pierre A. Humblet wrote:
>>I have been rereading the chdir thread to see how things 
>>went wrong. The problem with find is due to the following change
>><http://cygwin.com/ml/cygwin-patches/2004-q2/msg00063.html>
>>
>>> That means that cwd.set always attempts to build the
>>> Posix wd through the mount table.
>>> Up to now that was only the case when a symlink was
>>> involved in the translation, or there was a ":" or a "\" 
>>> in the directory name, or check_case was not relaxed.
>>
>>It follows that "find /" has always been broken when 
>>check_case != "relax". 
>>
>>Please review carefully! In addition to fixing the find bug the
>>patch fixes the handling of paths such as c:xxx and it calls
>>SetCurrentDirectory inside the muto region.
>
>I don't think that would be a problem.  It makes sense to do that
>in fact.
>
>(famous last words)
>
>I've checked this in and am generating a snapshot now.

...which died immediately.  I think you forgot a corresponding cygheap.h
change.  I've checked in the required stuff.

Thanks for fixing this problem.

cgf


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