Parser error with toml engine and CRLF.

[fimars]

1. gray-matter with CRLF, work well.
2. node-toml with CRLF, work well too.

BUT:

If you run the toml example of README with CRLF file, will get a parse error.

Minimal example repo here

[fimars]

I see the point, here of gray-matter output code like:

\r\n RawOfMatter \r

And node-toml doesn't trim the input thing for now. Maybe add trim here for engines or else, Any suggestions with this?

@robertmassaioli

[robertmassaioli]

I can verify this just by modifying line 15 of examples/toml.js to be:

].join('\r\n');

The error message suggests that the problem is not in the gray-matter library though. It suggests that the error is in the toml package.

In fact, this error message:

throw peg$buildException(null, peg$maxFailExpected, peg$maxFailPos);

Looks like it comes from the PEG.js library, which I assume the toml library is using internally.

Please go raise an issue here to get this problem fixed: https://github.com/BinaryMuse/toml-node

[fimars]

@robertmassaioli thanks for your reply, But don't you think it's a little strange to output \r as EOF

[zachleat]

I think this should be reopened. It causes errors on TOML parsing on Windows machines.

Input file: https://github.com/11ty/eleventy/blob/master/test/stubs/custom-frontmatter/template-toml.njk#L1
JavaScript test case: https://github.com/11ty/eleventy/blob/master/test/TemplateTest.js#L1732

Build failure on Windows machines from this: https://travis-ci.org/11ty/eleventy/jobs/550051432 (Works on Linux and Mac https://travis-ci.org/11ty/eleventy/builds/550051427)

[redblue9771]

I also think that this issue should be reopened to solve the bug that appears on windows