diff --git a/libcxx/utils/ci/buildkite-pipeline-clang.yml b/libcxx/utils/ci/buildkite-pipeline-clang.yml --- a/libcxx/utils/ci/buildkite-pipeline-clang.yml +++ b/libcxx/utils/ci/buildkite-pipeline-clang.yml @@ -19,7 +19,7 @@ steps: - label: "Format" commands: - - "! grep -rnI '[[:blank:]]$' clang/lib clang/include clang/docs || false" + - '! grep -rnI ''[[:blank:]]$'' clang/lib clang/include clang/docs || false' agents: queue: "libcxx-builders" diff --git a/libcxx/utils/ci/generate-buildkite-pipeline b/libcxx/utils/ci/generate-buildkite-pipeline --- a/libcxx/utils/ci/generate-buildkite-pipeline +++ b/libcxx/utils/ci/generate-buildkite-pipeline @@ -11,16 +11,4 @@ # This script generates the appropriate libc++ CI pipeline based on which project(s) were changed. # -if git diff --name-only HEAD~1 | grep -q -E "^libcxx/|^libcxxabi/|^libunwind/|^runtimes/|^cmake/"; then - LIBCXX_CHANGED=true -fi - -if git diff --name-only HEAD~1 | grep -q -E "^clang/"; then - CLANG_CHANGED=true -fi - -if [[ "${CLANG_CHANGED}" == "true" && "${LIBCXX_CHANGED}" != "true" ]]; then - cat libcxx/utils/ci/buildkite-pipeline-clang.yml -else - cat libcxx/utils/ci/buildkite-pipeline.yml -fi +cat libcxx/utils/ci/buildkite-pipeline-clang.yml diff --git a/libcxx/utils/ci/run-buildbot b/libcxx/utils/ci/run-buildbot --- a/libcxx/utils/ci/run-buildbot +++ b/libcxx/utils/ci/run-buildbot @@ -203,6 +203,9 @@ # Check if the diff is empty, fail otherwise. ! grep -q '^--- a' ${BUILD_DIR}/clang-format.patch ;; +check-format-clang) + ! grep -rnI '[[:blank:]]$' clang/lib clang/include clang/docs +;; check-generated-output) # `! foo` doesn't work properly with `set -e`, use `! foo || false` instead. # https://stackoverflow.com/questions/57681955/set-e-does-not-respect-logical-not