Skip to content

Commit a3b3f19

Browse files
authored
Merge pull request #1302 from nakrovati/update-rebuild-single.js
refactor: rebuild-single.js
2 parents 8f1502c + d5975ed commit a3b3f19

File tree

1 file changed

+28
-38
lines changed

1 file changed

+28
-38
lines changed

rebuild-single.js

Lines changed: 28 additions & 38 deletions
Original file line numberDiff line numberDiff line change
@@ -1,18 +1,15 @@
1-
var _ = require('lodash');
2-
var exec = require('child_process').execSync;
3-
var fs = require('fs');
4-
var path = require('path');
5-
var yargs = require('yargs');
6-
const rimraf = require('rimraf');
1+
const { execSync } = require("child_process");
72

8-
let args = process.argv.length <= 2 ? [] : process.argv.slice(2, process.argv.length);
3+
const args = process.argv.slice(2);
4+
const argsString = args.join(" ");
95

106
// Use npm ci or npm install ?
11-
let ci = args.includes("--ci");
7+
const ci = args.includes("--ci");
8+
129
// Copy package-lock back for docker build or build locally?
13-
let docker = args.includes("--docker");
10+
const docker = args.includes("--docker");
1411

15-
let frameworks = args.filter(a => !a.startsWith("--"));
12+
const frameworks = args.filter((a) => !a.startsWith("--"));
1613

1714
console.log("args", args, "ci", ci, "docker", docker, "frameworks", frameworks);
1815

@@ -30,33 +27,26 @@ Pass list of frameworks
3027
*/
3128

3229
try {
33-
if (frameworks.length == 0) {
34-
console.log("ERROR: Missing arguments. Command: docker-rebuild keyed/framework1 non-keyed/framework2 ...");
35-
process.exit(1);
36-
}
37-
38-
if (docker) {
39-
let build_cmd = `docker exec -it js-framework-benchmark cp /src/rebuild-build-single.js /build/ && docker exec -it js-framework-benchmark node rebuild-build-single.js ${args.join(" ")}`;
40-
console.log(build_cmd);
41-
exec(build_cmd,
42-
{
43-
stdio: 'inherit'
44-
});
45-
} else {
46-
let build_cmd = `node rebuild-build-single.js ${args.join(" ")}`;
47-
console.log(build_cmd);
48-
exec(build_cmd,
49-
{
50-
stdio: 'inherit'
51-
});
52-
}
53-
54-
let check_cmd = `node rebuild-check-single.js ${args.join(" ")}`;
55-
console.log(check_cmd);
56-
exec(check_cmd,
57-
{
58-
stdio: 'inherit'
59-
});
30+
if (frameworks.length == 0) {
31+
console.log(
32+
"ERROR: Missing arguments. Command: docker-rebuild keyed/framework1 non-keyed/framework2 ..."
33+
);
34+
process.exit(1);
35+
}
36+
37+
const buildCmd = docker
38+
? `docker exec -it js-framework-benchmark cp /src/rebuild-build-single.js /build/ && docker exec -it js-framework-benchmark node rebuild-build-single.js ${argsString}`
39+
: `node rebuild-build-single.js ${argsString}`;
40+
console.log(buildCmd);
41+
execSync(buildCmd, {
42+
stdio: "inherit",
43+
});
44+
45+
const checkCmd = `node rebuild-check-single.js ${argsString}`;
46+
console.log(checkCmd);
47+
execSync(checkCmd, {
48+
stdio: "inherit",
49+
});
6050
} catch (e) {
61-
console.log(`ERROR: Rebuilding ${args.join(" ")} was not successful`);
51+
console.log(`ERROR: Rebuilding ${argsString} was not successful`);
6252
}

0 commit comments

Comments
 (0)