Skip to content

zlib: disable static output on windows#490607

Merged
wolfgangwalther merged 1 commit intoNixOS:staging-nextfrom
marcin-serwin:push-lqppttqnwqnv
Feb 17, 2026
Merged

zlib: disable static output on windows#490607
wolfgangwalther merged 1 commit intoNixOS:staging-nextfrom
marcin-serwin:push-lqppttqnwqnv

Conversation

@marcin-serwin
Copy link
Copy Markdown
Contributor

@marcin-serwin marcin-serwin commented Feb 15, 2026

The install script ignores the configured libdir in case of windows. We could move it manually like it was done before #476830 but even before that we did not move libz.dll.a so I think it makes sense to just disable this split on windows.

Fixes: #476830 (comment)

Things done

  • Built on platform:
    • x86_64-linux
    • aarch64-linux (pkgsCross.ucrt64)
    • x86_64-darwin
    • aarch64-darwin
  • Tested, as applicable:
  • Ran nixpkgs-review on this PR. See nixpkgs-review usage.
  • Tested basic functionality of all binary files, usually in ./result/bin/.
  • Nixpkgs Release Notes
    • Package update: when the change is major or breaking.
  • NixOS Release Notes
    • Module addition: when adding a new NixOS module.
    • Module update: when the change is significant.
  • Fits CONTRIBUTING.md, pkgs/README.md, maintainers/README.md and other READMEs.

The install script ignores the configured libdir in case of
windows. We could move it manually like it was done before
<NixOS#476830> but even before that we
did not move `libz.dll.a` so I think it makes sense to just disable this
split on windows.

Fixes: NixOS#476830 (comment)
Signed-off-by: Marcin Serwin <marcin@serwin.dev>
@doronbehar

This comment was marked as outdated.

@nixpkgs-ci nixpkgs-ci bot added 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 0 This PR does not cause any packages to rebuild on Linux. labels Feb 15, 2026
@wolfgangwalther
Copy link
Copy Markdown
Contributor

This fixes pkgsCross.ucrt64.zlib on staging-next for me, so let's do it and then iterate on a better solution in the other PR.

@wolfgangwalther wolfgangwalther added this pull request to the merge queue Feb 17, 2026
Merged via the queue into NixOS:staging-next with commit b346aa4 Feb 17, 2026
32 of 34 checks passed
@marcin-serwin marcin-serwin deleted the push-lqppttqnwqnv branch March 3, 2026 16:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

10.rebuild-darwin: 0 This PR does not cause any packages to rebuild on Darwin. 10.rebuild-linux: 0 This PR does not cause any packages to rebuild on Linux.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants