Re: svn commit: r35313 - in branches/http-protocol-v2: . subversion/mod_dav_svn subversion/mod_dav_svn/reports

Greg Stein wrote:
> On Sun, Jan 18, 2009 at 17:37, Ben Collins-Sussman <sussman_at_red-bean.com> wrote:
>> ...
>> +++ branches/http-protocol-v2/subversion/mod_dav_svn/repos.c Sun Jan 18 08:37:15 2009 (r35313)
>> ...
>> @@ -307,6 +307,36 @@ parse_vcc_uri(dav_resource_combined *com
>>
>>
>> static int
>> +parse_rootstub_uri(dav_resource_combined *comb,
>> + const char *path,
>> + const char *label,
>> + int use_checked_in)
>> +{
>> + /* There are no components after the root stub, i.e. just
>> + "!svn/me/".
>
> Is this true? That a trailing slash can/will exist? I thought this
> resource was specified as NO trailing slash.
>
>> ...
>
> Changes like this one can/should appear on trunk. There is no reason
> to hide this on the branch.

I dunno, dude. With 1.6 looming in the near future, changes like this that
bring no value to the trunk -- merely the risks associated with
intended-to-be-but-not-really dead code -- are IMO unwelcome there.

-- 
C. Michael Pilato <cmpilato_at_collab.net>
CollabNet   <>   www.collab.net   <>   Distributed Development On Demand
------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=1041547