On 9/25/06, Peter Samuelson <peter@p12n.org> wrote:
>
> [Erik Huelsmann]
> > I started working on a patch to record the size of the (unedited)
> > working copy file in the entries file. Using 1 stat() call and this
> > value, we can detect whether the working copy file has changed,
> > without the need to detranslate.
>
> If you're adding fields to the entries file anyway, with the aim of
> independence from mtimes, I suggest an inode number field too. (Yes,
> apr makes an inode number available on win32 as well as unix). The
> idea is to require the inode number to match, as well as the mtime,
> when deciding to skip the file compare.
>
> This prevents a misunderstanding with 'svn update foo1 foo2; mv foo1
> foo2'. (The 'svn update' will give foo1 and foo2 approximately the
> same mtime, and 'mv' does not change it, so at present this will not
> detect that foo2 has changed.)
Would this change eliminate the need for svn_sleep_for_timestamps()?
Currently we have 'sleep' statements in many places in Subversion to
help ensure that mtimes will be different.
David
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Wed Sep 27 19:56:09 2006