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

Re: svn commit: r1305641 - in /subversion/branches/inheritable-props/subversion: libsvn_ra_svn/client.c libsvn_ra_svn/protocol svnserve/serve.c

From: Daniel Shahaf <danielsh_at_elego.de>
Date: Tue, 27 Mar 2012 02:35:26 +0200

pburba_at_apache.org wrote on Mon, Mar 26, 2012 at 22:29:55 -0000:
> Author: pburba
> Date: Mon Mar 26 22:29:55 2012
> New Revision: 1305641
>
> URL: http://svn.apache.org/viewvc?rev=1305641&view=rev
> Log:
> On the inheritable-props branch: Implement getting inherited props over
> ra_svn.
>
> * subversion/libsvn_ra_svn/protocol
> (get-file,
> get-dir): Document the new protocol options.
>

The protocol document doesn't mention the new capability (which an
earlier commit added).

> +++ subversion/branches/inheritable-props/subversion/svnserve/serve.c Mon Mar 26 22:29:55 2012
> +static svn_error_t *
> +get_props(apr_hash_t **props,
> + apr_array_header_t **iprops,
> + svn_fs_root_t *root,
> + const char *path,
> + apr_pool_t *pool)
> {
> /* Get the properties. */
> - SVN_ERR(svn_fs_node_proplist(props, root, path, pool));
> + SVN_ERR(svn_fs_node_proplist2(props, iprops, root, path, pool, pool));
>

Don't you need an svn_repos_* wrapper here, where (eventually) authz
checks on the parents will be done?
Received on 2012-03-27 02:36:18 CEST

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