From 08f72d2ff2819eb40cabd21f0a482cc17a725370 Mon Sep 17 00:00:00 2001 From: Ryo Onodera Date: Fri, 9 Aug 2024 00:17:20 +0900 Subject: [PATCH] Make dcou faster and check lib targets as well (#2127) --- ci/buildkite-pipeline.sh | 5 +++-- ci/test-dev-context-only-utils.sh | 3 ++- scripts/check-dev-context-only-utils.sh | 10 ++++++---- 3 files changed, 11 insertions(+), 7 deletions(-) diff --git a/ci/buildkite-pipeline.sh b/ci/buildkite-pipeline.sh index 0c8327ee5961b3..9246ad27e4c6c4 100755 --- a/ci/buildkite-pipeline.sh +++ b/ci/buildkite-pipeline.sh @@ -181,8 +181,9 @@ wait_step() { all_test_steps() { command_step checks1 "ci/docker-run-default-image.sh ci/test-checks.sh" 20 check - command_step checks2 "ci/docker-run-default-image.sh ci/test-dev-context-only-utils.sh check-bins" 15 check - command_step checks3 "ci/docker-run-default-image.sh ci/test-dev-context-only-utils.sh check-all-targets" 15 check + command_step dcou-1-of-3 "ci/docker-run-default-image.sh ci/test-dev-context-only-utils.sh --partition 1/3" 15 check + command_step dcou-2-of-3 "ci/docker-run-default-image.sh ci/test-dev-context-only-utils.sh --partition 2/3" 15 check + command_step dcou-3-of-3 "ci/docker-run-default-image.sh ci/test-dev-context-only-utils.sh --partition 3/3" 15 check command_step miri "ci/docker-run-default-image.sh ci/test-miri.sh" 5 check command_step frozen-abi "ci/docker-run-default-image.sh ./test-abi.sh" 15 check wait_step diff --git a/ci/test-dev-context-only-utils.sh b/ci/test-dev-context-only-utils.sh index bec640cdf209f8..6694b457145d58 100755 --- a/ci/test-dev-context-only-utils.sh +++ b/ci/test-dev-context-only-utils.sh @@ -2,4 +2,5 @@ set -eo pipefail -scripts/check-dev-context-only-utils.sh "$@" +scripts/check-dev-context-only-utils.sh check-all-targets "$@" +scripts/check-dev-context-only-utils.sh check-bins-and-lib "$@" diff --git a/scripts/check-dev-context-only-utils.sh b/scripts/check-dev-context-only-utils.sh index 65382d20663e46..78dcd6cad98773 100755 --- a/scripts/check-dev-context-only-utils.sh +++ b/scripts/check-dev-context-only-utils.sh @@ -42,8 +42,10 @@ printf -v allowed '"%s",' "${tainted_packages[@]}" allowed="${allowed%,}" mode=${1:-full} +# consume the mode, so that other arguments are forwarded to cargo-hack +shift case "$mode" in - tree | check-bins | check-all-targets | full) + tree | check-bins-and-lib | check-all-targets | full) ;; *) echo "$0: unrecognized mode: $mode"; @@ -156,9 +158,9 @@ fi # consistency with other CI steps and for the possibility of new similar lints. export RUSTFLAGS="-D warnings -Z threads=8 $RUSTFLAGS" -if [[ $mode = "check-bins" || $mode = "full" ]]; then - _ cargo "+${rust_nightly}" hack check --bins +if [[ $mode = "check-bins-and-lib" || $mode = "full" ]]; then + _ cargo "+${rust_nightly}" hack "$@" check fi if [[ $mode = "check-all-targets" || $mode = "full" ]]; then - _ cargo "+${rust_nightly}" hack check --all-targets + _ cargo "+${rust_nightly}" hack "$@" check --all-targets fi