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

Re: svn commit: r1239926 - in /subversion/trunk/subversion: include/svn_repos.h libsvn_repos/hooks.c libsvn_repos/repos.c libsvn_repos/repos.h

From: Stefan Sperling <stsp_at_apache.org>
Date: Fri, 3 Feb 2012 02:07:59 +0100

On Thu, Feb 02, 2012 at 03:41:48PM -0800, Blair Zajac wrote:
> It doesn't look like it's multithreaded safe to set, but should it be?

I am not sure.

Note that this isn't changing the environment of the currently running
process. It takes effect in the child process that runs a hook.

The intended use is to set the hook environment once, after opening
the repository with svn_repos_open().

I've just committed some code that uses this from mod_dav_svn
in r1239966. Maybe this example will make the intented use clear, and
we can document behaviour WRT thread-safety or make the code thread-safe?
Received on 2012-02-03 02:08:42 CET

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