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

Re: Modifying transaction properties (svnadmin setrevprop)

From: Ryan J Ollos <rjollos_at_apache.org>
Date: Sat, 23 Apr 2016 17:43:44 -0700

On Sat, Apr 23, 2016 at 3:19 PM, Daniel Shahaf <danielsh_at_apache.org> wrote:

> Daniel Shahaf wrote on Sat, Apr 23, 2016 at 21:48:38 +0000:
> > Ryan J Ollos wrote on Sat, Apr 23, 2016 at 14:23:37 -0700:
> > > For a long time I've known the rule "don't modify a transaction in a
> > > pre-commit hook", documented in (1) and recently repeated in (2).
> > >
> > > I was therefore surprised to read about the "svnadmin setrevprop"
> command
> > > (3) in 1.9, and to see an example of modifying transaction properties
> in
> > > the hook-scripts: "persist-ephemeral-txnprops.py" (4).
> > >
> > > Are there some conditions under which it's okay to modify the
> transaction,
> > > or are there only specific use-cases when this should be done?
>
> I think I can give a simpler answer: anything you can do using only
> svnadmin/svnlook is okay. The rule you quote refers to people who
> directly use the svn_fs_* APIs, or DAV clients.
>
> Cheers,
>
> Daniel
>

Thanks, that makes sense. In the past, I have applied the rule in an
overly-restrictive way when considering pre-/post-commit hook use cases.

- Ryan
Received on 2016-04-24 02:43:47 CEST

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

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.