Skip to content

Commit 68c1756

Browse files
authored
aes: CI config cleanups (#209)
AES-NI support was not being properly tested on both 32-bit and 64-bit targets. Also adds better testing for various feature combinations.
1 parent c5016a7 commit 68c1756

File tree

1 file changed

+14
-6
lines changed

1 file changed

+14
-6
lines changed

.github/workflows/aes.yml

Lines changed: 14 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,7 @@ env:
1717
RUSTFLAGS: "-Dwarnings"
1818

1919
jobs:
20+
# Builds for no_std platforms
2021
build:
2122
runs-on: ubuntu-latest
2223
strategy:
@@ -36,7 +37,9 @@ jobs:
3637
profile: minimal
3738
override: true
3839
- run: cargo build --release --target ${{ matrix.target }}
39-
- run: cargo build --release --target ${{ matrix.target }} --all-features
40+
- run: cargo build --release --target ${{ matrix.target }} --features compact
41+
- run: cargo build --release --target ${{ matrix.target }} --features ctr
42+
- run: cargo build --release --target ${{ matrix.target }} --features compact,ctr
4043

4144
# Tests for the portable software backend
4245
soft:
@@ -99,13 +102,16 @@ jobs:
99102
- uses: actions/checkout@v1
100103
- uses: actions-rs/toolchain@v1
101104
with:
102-
profile: minimal
103105
toolchain: ${{ matrix.rust }}
106+
target: ${{ matrix.target }}
107+
profile: minimal
108+
override: true
104109
- run: ${{ matrix.deps }}
105-
- run: cargo check --all-features
106-
- run: cargo test --no-default-features
107-
- run: cargo test
108-
- run: cargo test --all-features
110+
- run: cargo check --target ${{ matrix.target }} --all-features
111+
- run: cargo test --release --target ${{ matrix.target }}
112+
- run: cargo test --release --target ${{ matrix.target }} --features compact
113+
- run: cargo test --release --target ${{ matrix.target }} --features ctr
114+
- run: cargo test --release --target ${{ matrix.target }} --all-features
109115

110116
# Cross-compiled tests
111117
cross:
@@ -137,3 +143,5 @@ jobs:
137143
- run: cargo install cross
138144
- run: cross test --release --target ${{ matrix.target }}
139145
- run: cross test --release --target ${{ matrix.target }} --features compact
146+
- run: cross test --release --target ${{ matrix.target }} --features ctr
147+
- run: cross test --release --target ${{ matrix.target }} --features compact,ctr

0 commit comments

Comments
 (0)