Skip to content

Commit

Permalink
ci: Remove duplicate -p glutin from cargo apk -- invocation
Browse files Browse the repository at this point in the history
`cargo-subcommand` doesn't yet deduplicate this and currently doesn't
support building multiple packages (even if this would result in
building a single package).

It's only used once when calling `cargo $CMD test $OPTIONS`, the other
invocation is guarded for when the target _is not_ Android anyway.
  • Loading branch information
MarijnS95 authored Dec 12, 2022
1 parent f0d907c commit d7f5d84
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@ jobs:
- { target: x86_64-unknown-linux-gnu, os: ubuntu-latest, options: --no-default-features, features: "egl,wayland" }
- { target: x86_64-unknown-linux-gnu, os: ubuntu-latest, options: --no-default-features, features: "egl,x11" }
- { target: x86_64-unknown-linux-gnu, os: ubuntu-latest, options: --no-default-features, features: glx }
- { target: aarch64-linux-android, os: ubuntu-latest, options: -p glutin, cmd: 'apk --' }
- { target: aarch64-linux-android, os: ubuntu-latest, cmd: 'apk --' }
- { target: x86_64-apple-darwin, os: macos-latest, }
# We don't support ios for now.
# - { target: x86_64-apple-ios, os: macos-latest, }
Expand Down Expand Up @@ -94,7 +94,7 @@ jobs:
!contains(matrix.platform.target, 'android') &&
!contains(matrix.platform.target, 'ios') &&
!contains(matrix.platform.target, 'wasm32'))
run: cargo $CMD test --verbose --target ${{ matrix.platform.target }} $OPTIONS --features $FEATURES
run: cargo test --verbose --target ${{ matrix.platform.target }} $OPTIONS --features $FEATURES

- name: Check documentation
shell: bash
Expand Down

0 comments on commit d7f5d84

Please sign in to comment.