Skip to content

ci: Debian Trixie CI image, CGO link liblzma with libunwind#287

Open
vporoshok wants to merge 2 commits intoppfrom
ci/debian-trixie-dockerfile
Open

ci: Debian Trixie CI image, CGO link liblzma with libunwind#287
vporoshok wants to merge 2 commits intoppfrom
ci/debian-trixie-dockerfile

Conversation

@vporoshok
Copy link
Copy Markdown
Collaborator

Summary

  • Dockerfile.ci: switch from Ubuntu 24.04 to Debian 13 (trixie-slim) so the devcontainer / CI image matches the Deckhouse module base (debian-trixie).
  • Toolchain: GCC 14, libunwind-dev, liblzma-dev, libstdc++-14-dev; LLVM 21 from apt.llvm.org (llvm-toolchain-trixie-21); modern apt keyring instead of apt-key.
  • pp/go/cppbridge/entrypoint.go: add -llzma / -l:liblzma.a to CGO LDFLAGS for static links — Debian’s libunwind.a pulls in lzma (minidebuginfo), same as required for module builds.
  • mcli: download uses GO_ARCH (arm64/amd64) instead of hardcoded amd64.
  • TESTING.md: note that the CI image is Debian Trixie.

Motivation

Align local/CI environment with the module image, avoid a module-only patch for lzma, and keep one set of link flags in-tree.

Verification

  • make build succeeded in ARM devcontainer after rebuild.
  • Dockerfile.ci image build verified on linux/amd64 host.

Made with Cursor

- Dockerfile.ci: debian:trixie-slim, apt.llvm.org trixie LLVM 21, gcc-14,
  libunwind-dev, liblzma-dev, libstdc++-14-dev; replace deprecated apt-key.
- entrypoint.go: -llzma / -l:liblzma.a for static links (same as Deckhouse module).
- mcli download uses GO_ARCH for arm64 builds.

Made-with: Cursor
@vporoshok vporoshok requested review from cherep58 and scaps1 April 8, 2026 21:14
GCC 14 libstdc++ uses sized-range fast paths in std::ranges::equal.
GenericBitset::size() is bit width, not iteration length (set-bit count),
so disable_sized_range restores correct element-wise comparison.

Made-with: Cursor
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