Skip to content

Commit 868b930

Browse files
committed
Auto merge of #148665 - yotamofek:pr/npm-ignore-scripts, r=Kobzol
Add `--ignore-scripts` flag to npm install invocation Newest version of rollup is broken: rollup/rollup#6168 This allows builds to pass until rollup releases a fix and/or we implement a better solution to the fact that npm is not honoring our package lock files
2 parents 843f8ce + 4f69a85 commit 868b930

File tree

1 file changed

+2
-0
lines changed

1 file changed

+2
-0
lines changed

src/build_helper/src/npm.rs

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,8 @@ pub fn install(src_root_path: &Path, out_dir: &Path, npm: &Path) -> Result<PathB
2929
// this makes tidy output less noisy, and also significantly improves runtime
3030
// of repeated tidy invocations.
3131
cmd.args(&["--audit=false", "--save=false", "--fund=false"]);
32+
// temporary workaround for rollup breakage
33+
cmd.arg("--ignore-scripts");
3234
cmd.current_dir(out_dir);
3335
let exit_status = cmd.spawn()?.wait()?;
3436
if !exit_status.success() {

0 commit comments

Comments
 (0)