Skip to content

Commit 3659d9d

Browse files
committed
bump-stage0: use IndexMap for determinism
When it used `HashMap`, `bump-stage0` would change `src/stage0` every time it ran, whereas `IndexMap` will keep insertion order -- matching the manifest file.
1 parent f8060d2 commit 3659d9d

File tree

1 file changed

+2
-4
lines changed

1 file changed

+2
-4
lines changed

src/tools/bump-stage0/src/main.rs

+2-4
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,5 @@
11
#![deny(unused_variables)]
22

3-
use std::collections::HashMap;
4-
53
use anyhow::{Context, Error};
64
use build_helper::stage0_parser::{parse_stage0_file, Stage0Config, VersionMetadata};
75
use curl::easy::Easy;
@@ -217,13 +215,13 @@ enum Channel {
217215
#[derive(Debug, serde::Serialize, serde::Deserialize)]
218216
struct Manifest {
219217
date: String,
220-
pkg: HashMap<String, ManifestPackage>,
218+
pkg: IndexMap<String, ManifestPackage>,
221219
}
222220

223221
#[derive(Debug, serde::Serialize, serde::Deserialize)]
224222
struct ManifestPackage {
225223
version: String,
226-
target: HashMap<String, ManifestTargetPackage>,
224+
target: IndexMap<String, ManifestTargetPackage>,
227225
}
228226

229227
#[derive(Debug, serde::Serialize, serde::Deserialize)]

0 commit comments

Comments
 (0)