Merge pull request #3529 from reubenmiller/ci-refactor-build-script

ci: refactor build script and workflow

Author: reubenmiller

.github/workflows/build-workflow.yml

@@ -18,11 +18,6 @@ on:
         type: string
         required: false
         default: "10"
-      run_rust_tests:
-        description: Run Rust tests (generally this is not required as the PR check will run full Rust tests)
-        type: boolean
-        required: false
-        default: false
 
   # Use a manual approval process before PR's are given access to
   # the secrets which are required to run the integration tests.
@@ -57,49 +52,60 @@ jobs:
           - riscv64gc-unknown-linux-gnu
           - aarch64-apple-darwin
           - x86_64-apple-darwin
-        mode:
-          - --release
 
         rust_channel:
           - "1.78"
 
         include:
           - target: aarch64-unknown-linux-musl
+            build_with: auto
             host_os: ubuntu-22.04
             cargo_options: --no-run
 
           - target: armv7-unknown-linux-musleabihf
+            build_with: auto
             host_os: ubuntu-22.04
             cargo_options: --no-run
 
           - target: arm-unknown-linux-musleabihf
+            build_with: auto
             host_os: ubuntu-22.04
             cargo_options: --no-run
 
           - target: arm-unknown-linux-musleabi
+            build_with: auto
             host_os: ubuntu-22.04
             cargo_options: --no-run
 
           - target: armv5te-unknown-linux-musleabi
+            build_with: clang
             host_os: ubuntu-22.04
             cargo_options: --no-run
 
           - target: x86_64-unknown-linux-musl
+            build_with: auto
             host_os: ubuntu-22.04
 
           - target: i686-unknown-linux-musl
+            build_with: auto
             host_os: ubuntu-22.04
             cargo_options: --no-run
 
+          # Note: riscv64gc-unknown-linux-musl is meant to be supported from 1.82
+          # but there are still some build problems which prevent it from being used.
+          # So stick to gnu build only
           - target: riscv64gc-unknown-linux-gnu
+            build_with: auto
             host_os: ubuntu-22.04
             cargo_options: --no-run
 
           - target: aarch64-apple-darwin
+            build_with: clang
             host_os: macos-14
             cargo_options: --no-run
 
           - target: x86_64-apple-darwin
+            build_with: clang
             host_os: macos-14
 
     steps:
@@ -112,28 +118,17 @@ jobs:
           ref: ${{ github.event_name == 'pull_request_target' && github.event.pull_request.head.sha || '' }}
           fetch-depth: 0
 
-      - run: mk/install-build-tools.sh +${{ matrix.rust_channel }} --target=${{ matrix.target }}
-        shell: sh
-      - name: Build
-        run: |
-          . ./ci/build_scripts/version.sh
-          mk/cargo.sh +${{ matrix.rust_channel }} build --target=${{ matrix.target }} ${{ matrix.mode }}
-
-      - if: ${{ inputs.run_rust_tests && !contains(matrix.host_os, 'windows') }}
-        name: Test
-        run: |
-          mk/cargo.sh +${{ matrix.rust_channel }} test -vv --target=${{ matrix.target }} ${{ matrix.cargo_options }} ${{ matrix.mode }} --no-fail-fast --locked --all-features --all-targets
-
       # Install nfpm used to for linux packaging
       - uses: actions/setup-go@v5
         with:
           go-version: 'stable'
           cache: false
       - run: go install github.com/goreleaser/nfpm/v2/cmd/nfpm@latest
 
-      - name: Package
+      - uses: taiki-e/install-action@just
+      - name: build
         run: |
-          ./ci/build_scripts/build.sh ${{ matrix.target }} --skip-build --skip-deprecated-packages
+          just release ${{ matrix.target }} --toolchain ${{ matrix.rust_channel }} --build-with ${{ matrix.build_with }}
 
       - name: Upload packages as zip
         # https://github.com/marketplace/actions/upload-a-build-artifact