Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Fixes for Bazel 9#216

Draft
theoparis wants to merge 1 commit intogoogle:main from
tinted-software:push-wnvntmnqzvkx
Draft

Fixes for Bazel 9 #216
theoparis wants to merge 1 commit intogoogle:main from
tinted-software:push-wnvntmnqzvkx

Conversation

@theoparis
Copy link

@theoparis theoparis commented Dec 12, 2025
edited
Loading

This PR fixes building Skia with Bazel 9 and 10. I marked it as a draft PR because emsdk needs some changes to fix bazel test "//src/gpu for example and I'm waiting on the 9.0.0 release.

Also, https://github.com/bazelbuild/rules_webtesting is archived and I had to temporarily remove it from the build files. I'm not sure what the replacement for this would be.

Copy link

This PR (HEAD: 2328921) has been imported to Gerrit for code review.

Please visit review.skia.org/1123497 to see it. Please CC yourself to the Gerrit change.

Note:

  • Skia uses only Gerrit for reviews and submitting code (doc).
  • All comments are handled within Gerrit. Any comments on the GitHub PR will be ignored.
  • The PR author can continue to upload commits to the branch used by the PR in order to address feedback from Gerrit.
  • Once the code is ready to be merged, a maintainer will submit the change on Gerrit and skia-codereview-bot will close this PR.
  • Similarly, if a change is abandoned on Gerrit, the corresponding PR will be closed with a note.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /