Skip to content

Link statically - #11

Merged
dmeijboom merged 2 commits into
mainfrom
feat/link-statically
Aug 17, 2026
Merged

Link statically#11
dmeijboom merged 2 commits into
mainfrom
feat/link-statically

Conversation

@dmeijboom

Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the Nix flake to produce statically linked Linux builds (via musl) and to validate that the resulting binaries do not depend on dynamic loaders/libraries, while keeping the existing macOS “no Nix store dylib deps” safeguard.

Changes:

  • Adds per-system Rust target selection (musl targets for Linux) and configures the Rust toolchain to include those targets.
  • Switches Linux package builds to cross-configure nixpkgs for musl and sets Cargo target/rustflags accordingly.
  • Adds Linux post-fixup checks (via readelf) to fail the build if the resulting binary is dynamically linked.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@dmeijboom
dmeijboom force-pushed the feat/link-statically branch 2 times, most recently from 1c5d2ab to ce369bf Compare August 17, 2026 13:41
@dmeijboom
dmeijboom force-pushed the feat/link-statically branch from ce369bf to 5627fe0 Compare August 17, 2026 14:34
@dmeijboom
dmeijboom merged commit f3b2345 into main Aug 17, 2026
1 check passed
@dmeijboom
dmeijboom deleted the feat/link-statically branch August 17, 2026 17:03
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