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

Re: Should we maintain the backward comaptiblity with Subversion Operational Logging?

From: Malcolm Rowe <malcolm-svn-dev_at_farside.org.uk>
Date: 2007-06-26 13:07:17 CEST

On Tue, Jun 26, 2007 at 04:07:13PM +0530, Kamesh Jayachandran wrote:
> The recent enhancements to Subversion operational logging to trunk won't be
> backward compatible.

I don't really know how much of a problem that is - did we ever promise
that it would be?

> But I have one more improvement in mind that won't allow adding fields at
> the end.
> The improvement is consistent style of fields like revision should be the
> last field, so that parsing would be easier.
> i.e In 1.5.0-dev for 'checkout' we added revision number for commit and we
> added the 'common-parent-of changed-paths' to commit, adding this new info
> at the end would break the above improvement of consistent logging.

I couldn't parse that, sorry.

>
> 1.4.4 1.5.0-dev
> -----------------------------------------------------------------
> commit r8 commit '/trunk'
> r8
> list-dir /trunk list-dir
> '/trunk' r8
> blame /trunk/test.c blame
> '/trunk/test.c' r8:17
> diff-or-merge '/left_path' '/right_path' diff-or-merge '/trunk' r5:9
> Or diff-or-merge '/left_path@84' '/right_path@94'
> switch '/source_path' '/destination_path' switch '/left_path@84'
> '/right_path@94'
>
> In summary I would like to go-ahead with the format of
> 'command path(s) rREV".

While that's not a bad idea, I don't think we can shoehorn all possible
commands into that format. See, for example, the diff and switch
commands above. But I'm all for consistency when there's no reason not
to switch.

> This needs a change to 'revprop-change' log and 'replay'
> revprop-change r8, 'svn:log' -> revprop-change 'svn:log' r8

I'm -0 on that change: svn:log isn't a path, and for revprop-changes the
revision is the immediate target of the change.

> replay 8 '/trunk' -> replay '/trunk' r8

On the other hand, that looks like a good change to make.

Regards,
Malcolm

  • application/pgp-signature attachment: stored
Received on Tue Jun 26 13:27:58 2007

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