diff --git a/src/bootstrap/config.rs b/src/bootstrap/config.rs index d6f77fe6cd6d0..ccc6c49d2b246 100644 --- a/src/bootstrap/config.rs +++ b/src/bootstrap/config.rs @@ -387,7 +387,7 @@ macro_rules! derive_merge { derive_merge! { /// TOML representation of various global build decisions. - #[derive(Deserialize, Default, Clone)] + #[derive(Deserialize, Default)] #[serde(deny_unknown_fields, rename_all = "kebab-case")] struct Build { build: Option, @@ -434,7 +434,7 @@ derive_merge! { derive_merge! { /// TOML representation of various global install decisions. - #[derive(Deserialize, Default, Clone)] + #[derive(Deserialize)] #[serde(deny_unknown_fields, rename_all = "kebab-case")] struct Install { prefix: Option, @@ -449,7 +449,7 @@ derive_merge! { derive_merge! { /// TOML representation of how the LLVM build is configured. - #[derive(Deserialize, Default)] + #[derive(Deserialize)] #[serde(deny_unknown_fields, rename_all = "kebab-case")] struct Llvm { skip_rebuild: Option, @@ -483,7 +483,7 @@ derive_merge! { } derive_merge! { - #[derive(Deserialize, Default, Clone)] + #[derive(Deserialize)] #[serde(deny_unknown_fields, rename_all = "kebab-case")] struct Dist { sign_folder: Option, @@ -510,7 +510,7 @@ impl Default for StringOrBool { derive_merge! { /// TOML representation of how the Rust build is configured. - #[derive(Deserialize, Default)] + #[derive(Deserialize)] #[serde(deny_unknown_fields, rename_all = "kebab-case")] struct Rust { optimize: Option, @@ -565,7 +565,7 @@ derive_merge! { derive_merge! { /// TOML representation of how each build target is configured. - #[derive(Deserialize, Default)] + #[derive(Deserialize)] #[serde(deny_unknown_fields, rename_all = "kebab-case")] struct TomlTarget { cc: Option, diff --git a/src/bootstrap/toolstate.rs b/src/bootstrap/toolstate.rs index 2394c5e020d2b..08d0815806207 100644 --- a/src/bootstrap/toolstate.rs +++ b/src/bootstrap/toolstate.rs @@ -24,7 +24,7 @@ const OS: Option<&str> = None; type ToolstateData = HashMap, ToolState>; -#[derive(Copy, Clone, Debug, Deserialize, Serialize, PartialEq, Eq, PartialOrd)] +#[derive(Copy, Clone, Debug, Deserialize, Serialize, PartialEq, PartialOrd)] #[serde(rename_all = "kebab-case")] /// Whether a tool can be compiled, tested or neither pub enum ToolState { @@ -50,13 +50,6 @@ impl fmt::Display for ToolState { } } -impl Default for ToolState { - fn default() -> Self { - // err on the safe side - ToolState::BuildFail - } -} - /// Number of days after the last promotion of beta. /// Its value is 41 on the Tuesday where "Promote master to beta (T-2)" happens. /// The Wednesday after this has value 0. @@ -466,13 +459,11 @@ fn publish_test_results(current_toolstate: &ToolstateData) { t!(fs::write(&history_path, file)); } -#[derive(Debug, Serialize, Deserialize)] +#[derive(Debug, Deserialize)] struct RepoState { tool: String, windows: ToolState, linux: ToolState, - commit: String, - datetime: String, } impl RepoState {