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

RE: Eols are not preserved in DAV log reponse.

From: Alexander Kitaev <alex_at_tmate.org>
Date: 2006-02-28 20:36:21 CET

Thanks, Stefan!

> That's not quite true: the svn:log property is (like all other svn:
> properties) a Subversion property. And all Subversion
> properties *must* be in UTF8 and LF lineendings (as defined
> in the docs). A client must make sure that this is done right.
> If it were another property, *then* you're allowed to store
> whatever you want in whichever format you like, even with
> CRLF lineendings or binary data. But not with svn: properties.
Thanks for clarification, I understand it well now, and see that though
mod_dav_svn behaviour is not consistent with "svnserve" one, it is just an
output caused by incorrect (unexpected) input data and shouldn't be
considered as a bug.

Alexander Kitaev,
TMate Software,
http://tmate.org/
http://jetbrains.com/tmate/

> -----Original Message-----
> From: Stefan Kung [mailto:tortoisesvn@gmail.com]
> Sent: Tuesday, February 28, 2006 20:17
> To: dev@subversion.tigris.org
> Subject: Re: Eols are not preserved in DAV log reponse.
>
> Alexander Kitaev wrote:
> >> There shouldn't be any \r's in the svn:log value in the
> first place;
> >> they're supposed to be filtered out by the client. So the first
> >> question is, where are they coming from?
> > I'm setting those messages programatically through JavaSVN
> (actually
> > I'm doing commit and commit message includes \r's). I
> understand your
> > point and agree that client should convert commit message not to
> > contain \r's before making a commit.
> >
> > On the other side, as far as I understand, svn:log is not
> special kind
> > of property (except that it is set automatically on commit), so it
> > should be possible to set it to any value in any encoding or even
> > binary data as well as any other revision property (probably doing
> > "svn propset -rX --revprop svn:log -F fileWithWindowsEols"
> will do the
> > same). Svnserve behaves consistently here, preserving
> svn:log property
> > value in log message, but mod_dav_svn does not (the log message it
> > reports is not same as the value stored in revprop file of
> the corresponding revision).
>
> That's not quite true: the svn:log property is (like all other svn:
> properties) a Subversion property. And all Subversion
> properties *must* be in UTF8 and LF lineendings (as defined
> in the docs). A client must make sure that this is done right.
> If it were another property, *then* you're allowed to store
> whatever you want in whichever format you like, even with
> CRLF lineendings or binary data. But not with svn: properties.
>
> Stefan
>
> --
> ___
> oo // \\ "De Chelonian Mobile"
> (_,\/ \_/ \ TortoiseSVN
> \ \_/_\_/> The coolest Interface to (Sub)Version Control
> /_/ \_\ http://tortoisesvn.tigris.org
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
> For additional commands, e-mail: dev-help@subversion.tigris.org
>
>

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Tue Feb 28 20:37:31 2006

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

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.