Merge pull request #142 from luojia65/luojia65/patch-1

[ci] more proper name for check types, small fix

.github/workflows/Cargo.yml

@@ -27,13 +27,12 @@ jobs:
       - uses: actions/checkout@v4
       - uses: actions-rust-lang/setup-rust-toolchain@v1
         with:
-          toolchain: nightly-2024-10-01
+          toolchain: nightly
       - name: Setup env
         run: |
           rustup component add llvm-tools-preview
           rustup target add riscv64imac-unknown-none-elf
           cargo install cargo-binutils
       - name: Cargo Make
         run: |
-          rustc -vV
-          cargo make --release
+          cargo make --release

.github/workflows/Check.yml

@@ -8,6 +8,7 @@ on:
 
 jobs:
   check:
+    name: Determine Rust or C check
     runs-on: ubuntu-latest
 
     outputs:
@@ -30,12 +31,14 @@ jobs:
             echo "::set-output name=is_rust::false"
           fi
 
-  cargo_check:
+  rust_check:
+    name: Rust
     needs: check
     if: ${{ needs.check.outputs.is_rust == 'true' }}
     uses: ./.github/workflows/Cargo.yml
-  
+
   cmake_check:
+    name: CMake
     needs: check
     if: ${{ needs.check.outputs.is_rust == 'false' }}
     uses: ./.github/workflows/CMake.yml

rust/xtask/src/main.rs

@@ -275,9 +275,11 @@ fn find_binutils_prefix() -> Option<&'static str> {
         let mut command = Command::new(format!("{}objcopy", prefix));
         command.arg("--version");
         command.stdout(Stdio::null());
-        let status = command.status().unwrap();
-        if status.success() {
-            return Some(prefix);
+        let status = command.status();
+        if let Ok(status) = status {
+            if status.success() {
+                return Some(prefix);
+            }
         }
     }
     None