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

RE: 'svnadmin load-revprops' as first-level command?

From: Markus Schaber <m.schaber_at_codesys.com>
Date: Wed, 26 Apr 2017 06:25:58 +0000


From: Daniel Shahaf [mailto:d.s_at_daniel.shahaf.name]
> I was going through the 1.10 release notes and I found the new load-revprops
> / dump-revprops subcommands:
> https://subversion.apache.org/docs/release-notes/1.10#svnadmin-revprops
> New svnadmin dump-revprops and svnadmin load-revprops subcommands
> […] svnadmin dump-revprops will save the current values of all
> revision properties to a dump file. svnadmin load-revprops can be
> used to set revision properties in a repository to the values saved
> in a dump file.
> I don't dispute the need for such functionality, but is a separate top-level
> command the right home for it? I would have expected 'load-revprops' to be
> spelled "svnadmin load --only-revprops" or even "svndumpfilter revprops-only
> | svnadmin load".
> The rationale is that 'load' and 'load-revprops' take exactly the same data
> format and manipulate it in exactly the same way.

I guess that, especially for the "dump" case, "dump-revprops" is (or should be)
much more efficient than a "dump" piped through "svndumpfilter". For "load",
the additional overhead will be smaller, but still there. So I think this
functionality should be implemented in svnadmin itself.

Note, however, that I wouldn't mind it being an "--only-revprops" option to
the existing "load" and "dump" subcommands instead of a new subcommand.

And I also don't oppose the idea of "svndumpfilter revprops-only", which
clearly has its own use cases. :-)

Best regards

Markus Schaber

CODESYS® a trademark of 3S-Smart Software Solutions GmbH

Inspiring Automation Solutions

3S-Smart Software Solutions GmbH
Dipl.-Inf. Markus Schaber | Product Development Core Technology
Memminger Str. 151 | 87439 Kempten | Germany
Tel. +49-831-54031-979 | Fax +49-831-54031-50

E-Mail: m.schaber@codesys.com | Web: http://www.codesys.com | CODESYS store: http://store.codesys.com
CODESYS forum: http://forum.codesys.com

Managing Directors: Dipl.Inf. Dieter Hess, Dipl.Inf. Manfred Werner | Trade register: Kempten HRB 6186 | Tax ID No.: DE 167014915

This e-mail may contain confidential and/or privileged information. If you are not the intended recipient (or have received
this e-mail in error) please notify the sender immediately and destroy this e-mail. Any unauthorised copying, disclosure
or distribution of the material in this e-mail is strictly forbidden.
Received on 2017-04-26 08:26:16 CEST

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