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