Re: svn commit: r1241050 - /subversion/trunk/subversion/mod_dav_svn/mod_dav_svn.c

Daniel Shahaf wrote on Mon, Feb 06, 2012 at 18:20:28 +0200:
> Stefan Sperling wrote on Mon, Feb 06, 2012 at 17:12:32 +0100:
> > On Mon, Feb 06, 2012 at 05:59:04PM +0200, Daniel Shahaf wrote:
> > > This still strips whitespace around ='s in the value:
> > > SVNHooksEnv "name = x = y"
> > > will result in
> > > setenv("name", "x=y", 1)
> > > whereas I believe it should result in
> > > setenv("name", "x = y", 1)
> > > (and, to be honest, I'd be happy with
> > > setenv("name ", " x = y", 1)
> > > as well).
> > >
> > > WDYT? How should it behave?
> >
> > I agree.
> > would telling svn_cstring_split() to no strip whitespace suffice?
>
> I assume that should result in the third setenv() case above, so +1.

Ping? trunk_at_HEAD still strips whitespace around equal signs in the value.
Received on 2012-03-04 11:57:14 CET