fix: refactor CI workflows for Windows and update RISC-V config

- Split cargo-hack checks and clippy runs into separate steps for TLS options
- Simplify artifact naming in cross-compile workflow
- Update RISC-V target to use rustls-tls and remove OpenSSL steps

Author: roderickvd

.github/workflows/build.yml

@@ -74,15 +74,18 @@ jobs:
       - name: Install cargo-hack
         uses: taiki-e/install-action@cargo-hack
 
-      - name: Check workspace (each feature combination)
-        run: |
-          # Test packages that don't require TLS
-          cargo hack check -p librespot-protocol --each-feature
-          # Test workspace root with each TLS option
-          cargo hack check -p librespot --each-feature --exclude-all-features \
-            --include-features native-tls --exclude-features rustls-tls
-          cargo hack check -p librespot --each-feature --exclude-all-features \
-            --include-features rustls-tls --exclude-features native-tls
+      - name: Check packages without TLS requirements
+        run: cargo hack check -p librespot-protocol --each-feature
+
+      - name: Check workspace with native-tls
+        run: >
+          cargo hack check -p librespot --each-feature --exclude-all-features
+          --include-features native-tls --exclude-features rustls-tls
+
+      - name: Check workspace with rustls-tls
+        run: >
+          cargo hack check -p librespot --each-feature --exclude-all-features
+          --include-features rustls-tls --exclude-features native-tls
 
       - name: Build binary with default features
         run: cargo build --frozen

.github/workflows/cross-compile.yml

@@ -73,8 +73,6 @@ jobs:
       - name: Upload debug artifacts
         uses: actions/upload-artifact@v4
         with:
-          name: >
-            librespot-${{ matrix.platform.runs-on }}-
-            ${{ matrix.platform.arch }}-${{ matrix.toolchain }}
+          name: librespot-${{ matrix.platform.runs-on }}-${{ matrix.platform.arch }}-${{ matrix.toolchain }} # yamllint disable-line rule:line-length
           path: target/${{ matrix.platform.target }}/debug/librespot
           if-no-files-found: error

.github/workflows/quality.yml

@@ -65,12 +65,15 @@ jobs:
       - name: Install cargo-hack
         uses: taiki-e/install-action@cargo-hack
 
-      - name: Run clippy (each feature combination)
-        run: |
-          # Test packages that don't require TLS
-          cargo hack clippy -p librespot-protocol --each-feature
-          # Test workspace root with each TLS option
-          cargo hack clippy -p librespot --each-feature --exclude-all-features \
-            --include-features native-tls --exclude-features rustls-tls
-          cargo hack clippy -p librespot --each-feature --exclude-all-features \
-            --include-features rustls-tls --exclude-features native-tls
+      - name: Run clippy on packages without TLS requirements
+        run: cargo hack clippy -p librespot-protocol --each-feature
+
+      - name: Run clippy with native-tls
+        run: >
+          cargo hack clippy -p librespot --each-feature --exclude-all-features
+          --include-features native-tls --exclude-features rustls-tls
+
+      - name: Run clippy with rustls-tls
+        run: >
+          cargo hack clippy -p librespot --each-feature --exclude-all-features
+          --include-features rustls-tls --exclude-features native-tls

Cross.toml

@@ -6,10 +6,7 @@ pre-build = [
 ]
 
 [target.riscv64gc-unknown-linux-gnu]
-# RISC-V: Try system packages first, fall back to vendored OpenSSL via feature flag
-# Building with --no-default-features to avoid Rodio (which needs ALSA)
-pre-build = [
-    "dpkg --add-architecture $CROSS_DEB_ARCH",
-    "apt-get update",
-    "apt-get --assume-yes install libssl-dev:$CROSS_DEB_ARCH || true",
-]
+# RISC-V: Uses rustls-tls (no system dependencies needed)
+# Building with --no-default-features --features rustls-tls
+# No pre-build steps required - rustls is pure Rust
+pre-build = []