Skip to content

Commit c5ab958

Browse files
alan-agius4vikerman
authored andcommitted
fix(@angular/cli): handle null stderr during npm install
1 parent da6d74a commit c5ab958

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

packages/angular/cli/tasks/install-package.ts

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -48,18 +48,18 @@ export function installPackage(
4848
installArgs.push(packageManagerArgs.saveDev);
4949
}
5050

51-
const { status, stderr } = spawnSync(packageManager, [...installArgs, ...extraArgs], {
51+
const { status, stderr, stdout } = spawnSync(packageManager, [...installArgs, ...extraArgs], {
5252
stdio: 'pipe',
5353
encoding: 'utf8',
5454
cwd,
5555
});
5656

5757
if (status !== 0) {
58-
let errors = stderr.trim();
59-
if (errors.length) {
60-
errors += '\n';
58+
let error = (stderr || stdout).trim();
59+
if (error) {
60+
error += '\n';
6161
}
62-
throw new Error(errors + `Package install failed${errors.length ? ', see above' : ''}.`);
62+
throw new Error(error + `Package install failed${error ? ', see above' : ''}.`);
6363
}
6464

6565
logger.info(colors.green(`Installed packages for tooling via ${packageManager}.`));

0 commit comments

Comments
 (0)