there is no noperfctr in node 12. also nosign is redundant

This commit is contained in:
Igor Klopov 2019-05-07 00:56:56 +03:00
parent 8b36b70e00
commit b5a1eec53d

View File

@ -49,7 +49,9 @@ async function applyPatches (nodeVersion) {
async function compileOnWindows (nodeVersion, targetArch) {
const args = [];
args.push('/c', 'vcbuild.bat', targetArch, 'nosign', 'noetw', 'noperfctr');
args.push('/c', 'vcbuild.bat', targetArch, 'noetw');
const major = nodeVersion.match(/^v?(\d+)/)[1] | 0;
if (major <= 10) args.push('nosign', 'noperfctr');
const promise = spawn('cmd', args, { cwd: nodePath });
progress(promise, thresholds('vcbuild', nodeVersion));
await promise;