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

Re: svn commit: r1375675 - in /subversion/trunk/subversion: include/svn_repos.h libsvn_repos/fs-wrap.c libsvn_repos/hooks.c libsvn_repos/repos.c libsvn_repos/repos.h tests/cmdline/commit_tests.py tests/cmdline/svntest/actions.py

From: Philip Martin <philip.martin_at_wandisco.com>
Date: Tue, 21 Aug 2012 19:45:13 +0100

Blair Zajac <blair_at_orcaware.com> writes:

> On 08/21/2012 11:09 AM, C. Michael Pilato wrote:
>>
>> I actually considered using "post-create-txn" and renaming "start-commit" to
>> "pre-create-txn" (with code to run "start-commit" iff not "pre-create-txn"
>> hook exists, for compat purposes).
>
> +1. I always have to remember which comes first, start-commit or
> pre-commit, so this renaming helps.

Suppose both pre-create-txn and start-commit exist. Is it an error?
If not which one is run?

We have already bumped the FSFS format in 1.8 but we have not yet bumped
the repos format. Perhaps we could bump and have an upgrade that
renames the hook?

-- 
Certified & Supported Apache Subversion Downloads:
http://www.wandisco.com/subversion/download
Received on 2012-08-21 20:46:01 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.