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

Re: Official way to create an empty revision

From: Daniel Shahaf <d.s_at_daniel.shahaf.name>
Date: Sun, 5 Oct 2014 19:53:06 +0000

Konstantin Kolinko wrote on Thu, Oct 02, 2014 at 03:40:51 +0400:
> 2014-10-02 2:59 GMT+04:00 Daniel Shahaf <d.s_at_daniel.shahaf.name>:
> > Julian Foad wrote on Wed, Oct 01, 2014 at 10:52:42 +0100:
> >> Daniel Shahaf wrote in the thread "No no-op changes":
> >> > Should we provide an "official" way to create an empty revision? That
> >> > is, a revision whose changed-paths list is empty?
> >> >
> >> > Use-cases:
> >> >
> >> > 1. Suppose last backup is r100 and revisions r101:r105 were lost; then
> >> > after restoring the backup, the admin would create 5 empty revisions.
> >> >
> >> > 2. Force an empty revision for whatever reason, such as to make the
> >> > revnums sync to something:
> >> > 2.1. See r3 of the regression test merge_tests.py#125 svnmucc_abuse_1().
> >> > 2.2. W hen loading our repository to the ASF repository, if Joe had
> >> > created 26 empty revisions, then The Offset would have been 840100
> >> > rather than 840074, which would make our mental math easier.
> >>
> >> What should the author and log message be on the empty revs? I suppose
> >> these need to be optionally specified, defaulting to blank?
> >>
>
> My thought:
>
> svnadmin bump -m "message" REPOS_PATH
> svnrdump bump -m "message" URL
>
> The command creates 1 empty revision and thus bumps the repository
> revision number. It can be repeated in a loop as necessary.
>

Two proof-of-concept patches implementing this are attached. They work,
but some bells and whistles are missing (e.g., setting the log message
from 'svnadmin bump' to a boilerplate value or to a user-provided value).

I'm not going to commit them just yet --- I'd rather let the idea soak
for a bit. (In part because the 1.9 branch point is around the corner.)

> > The author of a revision created by svnadmin is "the administrator"; we
> > have not defined a way to represent this value in an svn:author revprop.
> >
>
> If dependent tools can deal with missing svn:author property then it
> is OK to do not have one.
>
> Looking at authors.txt file used to configure svn-git mirroring for
> ASF [1], there is the following line at the end of the file:
>
> (no author) = No Author <dev-null_at_apache.org>
>
> So I think git-svn can deal with revisions that do not have svn:author
> property, and thus it is safe to create such revisions.
>

And even if git-svn didn't implement this, it would _still_ be okay to
create revisions with no svn:author revprops, since we stipulated that
was permissible a decade+ ago when we defined our API. Naturally, if
that were the case, we'd heads-up the git-svn folks and work with them
to minimize the impact on users.

Received on 2014-10-05 21:53:52 CEST

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.