Permalink
Browse files

fixed test suit runner to use modern node api

  • Loading branch information...
1 parent 21121f3 commit 7faaf435c54efb6b1165ad36581954c5db8dcbd9 @rsms rsms committed Oct 16, 2010
Showing with 2 additions and 2 deletions.
  1. +2 −2 test/all.js
View
@@ -33,9 +33,9 @@ function runNext() {
process.stdout.write(data);
});
child.stderr.on('data', function (data) {
- if (/^execvp\(\)/.test(data.asciiSlice(0,data.length))) {
+ if (data.length >= 7 && data.toString('ascii', 0, 7) === 'execvp(') {
console.error('>>> fail '+fn+' -- execvp('+sys.inspect(nodebin)+
- ', '+sys.inspect(args)+') failed: '+data);
+ ', '+sys.inspect(args)+') failed: '+data.toString('utf8'));
} else {
stderrWrite(data);
}

2 comments on commit 7faaf43

Owner

liesen replied Oct 16, 2010

thank you :-)

Collaborator

rsms replied Oct 23, 2010

:—)

Please sign in to comment.