Skip to content

Hash local BUILD overlays for http_archive - #634

Merged
tinder-maxwellelliott merged 1 commit into
Tinder:masterfrom
tamird:codex/bazel-diff-local-archive-overlay-20261106
Nov 7, 2026
Merged

tinder-maxwellelliott merged 1 commit into
Tinder:masterfrom
tamird:codex/bazel-diff-local-archive-overlay-20261106

Conversation

@tamird

@tamird tamird commented Nov 6, 2026

Copy link
Copy Markdown
Contributor

An http_archive can read build_file from a local_repository while the archive checksum stays unchanged. Repository input hashing follows files in the main workspace and misses this external overlay, so an edit can change a built output without selecting the archive's direct readers.

Hash the named regular file when the evaluated source records agree on a native or standard local_repository path. Require the standard http_archive origin and a LABEL build_file. Invalidate cached graphs and partial recipes recorded with the old digest.

The regression changes two overlays of the same archive independently and checks each direct reader. It also verifies that a tracked overlay edit in the workspace rebuilds a partial graph to match fresh generation.

Assisted by Codex.

An http_archive can read build_file from a local_repository while the
archive checksum stays unchanged. Repository input hashing follows files
in the main workspace and misses this external overlay, so an edit can
change a built output without selecting the archive's direct readers.

Hash the named regular file when the evaluated source records agree on a
native or standard local_repository path. Require the standard
http_archive origin and a LABEL build_file. Invalidate cached graphs and
partial recipes recorded with the old digest.

The regression changes two overlays of the same archive independently
and checks each direct reader. It also verifies that a tracked overlay
edit in the workspace rebuilds a partial graph to match fresh
generation.

Assisted by Codex.
@tinder-maxwellelliott
tinder-maxwellelliott merged commit 8594ab7 into Tinder:master Nov 7, 2026
37 of 38 checks passed
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