Skip to content

[8.5.0] Also persist facts in the hidden lockfile#27642

Merged
iancha1992 merged 1 commit intobazelbuild:release-8.5.0from
fmeum:27523-cherry
Nov 13, 2025
Merged

[8.5.0] Also persist facts in the hidden lockfile#27642
iancha1992 merged 1 commit intobazelbuild:release-8.5.0from
fmeum:27523-cherry

Conversation

@fmeum
Copy link
Copy Markdown
Collaborator

@fmeum fmeum commented Nov 12, 2025

Otherwise facts are not added to the default lockfile when an entry in the hidden lockfile for a reproducible extension is used.

Also bump the lockfile version out of caution - users may already have hidden lockfiles that lack facts.

Along the way fix the print behavior of Facts which resulted in Facts(<unknown object net.starlark.java.eval.Dict>) since it called the wrong repr override.

Closes #27522.

PiperOrigin-RevId: 828559479
Change-Id: I77f002fa0e74c23b9fab5bb05948bcbe76f8f49b
(cherry picked from commit bed7a1f)

Closes #27523

Otherwise facts are not added to the default lockfile when an entry in the hidden lockfile for a reproducible extension is used.

Also bump the lockfile version out of caution - users may already have hidden lockfiles that lack facts.

Along the way fix the `print` behavior of `Facts` which resulted in `Facts(<unknown object net.starlark.java.eval.Dict>)` since it called the wrong `repr` override.

Closes bazelbuild#27522.

PiperOrigin-RevId: 828559479
Change-Id: I77f002fa0e74c23b9fab5bb05948bcbe76f8f49b
(cherry picked from commit bed7a1f)
@fmeum fmeum requested a review from a team as a code owner November 12, 2025 10:06
@github-actions github-actions bot added team-ExternalDeps External dependency handling, remote repositiories, WORKSPACE file. awaiting-review PR is awaiting review from an assigned reviewer labels Nov 12, 2025
@iancha1992 iancha1992 enabled auto-merge November 12, 2025 16:28
@iancha1992 iancha1992 added this pull request to the merge queue Nov 13, 2025
Merged via the queue into bazelbuild:release-8.5.0 with commit ccb9cb8 Nov 13, 2025
47 checks passed
@github-actions github-actions bot removed the awaiting-review PR is awaiting review from an assigned reviewer label Nov 13, 2025
@fmeum fmeum deleted the 27523-cherry branch November 17, 2025 09:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

team-ExternalDeps External dependency handling, remote repositiories, WORKSPACE file.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants