search

Incubaid / arakoon

A consistent distributed key-value store
http://arakoon.org
Apache License 2.0
84 stars 17 forks source link

ini files #289

Open vbmithr opened 10 years ago

vbmithr commented 10 years ago

Why not use another nicer config file format ? INI files are kind of old-school. Suggestions: JSON, s-expressions.

incubaid-ci commented 10 years ago

On 12/29/2013 04:10 PM, Vincent Bernardoff wrote:

And I don't know if you noticed how horrible is the ini parser you're using :) Try putting a newline in the beginning of the file.

— Reply to this email directly or view it on GitHub https://github.com/Incubaid/arakoon/issues/289#issuecomment-31318586.

Can you send me an example of this ? (are you on OSX or Windows?)

Romain.

vbmithr commented 10 years ago

On 30/12/2013 17:41, incubaid-ci wrote:

Can you send me an example of this ? (are you on OSX or Windows?)

Romain.

I'm not sure anymore that the about the newlines (need to have a look at the parser again). But for example, the empty file is not a valid ini file according to the grammar of the parser.

toolslive commented 10 years ago

it's the lexer: let newline = '\n' +

(so no \r allowed)