Penthouse adds extra line breaks (only on Windows(?))

[pocketjoso]

Penthouse adds extra line breaks (only on Windows(?))

[pocketjoso]

Just some further info, although I'm still lost as to how to fix this problem:

On Windows

When running the penthouse.js standalone, if I just let the output be printed in the console, then it works (no extra line breaks). (same in Cygwin and Window's CMD).
./phantomjs js/penthouse.js http://192.168.1.249/criticalpathcssgenerator/ test.css

If I redirect the output to a new file however, then additional line breaks appear (as shown in last post).
./phantomjs js/penthouse.js http://192.168.1.249/criticalpathcssgenerator/ test.css > extra-linebreaks.css
This is also what happens when running via f.e. Grunt.

[pocketjoso]

I'll come back to this later, but for now I'm sure the problem has to do with newline encoding; something in the penthouse.js scripts (most likely fs) converting Windows style newlines (\r\n) to Nix style (\n), and then whenever the content gets rewritten to a Windows file the conversion goes wrong (becomes \r\r\n)?.

http://stackoverflow.com/questions/6891252/c-newline-character-under-windows-command-line-redirection

I don't know much about this topic... Perhaps the answer to this problem is obvious to someone, - @kopseng?

[pocketjoso]

Current fix for this now is simply replacing any existing \r\n initially with \n - this solves the problem. There might be more elegant solutions out there though..

Fixed in: a1dc313