> -----Original Message-----
> From: Vincent Lefevre [mailto:vincent-svn_at_vinc17.net]
> Sent: woensdag 4 juli 2012 14:46
> To: dev_at_subversion.apache.org
> Subject: Re: Incorrect LastChangedRev in working copy after committing
> directory move + modified file in subdir
>
> On 2012-07-04 13:28:21 +0100, Philip Martin wrote:
> > Vincent Lefevre <vincent-svn_at_vinc17.net> writes:
> > > The output ends with:
> > >
> > > + cat dir2/file
> > > $Header: file:///tmp/my-test-svn/svn/dir1/file 2 2012-07-04 11:57:43Z
> vlefevre $
> > > + svn cat file:///tmp/my-test-svn/svn/dir2/file_at_3
> > > $Header: file:///tmp/my-test-svn/svn/dir2/file 2 2012-07-04 11:57:43Z
> vlefevre $
> > >
> > > file:///tmp/my-test-svn/svn/dir1/file_at_2 exists but this isn't the
> > > real URL of the file.
> >
> > That's a keyword expansion bug, a variation on issue 1975.
> >
> > > file:///tmp/my-test-svn/svn/dir2/file_at_2 doesn't exist.
> > >
> > > IMHO, the most intuitive Header string should have
> > >
> > > file:///tmp/my-test-svn/svn/dir2/file 3
> >
> > Where would the revision 3 come from? LastChangedRev is 2. That's what
> > Subversion's cheap copy means.
>
> Yes, but I meant that LastChangedRev could be 3 after a move.
> I don't think this contradicts cheap copy: when doing
>
> svn cat file:///tmp/my-test-svn/svn/dir2/file_at_3
>
> Subversion gets the file via a COPY node or something like that
> (I don't know the exact internals) as the file hasn't changed,
> and it could get LastChangedRev from it.
>
> My point is that <URL>@<LastChangedRev> should always be a valid
> reference to the file, and it should be equivalent to <URL>@HEAD
> or just <URL>.
I think LastChangedRev should match the last changed rev of a plain
checkout.
I know there are problems in getting there, but your answer that
URL_at_LastChangedRev should work, would tell me that you think it is broken
for clean checkouts that contain files that were copied via their parent.
URL_at_revision -r <LastChangedRev> should get you the file as and at where it
was last changed
And URL_at_revision gives you a location that you can checkout from directly.
Bert
Received on 2012-07-04 14:55:25 CEST