Don't release Miri if its tests only failed on Windows

.github/workflows/ci.yml

@@ -349,6 +349,7 @@ jobs:
             env:
               SCRIPT: src/ci/docker/host-x86_64/x86_64-gnu-tools/checktools.sh x.py /tmp/toolstate/toolstates.json windows
               RUST_CONFIGURE_ARGS: "--build=x86_64-pc-windows-msvc --save-toolstates=/tmp/toolstate/toolstates.json"
+              DEPLOY_TOOLSTATES_JSON: toolstates-windows.json
             os: windows-latest-xl
           - name: i686-mingw-1
             env:

src/ci/github-actions/ci.yml

@@ -531,6 +531,7 @@ jobs:
             env:
               SCRIPT: src/ci/docker/host-x86_64/x86_64-gnu-tools/checktools.sh x.py /tmp/toolstate/toolstates.json windows
               RUST_CONFIGURE_ARGS: --build=x86_64-pc-windows-msvc --save-toolstates=/tmp/toolstate/toolstates.json
+              DEPLOY_TOOLSTATES_JSON: toolstates-windows.json
             <<: *job-windows-xl
 
           # 32/64-bit MinGW builds.

src/tools/build-manifest/src/main.rs

@@ -257,21 +257,20 @@ impl Builder {
     /// If a tool does not pass its tests, don't ship it.
     /// Right now, we do this only for Miri.
     fn check_toolstate(&mut self) {
-        let toolstates: Option<HashMap<String, String>> =
+        for file in &["toolstates-linux.json", "toolstates-windows.json"] {
-            File::open(self.input.join("toolstates-linux.json"))
+            let toolstates: Option<HashMap<String, String>> = File::open(self.input.join(file))
                 .ok()
                 .and_then(|f| serde_json::from_reader(&f).ok());
-        let toolstates = toolstates.unwrap_or_else(|| {
+            let toolstates = toolstates.unwrap_or_else(|| {
-            println!(
+                println!("WARNING: `{}` missing/malformed; assuming all tools failed", file);
-                "WARNING: `toolstates-linux.json` missing/malformed; \
+                HashMap::default() // Use empty map if anything went wrong.
-                assuming all tools failed"
+            });
-            );
+            // Mark some tools as missing based on toolstate.
-            HashMap::default() // Use empty map if anything went wrong.
+            if toolstates.get("miri").map(|s| &*s as &str) != Some("test-pass") {
-        });
+                println!("Miri tests are not passing, removing component");
-        // Mark some tools as missing based on toolstate.
+                self.versions.disable_version(&PkgType::Miri);
-        if toolstates.get("miri").map(|s| &*s as &str) != Some("test-pass") {
+                break;
-            println!("Miri tests are not passing, removing component");
+            }
-            self.versions.disable_version(&PkgType::Miri);
         }
     }