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

Re: svn commit: r1051763 - in /subversion/trunk/subversion: include/private/svn_repos_private.h libsvn_repos/commit.c mod_dav_svn/version.c

From: Blair Zajac <blair_at_orcaware.com>
Date: Wed, 22 Dec 2010 10:57:24 -0800

On 12/22/10 5:47 AM, Daniel Shahaf wrote:
> blair_at_apache.org wrote on Wed, Dec 22, 2010 at 05:46:45 -0000:
>> Author: blair
>> Date: Wed Dec 22 05:46:45 2010
>> New Revision: 1051763
>>
>> URL: http://svn.apache.org/viewvc?rev=1051763&view=rev
>> Log:
>> Add a private function that takes the error returned from
>> svn_repos_fs_commit_txn() and builds a error message string containing
>> either or both of the svn_fs_commit_txn() error and the
>> SVN_ERR_REPOS_POST_COMMIT_HOOK_FAILED wrapped post-commit error. The
>> function skips over tracing errors.

>> + "post-commit hook had error '%s'.",
>> + err->message ? err->message
>> + : "(no error message)",
>> + hook_err2->message ? hook_err2->message
>> + : "(no error message)");
>> + }
>> + }
>> + else
>> + {
>> + if (err->message)
>> + return apr_pstrdup(pool, err->message);
>
> In this case, and in the apr_pstrdup() above, might it be useful to say
> explicitly whether the given error is from the post-commit hook or from
> some FS post-commit fiddling? You've skipped over the
> SVN_ERR_REPOS_POST_COMMIT_HOOK_FAILED link, so the "post-commit hook
> failed" message that used to be there would be gone.

Turns out we don't need to say that because the wrapped error message is already
pretty self-describing.

Further fixed in r1052029.

Blair
Received on 2010-12-22 19:58:06 CET

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.