Matrixify docker platform in CI

cross-rs · Dec 19, 2024 · 6428332 · 6428332
1 parent 101281f
commit 6428332
Show file tree

Hide file tree

Showing 3 changed files with 21 additions and 17 deletions.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -186,7 +186,7 @@ jobs:
         id: build-docker-image
         if: steps.prepare-meta.outputs.has-image
         timeout-minutes: 120
-        run: cargo xtask build-docker-image -v "${TARGET}${SUB:+.$SUB}" ${{ matrix.verbose && '-v' || ''  }}
+        run: cargo xtask build-docker-image -v --platform ${{ matrix.platform }} "${TARGET}${SUB:+.$SUB}" ${{ matrix.verbose && '-v' || ''  }}
         env:
           TARGET: ${{ matrix.target }}
           SUB: ${{ matrix.sub }}

diff --git a/docker/cross-toolchains b/docker/cross-toolchains
diff --git a/xtask/src/ci/target_matrix.rs b/xtask/src/ci/target_matrix.rs
@@ -101,20 +101,25 @@ impl TargetMatrix {
 
         let matrix = matrix
             .iter()
-            .map(|target| TargetMatrixElement {
-                pretty: target.to_image_target().alt(),
-                platforms: target.platforms(),
-                target: &target.target,
-                sub: target.sub.as_deref(),
-                os: &target.os,
-                run: target.run.map(|b| b as u8),
-                deploy: target.deploy.map(|b| b as u8),
-                build_std: target.build_std.map(|b| b as u8),
-                cpp: target.cpp.map(|b| b as u8),
-                dylib: target.dylib.map(|b| b as u8),
-                runners: target.runners.as_deref(),
-                std: target.std.map(|b| b as u8),
-                verbose: app.verbose,
+            .flat_map(|target| {
+                target
+                    .platforms()
+                    .iter()
+                    .map(move |platform| TargetMatrixElement {
+                        pretty: format!("{} ({platform})", target.to_image_target().alt()),
+                        platform: platform.to_string(),
+                        target: &target.target,
+                        sub: target.sub.as_deref(),
+                        os: &target.os,
+                        run: target.run.map(|b| b as u8),
+                        deploy: target.deploy.map(|b| b as u8),
+                        build_std: target.build_std.map(|b| b as u8),
+                        cpp: target.cpp.map(|b| b as u8),
+                        dylib: target.dylib.map(|b| b as u8),
+                        runners: target.runners.as_deref(),
+                        std: target.std.map(|b| b as u8),
+                        verbose: app.verbose,
+                    })
             })
             .collect::<Vec<_>>();
 
@@ -244,7 +249,7 @@ fn process_try_comment(message: &str) -> cross::Result<(bool, TargetMatrixArgs)>
 #[serde(rename_all = "kebab-case")]
 struct TargetMatrixElement<'a> {
     pretty: String,
-    platforms: &'a [String],
+    platform: String,
     target: &'a str,
     #[serde(skip_serializing_if = "Option::is_none")]
     sub: Option<&'a str>,