diff options
Diffstat (limited to 'cli')
| -rw-r--r-- | cli/Cargo.toml | 4 | ||||
| -rw-r--r-- | cli/mainrt.rs | 3 | ||||
| -rw-r--r-- | cli/standalone/binary.rs | 24 | ||||
| -rw-r--r-- | cli/standalone/virtual_fs.rs | 7 |
4 files changed, 24 insertions, 14 deletions
diff --git a/cli/Cargo.toml b/cli/Cargo.toml index 5d329abce..85a7bc9bb 100644 --- a/cli/Cargo.toml +++ b/cli/Cargo.toml @@ -72,13 +72,13 @@ deno_emit = "=0.44.0" deno_graph = { version = "=0.81.2" } deno_lint = { version = "=0.63.1", features = ["docs"] } deno_lockfile.workspace = true -deno_npm = "=0.22.0" +deno_npm = "=0.23.0" deno_package_json.workspace = true deno_runtime = { workspace = true, features = ["include_js_files_for_snapshotting"] } deno_semver = "=0.5.10" deno_task_shell = "=0.17.0" deno_terminal.workspace = true -eszip = "=0.74.0" +eszip = "=0.75.0" libsui = "0.3.0" napi_sym.workspace = true node_resolver.workspace = true diff --git a/cli/mainrt.rs b/cli/mainrt.rs index 9bf318953..fd03224be 100644 --- a/cli/mainrt.rs +++ b/cli/mainrt.rs @@ -31,6 +31,7 @@ use deno_runtime::fmt_errors::format_js_error; use deno_runtime::tokio_util::create_and_run_current_thread_with_maybe_metrics; pub use deno_runtime::UNSTABLE_GRANULAR_FLAGS; use deno_terminal::colors; +use indexmap::IndexMap; use std::borrow::Cow; use std::collections::HashMap; @@ -73,7 +74,7 @@ fn unwrap_or_exit<T>(result: Result<T, AnyError>) -> T { } } -fn load_env_vars(env_vars: &HashMap<String, String>) { +fn load_env_vars(env_vars: &IndexMap<String, String>) { env_vars.iter().for_each(|env_var| { if env::var(env_var.0).is_err() { std::env::set_var(env_var.0, env_var.1); diff --git a/cli/standalone/binary.rs b/cli/standalone/binary.rs index 168310efd..d80f8a969 100644 --- a/cli/standalone/binary.rs +++ b/cli/standalone/binary.rs @@ -100,6 +100,8 @@ pub struct SerializedWorkspaceResolver { pub pkg_json_resolution: PackageJsonDepResolution, } +// Note: Don't use hashmaps/hashsets. Ensure the serialization +// is deterministic. #[derive(Deserialize, Serialize)] pub struct Metadata { pub argv: Vec<String>, @@ -111,7 +113,7 @@ pub struct Metadata { pub ca_stores: Option<Vec<String>>, pub ca_data: Option<Vec<u8>>, pub unsafely_ignore_certificate_errors: Option<Vec<String>>, - pub env_vars_from_env_file: HashMap<String, String>, + pub env_vars_from_env_file: IndexMap<String, String>, pub workspace_resolver: SerializedWorkspaceResolver, pub entrypoint_key: String, pub node_modules: Option<NodeModules>, @@ -667,8 +669,10 @@ impl<'a> DenoCompileBinaryWriter<'a> { // but also don't make this dependent on the registry url let root_path = npm_resolver.global_cache_root_folder(); let mut builder = VfsBuilder::new(root_path)?; - for package in npm_resolver.all_system_packages(&self.npm_system_info) - { + let mut packages = + npm_resolver.all_system_packages(&self.npm_system_info); + packages.sort_by(|a, b| a.id.cmp(&b.id)); // determinism + for package in packages { let folder = npm_resolver.resolve_pkg_folder_from_pkg_id(&package.id)?; builder.add_dir_recursive(&folder)?; @@ -729,11 +733,13 @@ impl<'a> DenoCompileBinaryWriter<'a> { cli_options.workspace().root_dir().to_file_path().unwrap(), ); while let Some(pending_dir) = pending_dirs.pop_front() { - let entries = fs::read_dir(&pending_dir).with_context(|| { - format!("Failed reading: {}", pending_dir.display()) - })?; + let mut entries = fs::read_dir(&pending_dir) + .with_context(|| { + format!("Failed reading: {}", pending_dir.display()) + })? + .collect::<Result<Vec<_>, _>>()?; + entries.sort_by_cached_key(|entry| entry.file_name()); // determinism for entry in entries { - let entry = entry?; let path = entry.path(); if !path.is_dir() { continue; @@ -755,8 +761,8 @@ impl<'a> DenoCompileBinaryWriter<'a> { /// in the passed environment file. fn get_file_env_vars( filename: String, -) -> Result<HashMap<String, String>, dotenvy::Error> { - let mut file_env_vars = HashMap::new(); +) -> Result<IndexMap<String, String>, dotenvy::Error> { + let mut file_env_vars = IndexMap::new(); for item in dotenvy::from_filename_iter(filename)? { let Ok((key, val)) = item else { continue; // this failure will be warned about on load diff --git a/cli/standalone/virtual_fs.rs b/cli/standalone/virtual_fs.rs index c44e2227b..53d045b62 100644 --- a/cli/standalone/virtual_fs.rs +++ b/cli/standalone/virtual_fs.rs @@ -90,8 +90,11 @@ impl VfsBuilder { let read_dir = std::fs::read_dir(path) .with_context(|| format!("Reading {}", path.display()))?; - for entry in read_dir { - let entry = entry?; + let mut dir_entries = + read_dir.into_iter().collect::<Result<Vec<_>, _>>()?; + dir_entries.sort_by_cached_key(|entry| entry.file_name()); // determinism + + for entry in dir_entries { let file_type = entry.file_type()?; let path = entry.path(); |