On 02/02/2012 03:29 PM, s...@apache.org wrote:
Author: stsp
Date: Thu Feb  2 23:29:11 2012
New Revision: 1239926

URL: http://svn.apache.org/viewvc?rev=1239926&view=rev
Log:
Add a public libsvn_repos API to set the environment of hook scripts.

Modified:
     subversion/trunk/subversion/include/svn_repos.h
     subversion/trunk/subversion/libsvn_repos/hooks.c
     subversion/trunk/subversion/libsvn_repos/repos.c
     subversion/trunk/subversion/libsvn_repos/repos.h

Modified: subversion/trunk/subversion/include/svn_repos.h
URL: 
http://svn.apache.org/viewvc/subversion/trunk/subversion/include/svn_repos.h?rev=1239926&r1=1239925&r2=1239926&view=diff
==============================================================================
--- subversion/trunk/subversion/include/svn_repos.h (original)
+++ subversion/trunk/subversion/include/svn_repos.h Thu Feb  2 23:29:11 2012
@@ -764,6 +764,13 @@ const char *
  svn_repos_post_unlock_hook(svn_repos_t *repos,
                             apr_pool_t *pool);

+/** Set the environment that @ repos's hooks will inherit.
+ * If this function is not called, hooks will run in an empty environment.
+ * @since New in 1.8. */

Is NULL OK? How is the environment formatted? It's not of the same form as execve(), which I was surprised by.

It doesn't look like it's multithreaded safe to set, but should it be?

Blair


Reply via email to