Skip to content

Commit d246b9f

Browse files
clippy.sh: simplify script
Now that all eBPF crates compile on the host architecture by default, we can simplify the clippy.sh script to just run on all crates at once.
1 parent 0efde5c commit d246b9f

File tree

1 file changed

+1
-16
lines changed

1 file changed

+1
-16
lines changed

clippy.sh

Lines changed: 1 addition & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -2,19 +2,4 @@
22

33
set -eux
44

5-
# We cannot run clippy over the whole workspace at once due to feature unification. Since both
6-
# integration-test and integration-ebpf depend on integration-common and integration-test activates
7-
# integration-common's aya dependency, we end up trying to compile the panic handler twice: once
8-
# from the bpf program, and again from std via aya.
9-
#
10-
# `-C panic=abort` because "unwinding panics are not supported without std"; integration-ebpf
11-
# contains `#[no_std]` binaries.
12-
#
13-
# `-Zpanic_abort_tests` because "building tests with panic=abort is not supported without
14-
# `-Zpanic_abort_tests`"; Cargo does this automatically when panic=abort is set via profile but we
15-
# want to preserve unwinding at runtime - here we are just running clippy so we don't care about
16-
# unwinding behavior.
17-
#
18-
# `+nightly` because "the option `Z` is only accepted on the nightly compiler".
19-
cargo +nightly hack clippy "$@" --exclude integration-ebpf --all-targets --feature-powerset --workspace -- --deny warnings
20-
cargo +nightly hack clippy "$@" --package integration-ebpf --all-targets --feature-powerset -- --deny warnings -C panic=abort -Zpanic_abort_tests
5+
cargo +nightly hack clippy "$@" --all-targets --feature-powerset --workspace -- --deny warnings

0 commit comments

Comments
 (0)