diff options
| author | Ben Noordhuis <info@bnoordhuis.nl> | 2016-07-08 11:03:55 +0200 |
|---|---|---|
| committer | Ben Noordhuis <info@bnoordhuis.nl> | 2016-07-11 23:15:34 +0200 |
| commit | cfe76f2d6bd77d3c0a73335c9643bcd6e8dd24fc (patch) | |
| tree | 06862c668fca20124f90fb54f45c2d751351609f /test | |
| parent | e8a7003e940928e2d1684c12881dd2a39af2d7cc (diff) | |
| download | android-node-v8-cfe76f2d6bd77d3c0a73335c9643bcd6e8dd24fc.tar.gz android-node-v8-cfe76f2d6bd77d3c0a73335c9643bcd6e8dd24fc.tar.bz2 android-node-v8-cfe76f2d6bd77d3c0a73335c9643bcd6e8dd24fc.zip | |
src: disable stdio buffering
Disable stdio buffering, it interacts poorly with printf() calls from
elsewhere in the program (e.g., any logging from V8.) Unbreaks among
other things the `--trace_debug_json` switch.
Undoes commit 0966ab99 ("src: force line buffering for stderr"), which
in retrospect is not a proper fix. Turning on line buffering fixed a
flaky test on SmartOS but the test wasn't failing on other platforms,
where stderr wasn't line-buffered either. Mark the test flaky again,
it failed once in a run of 333 tries on the smartos-64 buildbot.
Disabling buffering should be safe even when mixed with non-blocking
stdio I/O because libuv goes to great lengths to reopen the tty file
descriptors and falls back to blocking I/O when that fails.
PR-URL: https://github.com/nodejs/node/pull/7610
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Diffstat (limited to 'test')
| -rw-r--r-- | test/parallel/parallel.status | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/test/parallel/parallel.status b/test/parallel/parallel.status index e3a8e345f7..fc71d41657 100644 --- a/test/parallel/parallel.status +++ b/test/parallel/parallel.status @@ -22,6 +22,7 @@ test-tls-connect-address-family : PASS,FLAKY [$system==macos] [$system==solaris] # Also applies to SmartOS +test-debug-signal-cluster : PASS,FLAKY [$system==freebsd] |