From ef40dd4b0bd6f6cf8e0019353180cd403ae17ad3 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Henrik=20Tj=C3=A4der?= Date: Thu, 15 Dec 2022 23:56:01 +0100 Subject: [PATCH] CI: Run rustup and cargo directly --- .github/workflows/build.yml | 141 ++++++++++++------------------------ 1 file changed, 47 insertions(+), 94 deletions(-) diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml index 8205d63a4e..f84fe05770 100644 --- a/.github/workflows/build.yml +++ b/.github/workflows/build.yml @@ -21,22 +21,14 @@ jobs: - name: Checkout uses: actions/checkout@v2 - - name: Install Rust - uses: actions-rs/toolchain@v1 - with: - profile: minimal - toolchain: stable - override: true - components: rustfmt - - name: Fail on warnings run: sed -i 's,//deny_warnings_placeholder_for_ci,#![deny(warnings)],' src/lib.rs macros/src/lib.rs + - name: Add Rust component + run: rustup component add rustfmt + - name: cargo fmt --check - uses: actions-rs/cargo@v1 - with: - command: fmt - args: --all -- --check + run: cargo fmt --all -- --check # Compilation check check: @@ -54,12 +46,13 @@ jobs: - name: Checkout uses: actions/checkout@v2 - - name: Install Rust ${{ matrix.toolchain }} with target (${{ matrix.target }}) - uses: actions-rs/toolchain@v1 - with: - toolchain: ${{ matrix.toolchain }} - target: ${{ matrix.target }} - override: true + - name: Install Rust ${{ matrix.toolchain }} + run: | + rustup set profile minimal + rustup override set ${{ matrix.toolchain }} + + - name: Configure Rust target (${{ matrix.target }}) + run: rustup target add ${{ matrix.target }} - name: Fail on warnings run: sed -i 's,//deny_warnings_placeholder_for_ci,#![deny(warnings)],' src/lib.rs macros/src/lib.rs @@ -68,11 +61,7 @@ jobs: uses: Swatinem/rust-cache@v1 - name: cargo check - uses: actions-rs/cargo@v1 - with: - use-cross: false - command: check - args: --target=${{ matrix.target }} + run: cargo check --target=${{ matrix.target }} # Clippy clippy: @@ -82,13 +71,6 @@ jobs: - name: Checkout uses: actions/checkout@v2 - - name: Install Rust stable - uses: actions-rs/toolchain@v1 - with: - toolchain: stable - target: x86_64-unknown-linux-gnu - override: true - - name: Fail on warnings run: sed -i 's,//deny_warnings_placeholder_for_ci,#![deny(warnings)],' src/lib.rs macros/src/lib.rs @@ -96,10 +78,7 @@ jobs: uses: Swatinem/rust-cache@v1 - name: cargo clippy - uses: actions-rs/cargo@v1 - with: - use-cross: false - command: clippy + run: clippy # Verify all examples, checks checkexamples: @@ -116,23 +95,22 @@ jobs: - name: Checkout uses: actions/checkout@v2 - - name: Install Rust ${{ matrix.toolchain }} with target (${{ matrix.target }}) - uses: actions-rs/toolchain@v1 - with: - toolchain: ${{ matrix.toolchain }} - target: ${{ matrix.target }} - override: true - components: llvm-tools-preview + - name: Install Rust ${{ matrix.toolchain }} + run: | + rustup set profile minimal + rustup override set ${{ matrix.toolchain }} + + - name: Configure Rust target (${{ matrix.target }}) + run: rustup target add ${{ matrix.target }} + + - name: Add Rust component llvm-tools-preview + run: rustup component add llvm-tools-preview - name: Cache Dependencies uses: Swatinem/rust-cache@v1 - name: Check the examples - uses: actions-rs/cargo@v1 - with: - use-cross: false - command: check - args: --examples --target=${{ matrix.target }} + run: cargo check --examples --target=${{ matrix.target }} # Verify the example output with run-pass tests testexamples: @@ -149,20 +127,20 @@ jobs: - name: Checkout uses: actions/checkout@v2 - - name: Install Rust ${{ matrix.toolchain }} with target (${{ matrix.target }}) - uses: actions-rs/toolchain@v1 - with: - toolchain: ${{ matrix.toolchain }} - target: ${{ matrix.target }} - override: true - components: llvm-tools-preview + - name: Install Rust ${{ matrix.toolchain }} + run: | + rustup set profile minimal + rustup override set ${{ matrix.toolchain }} + + - name: Configure Rust target (${{ matrix.target }}) + run: rustup target add ${{ matrix.target }} + + - name: Add Rust component llvm-tools-preview + run: rustup component add llvm-tools-preview # Use precompiled binutils - name: cargo install cargo-binutils - uses: actions-rs/install@v0.1 - with: - crate: cargo-binutils - version: latest + run: cargo install cargo-binutils - name: Cache Dependencies uses: Swatinem/rust-cache@v1 @@ -176,8 +154,7 @@ jobs: run: sed -i 's,//deny_warnings_placeholder_for_ci,#![deny(warnings)],' src/lib.rs macros/src/lib.rs - name: Run-pass tests - run: - cargo xtask --target ${{ matrix.target }} + run: cargo xtask --target ${{ matrix.target }} # Check the correctness of macros/ crate checkmacros: @@ -193,12 +170,13 @@ jobs: - name: Checkout uses: actions/checkout@v2 - - name: Install Rust ${{ matrix.toolchain }} with target (${{ matrix.target }}) - uses: actions-rs/toolchain@v1 - with: - toolchain: ${{ matrix.toolchain }} - target: ${{ matrix.target }} - override: true + - name: Install Rust ${{ matrix.toolchain }} + run: | + rustup set profile minimal + rustup override set ${{ matrix.toolchain }} + + - name: Configure Rust target (${{ matrix.target }}) + run: rustup target add ${{ matrix.target }} - name: Cache Dependencies uses: Swatinem/rust-cache@v1 @@ -207,11 +185,7 @@ jobs: run: sed -i 's,//deny_warnings_placeholder_for_ci,#![deny(warnings)],' src/lib.rs macros/src/lib.rs - name: cargo check - uses: actions-rs/cargo@v1 - with: - use-cross: false - command: check - args: --manifest-path macros/Cargo.toml --target=${{ matrix.target }} + run: cargo check --manifest-path macros/Cargo.toml --target=${{ matrix.target }} # Run the macros test-suite testmacros: @@ -221,13 +195,6 @@ jobs: - name: Checkout uses: actions/checkout@v2 - - name: Install Rust - uses: actions-rs/toolchain@v1 - with: - toolchain: stable - target: x86_64-unknown-linux-gnu - override: true - - name: Cache Dependencies uses: Swatinem/rust-cache@v1 @@ -235,11 +202,7 @@ jobs: run: sed -i 's,//deny_warnings_placeholder_for_ci,#![deny(warnings)],' src/lib.rs macros/src/lib.rs - name: cargo check - uses: actions-rs/cargo@v1 - with: - use-cross: false - command: test - args: --manifest-path macros/Cargo.toml + run: cargo test --manifest-path macros/Cargo.toml # Run test suite tests: @@ -249,24 +212,14 @@ jobs: - name: Checkout uses: actions/checkout@v2 - - name: Install Rust - uses: actions-rs/toolchain@v1 - with: - toolchain: stable - target: x86_64-unknown-linux-gnu - override: true - - name: Cache Dependencies uses: Swatinem/rust-cache@v1 - name: Fail on warnings run: sed -i 's,//deny_warnings_placeholder_for_ci,#![deny(warnings)],' src/lib.rs macros/src/lib.rs - - uses: actions-rs/cargo@v1 - with: - use-cross: false - command: test - args: --test tests + - name: Run cargo test + run: cargo test --test tests # Build documentation, check links docs: