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

Re: svn commit: r1393521 - in /subversion/trunk/subversion: include/svn_props.h tests/cmdline/svnlook_tests.py

From: C. Michael Pilato <cmpilato_at_collab.net>
Date: Thu, 4 Oct 2012 08:54:56 -0400

On 10/03/2012 03:39 PM, Mark Phippard wrote:
> On Wed, Oct 3, 2012 at 10:51 AM, <cmpilato_at_apache.org> wrote:
>> Author: cmpilato
>> Date: Wed Oct 3 14:51:36 2012
>> New Revision: 1393521
>>
>> URL: http://svn.apache.org/viewvc?rev=1393521&view=rev
>> Log:
>> Use "svn:txn:" as the namespace for ephemeral txnprops, not
>> "svn:txn-". I was operating under this memory of Subversion having
>> trouble with colons in property names, but apparently that memory was
>> either obsolete or altogether bogus.
>
> I was looking at the open Serf issues for 1.8 and saw this:
>
> http://subversion.tigris.org/issues/show_bug.cgi?id=1971
>
> Is that what you were remembering?
>

Yes, I believe it was. I remembered that PROPFINDs and PROPPATCHs use the
property name as an XML tag, so we already require that property names be
essentially well-formed XML "Names". The book mentions this, too:

{{{
There are some restrictions on the names you can use for properties. A
property name must start with a letter, a colon (:), or an underscore (_);
after that, you can also use digits, hyphens (-), and periods (.)
}}}

But a picky XML parser would see "svn:txn:foo" or "svn:config:foo" and want
to know about the XML namespaces "svn:txn" and "svn:config", which we don't
bother to describe. There's a fairly straightforward workaround, I think,
where we simply generate xmlns definitions on the fly for all namespaces we
need for that particular request/response. I think this is essentially what
was being described here:

http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=480154

-- 
C. Michael Pilato <cmpilato_at_collab.net>
CollabNet   <>   www.collab.net   <>   Enterprise Cloud Development
Received on 2012-10-04 14:55:36 CEST

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