Skip to content

kythe: fix Bazel 9 workflow bugs, add local runner, scope to //proto/...#11479

Merged
dan-stowell merged 3 commits intomasterfrom
bbdan-kythe-loop
Mar 3, 2026
Merged

kythe: fix Bazel 9 workflow bugs, add local runner, scope to //proto/...#11479
dan-stowell merged 3 commits intomasterfrom
bbdan-kythe-loop

Conversation

@dan-stowell
Copy link
Contributor

@dan-stowell dan-stowell commented Mar 3, 2026

Summary

  • Bazel 9 fixes for "Generate Kythe Annotations" workflow: add --incompatible_autoload_externally for Java rules Kythe needs, add --nokeep_going to fail fast instead of inheriting extractors.bazelrc's --keep_going, and add --deleted_packages to prevent the local Kythe archive from shadowing @kythe_release
  • Scope workflow to //proto/...: narrow the Kythe build from //... to just proto targets, with a kytheBuildTargets() helper that also supports "default" and "full" modes for future use
  • Local Kythe runner (tools/kythe/local_kythe.sh): mirrors the CI workflow locally for faster iteration, with flags for --scope, --skip_cxx, --bazel_version, --distdir, and --dry_run

Testing: built a single proto target successfully. Building //proto/... on a tiny VM filled the disk.

🤖 Generated with Claude Code

@dan-stowell dan-stowell changed the title Move "Generate Kythe Annotations" closer to working on a smaller subset of the codebase Limit Kythe indexing to proto targets and fix Bazel 9 workflow failures Mar 3, 2026
@dan-stowell dan-stowell changed the title Limit Kythe indexing to proto targets and fix Bazel 9 workflow failures kythe: fix Bazel 9 workflow bugs, add local runner, scope to //proto/... Mar 3, 2026
@dan-stowell dan-stowell marked this pull request as ready for review March 3, 2026 17:58
@dan-stowell dan-stowell enabled auto-merge (squash) March 3, 2026 18:08
@dan-stowell dan-stowell merged commit 534afa6 into master Mar 3, 2026
13 checks passed
@dan-stowell dan-stowell deleted the bbdan-kythe-loop branch March 3, 2026 22:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants