Inserting CR in addition to LF in util.inspect() is not right on UNIX OSes

[alanmimms]

Reproduce with this:

console.log('foo');
console.log(require('util').inspect());

Run this with its output redirected to a file then open that file with, e.g., the Emacs editor. Note the first line has only a LF on the end of it as is traditional for thousands of years in Linux/UNIX OSes. Note that lines containing util.inspect() output end in CRLF, which displays in Emacs as an ugly and wrong ^M on the end of that line.

This behavior of mixing the line ending styles on UNIX OSes will cause all sorts of problems with editors, source code management systems, various text processing utilities, etc. It seems clear that util.inspect() should not unilaterally introduce separate line ending semantics from the rest of the system, or ones that are antithetical to the most common OS node.js runs on.

• Version: v6.4.0
• Platform: Linux alan-laptop 4.4.0-34-generic [streams] _flush not being called #53-Ubuntu SMP Wed Jul 27 16:06:39 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux
• Subsystem: util

[addaleax]

That change is being reverted in #8143, fwiw.

[jasnell]

Closing as a duplicate. @alanmimms the fix for this should most likely go out in the next patch release for v6

[alexindigo]

I noticed similar behavior without inspect, when console.log outputs multiline objects:

$ nvm exec 6.4 sh -c "node test6.js | wc -c"
Running node v6.4.0 (npm v3.10.3)
     107
$ nvm exec 6.3 sh -c "node test6.js | wc -c"
Running node v6.3.1 (npm v3.10.3)
     102

test6.js:

var obj = {custom: 42, acustom: 'boom', 'aaa': 'bbbb', 'cccc': 'ddddd', 'eeeee': 'ggggg', 'xxxxx': 'yyyyy'};

console.log(obj);

Thank you.