foster/denoland-deno
1
0
Fork 0
mirror of https://github.com/denoland/deno.git synced 2024-11-21 15:04:11 -05:00
Code Issues Packages 1 Wiki Activity

fix(unstable): panic when running deno install with DENO_FUTURE=1 (#24866)

Browse source 
Not bothering to add a test for this because we're going to change this
to be the default in a couple weeks.
This commit is contained in:
David Sherret 2024-08-05 10:36:09 +02:00 committed by GitHub
parent 71ca61e189
commit 3f79db1486
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
cli/factory.rs
View file

@ -355,7 +355,7 @@ impl CliFactory {
let fs = self.fs(); let fs = self.fs();
let cli_options = self.cli_options()?; let cli_options = self.cli_options()?;
// For `deno install` we want to force the managed resolver so it can set up `node_modules/` directory. // For `deno install` we want to force the managed resolver so it can set up `node_modules/` directory.
create_cli_npm_resolver(if cli_options.use_byonm() && !matches!(cli_options.sub_command(), DenoSubcommand::Install(_)) { create_cli_npm_resolver(if cli_options.use_byonm() && !matches!(cli_options.sub_command(), DenoSubcommand::Install(_) | DenoSubcommand::Add(_)) {
CliNpmResolverCreateOptions::Byonm(CliNpmResolverByonmCreateOptions { CliNpmResolverCreateOptions::Byonm(CliNpmResolverByonmCreateOptions {
fs: fs.clone(), fs: fs.clone(),
root_node_modules_dir: Some(match cli_options.node_modules_dir_path() { root_node_modules_dir: Some(match cli_options.node_modules_dir_path() {