[v0.6] Handle stdout suppressing correctly

mmalecki · mmalecki · commit 006476ffb0e1 · 2011-10-13T19:16:03.000+02:00
In node v0.5.x `process.stdout` is a getter, so assigning to it makes no difference (see nodejs/node-v0.x-archive#1878 - thanks @bnoordhuis).
diff --git a/bin/vows b/bin/vows
@@ -168,8 +168,11 @@ while (arg = argv.shift()) {
 }
 
 if (options.supressStdout) {
-  _reporter.setStream && _reporter.setStream(process.stdout);
-  process.stdout = fs.createWriteStream('/dev/null');
+    _reporter.setStream && _reporter.setStream(process.stdout);
+    var devNullStream = fs.createWriteStream('/dev/null');
+    process.__defineGetter__('stdout', function () {
+        return devNullStream;
+    });
 }
 
 if (options.watch) {

Original file line number	Diff line number	Diff line change
`@@ -168,8 +168,11 @@ while (arg = argv.shift()) {`
`168`	`168`	`}`
`169`	`169`
`170`	`170`	`if (options.supressStdout) {`
`171`		`- _reporter.setStream && _reporter.setStream(process.stdout);`
`172`		`- process.stdout = fs.createWriteStream('/dev/null');`
	`171`	`+ _reporter.setStream && _reporter.setStream(process.stdout);`
	`172`	`+ var devNullStream = fs.createWriteStream('/dev/null');`
	`173`	`+ process.__defineGetter__('stdout', function () {`
	`174`	`+ return devNullStream;`
	`175`	`+ });`
`173`	`176`	`}`
`174`	`177`
`175`	`178`	`if (options.watch) {`