Skip to content

Remove buildinfo package#34

Merged
shykes merged 1 commit into
mainfrom
remove-buildinfo-package
Jul 20, 2026
Merged

Remove buildinfo package#34
shykes merged 1 commit into
mainfrom
remove-buildinfo-package

Conversation

@shykes

@shykes shykes commented Jul 18, 2026

Copy link
Copy Markdown
Contributor

buildinfo wrapped runtime/debug.ReadBuildInfo so sandboxed builds — which have no .git for the Go toolchain's native VCS stamping to read — could inject commit/dirty/time metadata via -ldflags. It lived here as a standalone module to serve as a stable -ldflags -X injection point for the dagger repo and any other consumers.

That rationale no longer holds: the dagger repo has inlined its own copy (dagger/dagger#13508), and the injection approach is being superseded by stamping directly into native build info. With no remaining consumers, keeping it in a separate repo is pure maintenance surface — so this removes it.

No longer needed: the dagger repo inlined its own copy
(dagger/dagger#13508) and is moving to stamp VCS metadata into
native build info directly. No consumers remain.
@shykes
shykes force-pushed the remove-buildinfo-package branch from 0f1460c to a321d70 Compare July 18, 2026 18:54
@shykes
shykes merged commit d1c493d into main Jul 20, 2026
10 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.

1 participant