Commit 0a0f5c64 by Michael Loughry Committed by Michael Mifsud

Use true-case-path to fix re-entrancy issue on Windows (#2149)

* Use true-case-path

* Fix failing tests

* Delete package-lock.json

* .gitignore package-lock.json
parent 6b7b6790
...@@ -11,3 +11,4 @@ test/lcov.info ...@@ -11,3 +11,4 @@ test/lcov.info
test/fixtures/watching-css-out-01 test/fixtures/watching-css-out-01
test/fixtures/watching-css-out-02 test/fixtures/watching-css-out-02
coverage coverage
package-lock.json
\ No newline at end of file
...@@ -7,7 +7,8 @@ var eol = require('os').EOL, ...@@ -7,7 +7,8 @@ var eol = require('os').EOL,
pkg = require('../package.json'), pkg = require('../package.json'),
mkdir = require('mkdirp'), mkdir = require('mkdirp'),
path = require('path'), path = require('path'),
defaultBinaryPath = path.join(__dirname, '..', 'vendor'); defaultBinaryPath = path.join(__dirname, '..', 'vendor'),
trueCasePathSync = require('true-case-path');
/** /**
* Get the human readable name of the Platform that is running * Get the human readable name of the Platform that is running
...@@ -272,7 +273,7 @@ function getBinaryPath() { ...@@ -272,7 +273,7 @@ function getBinaryPath() {
binaryPath = path.join(defaultBinaryPath, getBinaryName().replace(/_(?=binding\.node)/, '/')); binaryPath = path.join(defaultBinaryPath, getBinaryName().replace(/_(?=binding\.node)/, '/'));
} }
return binaryPath; return trueCasePathSync(binaryPath) || binaryPath;
} }
/** /**
......
...@@ -70,7 +70,8 @@ ...@@ -70,7 +70,8 @@
"npmlog": "^4.0.0", "npmlog": "^4.0.0",
"request": "^2.79.0", "request": "^2.79.0",
"sass-graph": "^2.2.4", "sass-graph": "^2.2.4",
"stdout-stream": "^1.4.0" "stdout-stream": "^1.4.0",
"true-case-path": "^1.0.2"
}, },
"devDependencies": { "devDependencies": {
"coveralls": "^2.11.8", "coveralls": "^2.11.8",
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment