[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: svn commit: r23767 - in trunk/subversion: include libsvn_wc

From: Erik Huelsmann <ehuels_at_gmail.com>
Date: 2007-03-10 12:44:33 CET

On 3/10/07, dionisos@tigris.org <dionisos@tigris.org> wrote:
> Author: dionisos
> Date: Sat Mar 10 03:22:25 2007
> New Revision: 23767
> Log:
> Make the 'working file has changed' heuristic more robust:
> check for the size of the file too.

For weeks (months?) now, people on users@ have been arguing we're
inconsistent about treating mtimes: On one side we say we won't record
(and restore) them because it's impossible to depend on them, yet on
the other side we use them as the *only* indicator in our heuristic
for 'changedness'-detection. This is a problem for some people and has
shown to be a problem with 'recode' which restores the timestamp of a
file to what it was before it got recoded (bleh!).

This commit extends our heuristic to consider the size of the file as
an additional indicator.

I hope this eliminates the discussion and strengthens people's belief
in Subversion's robustness.



To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Sat Mar 10 12:44:49 2007

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.