Merge pull request #58 from atom/windows-bootstrap

Fix invocation of npm on non-Cygwin
This commit is contained in:
Cheng Zhao 2013-08-20 10:14:06 -07:00
commit 9105914b9e

View file

@ -57,8 +57,13 @@ def bootstrap_brightray(url):
def update_apm():
## NB: Without this, subprocess incorrectly searches for npm.exe
npm_cmd = 'npm'
if sys.platform == 'win32':
npm_cmd += '.cmd'
with scoped_cwd(os.path.join('vendor', 'apm')):
subprocess.check_call(['npm', 'install', '.'])
subprocess.check_call([npm_cmd, 'install', '.'])
def update_node_modules():