Modifying transaction properties (svnadmin setrevprop)
danielsh at apache.org
Mon Apr 25 03:19:16 CDT 2016
Ryan J Ollos wrote on Sat, Apr 23, 2016 at 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.
Then perhaps the book's warning box  should be clarified with the
information in this thread . CCing svnbook-dev at .
(Ryan: svnbook is maintained outside ASF by the same people that make up
the svn PMC)
More information about the svnbook-dev