Senthil Kumaran S <sent...@collab.net> writes:

> In the previous versions of subversion ie., 1.5.x or 1.4.x the keyword
> will not expand, but this is not the case with 1.6.x in which it gets
> expanded for the first update. This is what the issue #3471 is about
> and the committed patch tests that.

I'm still confused.  The test creates an unexpanded keyword and then
runs update and the keyword gets expanded.  Are you saying that update
should not expand the keyword, because that's the way 1.5 behaved?

I think it's right that update expands the keyword, and so when the
timestamp changes that is also correct.

What is a bug is that a *second* update causes the file's timestamp to
get changed a second time.  A second update doesn't change the file
contents and so the timestamp should not change.  But the test doesn't
run a second update.  If the test did run a second update it could use
do_sleep_for_timestamps rather than sleep.

-- 
Philip

Reply via email to