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 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.

cgf


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