diff options
author | Bartosz Sosnowski <bartosz@janeasystems.com> | 2016-09-22 14:00:27 +0300 |
---|---|---|
committer | Bartosz Sosnowski <bartosz@janeasystems.com> | 2016-09-26 18:23:50 +0300 |
commit | d5bc52aa2e324e2cbb6da38a505ff956058b4e76 (patch) | |
tree | 67f5de0ecaea079b58a61d85bbc1181473ebfc97 | |
parent | 9e5a06ecf67442afa3e6f2cb62077e7de12de4e1 (diff) |
benchmark: use 'yes' instead of echo in a loop
This changes child-process-exec-stdout benchmark to use 'yes' instead
of echo in a while loop. This makes this benchmark consistent with
child-process-read which already uses `yes` and allows this benchmark
to be executed on Windows.
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
PR-URL: https://github.com/nodejs/node/pull/8721
-rw-r--r-- | benchmark/child_process/child-process-exec-stdout.js | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/benchmark/child_process/child-process-exec-stdout.js b/benchmark/child_process/child-process-exec-stdout.js index 6194f66b55d..7b93aacff85 100644 --- a/benchmark/child_process/child-process-exec-stdout.js +++ b/benchmark/child_process/child-process-exec-stdout.js @@ -20,8 +20,7 @@ function main(conf) { const msg = `"${'.'.repeat(len)}"`; msg.match(/./); const options = {'stdio': ['ignore', 'pipe', 'ignore']}; - // NOTE: Command below assumes bash shell. - const child = exec(`while\n echo ${msg}\ndo :; done\n`, options); + const child = exec(`yes ${msg}`, options); var bytes = 0; child.stdout.on('data', function(msg) { |