Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ini files #289

Open
vbmithr opened this issue Dec 28, 2013 · 3 comments
Open

ini files #289

vbmithr opened this issue Dec 28, 2013 · 3 comments

Comments

@vbmithr
Copy link

vbmithr commented Dec 28, 2013

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

@incubaid-ci
Copy link

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
#289 (comment).

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

Romain.

@vbmithr
Copy link
Author

vbmithr commented Dec 30, 2013

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
Copy link
Contributor

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

(so no \r allowed)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants