nginx: fix compatibility with zlib-ng

[fpletz]

Fix #357522.

Things done

• Built on platform(s)
  • x86_64-linux
  • aarch64-linux
  • x86_64-darwin
  • aarch64-darwin
• For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)
  • sandbox = relaxed
  • sandbox = true
• Tested, as applicable:
  • NixOS test(s) (look inside nixos/tests)
  • and/or package tests
  • or, for functions and "core" functionality, tests in lib/tests or pkgs/test
  • made sure NixOS tests are linked to the relevant packages
• Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
• Tested basic functionality of all binary files (usually in ./result/bin/)
• 25.05 Release Notes (or backporting 24.11 and 25.05 Release notes)
  • (Package updates) Added a release notes entry if the change is major or breaking
  • (Module updates) Added a release notes entry if the change is significant
  • (Module addition) Added a release notes entry if adding a new NixOS module
• Fits CONTRIBUTING.md.

---

Add a 👍 reaction to pull requests you find important.

[Izorkin]

Small fix:

diff --git a/pkgs/servers/http/nginx/generic.nix b/pkgs/servers/http/nginx/generic.nix
index 12c3e2ed6302..c576031da8ea 100644
--- a/pkgs/servers/http/nginx/generic.nix
+++ b/pkgs/servers/http/nginx/generic.nix
@@ -156,7 +156,7 @@ stdenv.mkDerivation {
       '';
     })
     ./nix-skip-check-logs-path.patch
-  ] ++ lib.optionals (lib.elem pname ["nginx" "angie"]) [
+  ] ++ lib.optionals (lib.elem pname [ "angie" "angieQuic" "nginx" "nginxQuic" ]) [
     # https://github.com/NixOS/nixpkgs/issues/357522
     # https://github.com/zlib-ng/patches/blob/5a036c0a00120c75ee573b27f4f44ade80d82ff2/nginx/README.md
     (fetchpatch {

[fpletz]

Thanks for the hint! I think I'll just reverse the pname check because the only incompatible variant is actually openresty.

[Izorkin]

There's also tengine package.

[fpletz]

There's also tengine package.

tengine doesn't actually use the generic nginx expression and depends on regular zlib anyway. 😏

[Izorkin]

I was thinking of redesigning it to use generic nginx.

[fpletz]

Pushed another small fix for Darwin.

[fpletz]

ofborg stuck again. 😿

[github-actions]

Successfully created backport PR for release-24.11:

• [Backport release-24.11] nginx: fix compatibility with zlib-ng #359579