From 940d22ae030e4c2b1356b805f595a04716878288 Mon Sep 17 00:00:00 2001 From: Thomas Eizinger Date: Thu, 27 Mar 2025 11:06:32 +1100 Subject: [PATCH] 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. --- clippy.sh | 17 +---------------- 1 file changed, 1 insertion(+), 16 deletions(-) diff --git a/clippy.sh b/clippy.sh index b690f988..1fd8f314 100755 --- a/clippy.sh +++ b/clippy.sh @@ -2,19 +2,4 @@ set -eux -# We cannot run clippy over the whole workspace at once due to feature unification. Since both -# integration-test and integration-ebpf depend on integration-common and integration-test activates -# integration-common's aya dependency, we end up trying to compile the panic handler twice: once -# from the bpf program, and again from std via aya. -# -# `-C panic=abort` because "unwinding panics are not supported without std"; integration-ebpf -# contains `#[no_std]` binaries. -# -# `-Zpanic_abort_tests` because "building tests with panic=abort is not supported without -# `-Zpanic_abort_tests`"; Cargo does this automatically when panic=abort is set via profile but we -# want to preserve unwinding at runtime - here we are just running clippy so we don't care about -# unwinding behavior. -# -# `+nightly` because "the option `Z` is only accepted on the nightly compiler". -cargo +nightly hack clippy "$@" --exclude integration-ebpf --all-targets --feature-powerset --workspace -- --deny warnings -cargo +nightly hack clippy "$@" --package integration-ebpf --all-targets --feature-powerset -- --deny warnings -C panic=abort -Zpanic_abort_tests +cargo +nightly hack clippy "$@" --all-targets --feature-powerset --workspace -- --deny warnings