Hide npm output by default (#431)
Fixes: https://github.com/emscripten-core/emscripten/issues/10291
diff --git a/.flake8 b/.flake8
index d007a0d..31829b5 100644
--- a/.flake8
+++ b/.flake8
@@ -9,6 +9,7 @@
./gnu
./upstream
./fastcomp
+ ./fastcomp-clang
./releases
./clang
./emscripten
diff --git a/emsdk.py b/emsdk.py
index d382d31..d460e3d 100755
--- a/emsdk.py
+++ b/emsdk.py
@@ -1276,7 +1276,17 @@
npm = os.path.join(node_path, 'npm' + ('.cmd' if WINDOWS else ''))
env = os.environ.copy()
env["PATH"] = node_path + os.pathsep + env["PATH"]
- subprocess.check_call([npm, 'ci', '--production'], cwd=directory, env=env)
+ print('Running post-install step: npm ci ...')
+ try:
+ subprocess.check_output(
+ [npm, 'ci', '--production'],
+ cwd=directory, stderr=subprocess.STDOUT, env=env,
+ universal_newlines=True)
+ except subprocess.CalledProcessError as e:
+ print('Error running %s:\n%s' % (e.cmd, e.output))
+ return False
+
+ print('Done running: npm ci')
return True