Skip to content
This repository was archived by the owner on Jun 30, 2022. It is now read-only.

Decrease command execution overhead #47

Closed
joao-paulo-parity wants to merge 6 commits into from
Closed

Decrease command execution overhead #47

Changes from 1 commit
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
3373401
decrease command execution overhead
joao-paulo-parity Jul 13, 2021
fdb04e7
inherit -> ignore
joao-paulo-parity Jul 13, 2021
a860dd5
execute the command through the shell
joao-paulo-parity Jul 13, 2021
52d5518
reintroduce shelljs
joao-paulo-parity Jul 13, 2021
a815418
make runTask synchronous again
joao-paulo-parity Jul 13, 2021
975c807
reintroduce shell execution
joao-paulo-parity Jul 13, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
inherit -> ignore
  • Loading branch information
@joao-paulo-parity
joao-paulo-parity committed Jul 13, 2021
commit fdb04e799cc7f57e9d6411456bbcb98acb016929
2 changes: 1 addition & 1 deletion bench.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@ function BenchContext(app, config) {
try {
if (shouldLogOutput) {
console.log(`<=== Start command output (cwd: ${process.cwd()})`)
cp.execFileSync("/bin/dash", ["-c", `${cmd} | tee ${runnerOutput}`], { stdio: "inherit" })
cp.execFileSync("/bin/dash", ["-c", `${cmd} | tee ${runnerOutput}`], { stdio: "ignore" })
stdout = fs.readFileSync(runnerOutput).toString()
} else {
stdout = cp.execSync(cmd, { stdio: "pipe", shell: true }).toString()
Expand Down