Stefan,

Thanks for checking that. Bad on my part, it had worked before(admin deleted 
the executable), although I'm not sure how-- but in checking my other installs 
of various versions-- 1.6.12, 1.6.17, all have the line without leading space 
so I guess I won't worry about it. Possibly, I had a very much older version on 
the box-- maybe a 1.5.x..

Thanks,
Will

-----Original Message-----
From: Stefan Sperling [mailto:s...@elego.de] 
Sent: Thursday, April 26, 2012 12:50
To: Will Fleming
Cc: users@subversion.apache.org
Subject: Re: annoyance: servers file format-- worked before 1.6.11

On Thu, Apr 26, 2012 at 12:31:29PM -0600, Will Fleming wrote:
> [etldev@odlbods1 ~]$ svn help
> svn: /home/etldev/.subversion/servers:153: Option expected
> [etldev@odlbods1 ~]$
> 
> I removed the leading spaces to the line setting and then it worked 
> ......
> # No neon-debug-mask, so neon debugging is disabled.
> # ssl-authority-files = /path/to/CAcert.pem;/path/to/CAcert2.pem
> #
> # Password / passphrase caching parameters:
> # store-passwords = no
> store-plaintext-passwords = no
> # store-ssl-client-cert-pp = no
> # store-ssl-client-cert-pp-plaintext = no ....
> 
> Prior to 1.6.11, there could be leading spaces on that line

Are you sure about that? I don't have a 1.6.10 release build around but I see 
the same behaviour with a Subversion 1.5 build. 

Note that the parser does this because a leading space is usually used in 
multi-line configuration statements.
For instance, given an entry such as:

global-ignores = *.o *.lo *.la *.al .libs *.so *.so.[0-9]* *.a *.pyc *.pyo
   *.rej *~ #*# .#* .*.swp .DS_Store

both lines will be merged into one by the config parser. The second line is 
part of the previous one because it starts with whitespace.

I don't think this is very intuitive myself. Alas, that's how it was designed.

Reply via email to