Skip to content

Commit d9533dd

Browse files
authoredMay 14, 2019
fix: disable parallel on WSL due bugs (#90)
1 parent 449e0fc commit d9533dd

File tree

3 files changed

+21
-11
lines changed

3 files changed

+21
-11
lines changed
 

‎package-lock.json

+12-7
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

‎package.json

+1
Original file line numberDiff line numberDiff line change
@@ -37,6 +37,7 @@
3737
"dependencies": {
3838
"cacache": "^11.3.2",
3939
"find-cache-dir": "^2.0.0",
40+
"is-wsl": "^1.1.0",
4041
"schema-utils": "^1.0.0",
4142
"serialize-javascript": "^1.7.0",
4243
"source-map": "^0.6.1",

‎src/TaskRunner.js

+8-4
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@ import cacache from 'cacache';
44
import findCacheDir from 'find-cache-dir';
55
import workerFarm from 'worker-farm';
66
import serialize from 'serialize-javascript';
7+
import isWsl from 'is-wsl';
78

89
import minify from './minify';
910

@@ -19,10 +20,13 @@ export default class TaskRunner {
1920
// In some cases cpus() returns undefined
2021
// https://github.com/nodejs/node/issues/19022
2122
const cpus = os.cpus() || { length: 1 };
22-
this.maxConcurrentWorkers =
23-
parallel === true
24-
? cpus.length - 1
25-
: Math.min(Number(parallel) || 0, cpus.length - 1);
23+
// WSL sometimes freezes, error seems to be on the WSL side
24+
// https://github.com/webpack-contrib/terser-webpack-plugin/issues/21
25+
this.maxConcurrentWorkers = isWsl
26+
? 1
27+
: parallel === true
28+
? cpus.length - 1
29+
: Math.min(Number(parallel) || 0, cpus.length - 1);
2630
}
2731

2832
run(tasks, callback) {

0 commit comments

Comments
 (0)
Please sign in to comment.