Re: [PATCH] Fix issue #2068

On Thu, Mar 12, 2009 at 3:03 AM, Bert Huijben <rhuijben_at_sharpsvn.net> wrote:
>> -----Original Message-----
>> From: Senthil Kumaran S [mailto:senthil_at_collab.net]
>> Sent: donderdag 12 maart 2009 10:09
>> To: dev_at_subversion.tigris.org
>> Subject: [PATCH] Fix issue #2068
>>
>> Hi,
>>
>> The attached patch gives a solution to fix issue #2068, though this is not
>> backward compatible with earlier versions of subversion.
>>
>> [[[
>> Fix issue #2068.
>>
>> NOTE: This is not backward compatible.
>>
>> * subversion/libsvn_subr/config_file.c
>>   (svn_config_ensure): Change the delimiter between properties to '|'
>>    from ';' in default config file.
>
> Maybe we should use a double semicolon as escape sequence for passing a
> semicolon. I don't like the idea of breaking all old config files, registry
> settings, etc..
>
> This fix only removes the semicolon problem, but it breaks putting |
> characters in properties.

That's a great suggestion, since in the previous format, any double
semicolon was completely redundant.

--dave

-- 
glasser_at_davidglasser.net | langtonlabs.org | flickr.com/photos/glasser/
------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=1313694