Skip to content

Commit

Permalink
Standardise on cross-spawn for spawning child processes
Browse files Browse the repository at this point in the history
There a known issues with `spawn` on Windows i.e. nodejs/node-v0.x-archive#2318.
As a work around we sometimes use
[cross-spawn](https://www.npmjs.com/package/cross-spawn). We will
move all usage of `spawn` to `cross-spawn`.
Closes sass#1015.
  • Loading branch information
xzyfer committed Aug 31, 2015
1 parent 7d416cd commit 33b3a66
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -50,6 +50,7 @@
"dependencies": {
"async-foreach": "^0.1.3",
"chalk": "^1.1.1",
"cross-spawn": "^2.0.0",
"gaze": "^0.5.1",
"get-stdin": "^4.0.1",
"glob": "^5.0.14",
Expand All @@ -63,7 +64,6 @@
},
"devDependencies": {
"coveralls": "^2.11.4",
"cross-spawn": "^2.0.0",
"istanbul": "^0.3.18",
"jshint": "^2.8.0",
"mocha": "^2.2.5",
Expand Down
2 changes: 1 addition & 1 deletion scripts/build.js
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ var eol = require('os').EOL,
fs = require('fs'),
mkdir = require('mkdirp'),
path = require('path'),
spawn = require('child_process').spawn;
spawn = require('cross-spawn');

require('../lib/extensions');

Expand Down

0 comments on commit 33b3a66

Please sign in to comment.