| commit | 1fa3a29e66bd220f33202be998ae0a883c749bd2 | [log] [tgz] |
|---|---|---|
| author | Kaylee Lubick <kjlubick@google.com> | Mon Sep 09 12:42:24 2024 +0000 |
| committer | SkCQ <skcq-be@skia-corp.google.com.iam.gserviceaccount.com> | Mon Sep 09 13:28:55 2024 +0000 |
| tree | 3ea600714bf948ce1ed5abb30e2bc260594d6e84 | |
| parent | 06cd203d0607e1c7b3e25d9f08eb2c67c11c268c [diff] |
Revert "Update to Bazel 7" This reverts commit 323911941086e3c9652b98a319bcbd7dd5c9210c. Reason for revert: Breaking Build-Mac-Clang-x86_64-Debug-Fontations and Housekeeper-PerCommit-CheckGeneratedFiles Original change's description: > Update to Bazel 7 > > In particular this allows us to use https://bazel.build/reference/be/c-cpp#cc_library.implementation_deps > > Note that to update to Bazel 8, we will need to migrate to Bazel > modules https://bazel.build/external/overview#bzlmod > > We do have to disable some tests that start failing, not because > anything changed with how the C++ code was made, but with how > the Bazel rules make files available. We don't have the staffing > to investigate these thoroughly at the moment, so I've disabled > the Bazel testing of CanvasKit (we still do BazelBuild) and the > experimental Bazel-test-on-android rules. > > Change-Id: I21371c8d6a7a28babe2b5242dfeb61290726998a > Reviewed-on: https://skia-review.googlesource.com/c/skia/+/897100 > Reviewed-by: Ben Wagner <bungeman@google.com> Change-Id: I1c0ecdc2206716453a9bb1431f091f605d383a08 Reviewed-on: https://skia-review.googlesource.com/c/skia/+/897636 Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com> Commit-Queue: Kaylee Lubick <kjlubick@google.com>