fx: 36.0.0 -> 36.0.3

[phanirithvij]

• 36.0.0 -> 36.0.3
• refactored the derivation

closed the automated pr #410250
if that is not okay feel free to close mine and reopen that one

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/)
• Nixpkgs 25.11 Release Notes (or backporting 24.11 and 25.05 Nixpkgs Release notes)
  • (Package updates) Added a release notes entry if the change is major or breaking
• NixOS 25.11 Release Notes (or backporting 24.11 and 25.05 NixOS Release notes)
  • (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.

[phanirithvij]

nixpkgs-review result

Generated using nixpkgs-review-gha

Command: nixpkgs-review pr 410458

Logs: https://github.com/phanirithvij/nixpkgs-review-gha/actions/runs/15362222088

---

x86_64-linux (sandbox = true)

✅ 1 package built:

• fx

---

aarch64-linux (sandbox = true)

✅ 1 package built:

• fx

---

x86_64-darwin (sandbox = true)

✅ 1 package built:

• fx

---

aarch64-darwin (sandbox = true)

✅ 1 package built:

• fx

[phanirithvij]

@wolfgangwalther could you review

[wolfgangwalther]

Personally, I think version update + refactor should always be split into two commits. You can find some more discussion around it here: #400934 (comment) (and elsewhere in that PR, too).

Especially in this case I don't understand why the automated PR wasn't kept for the update (Edit: ah, it's a newer patch OK) and the refactor added separately - I don't see the connection between either of the two.

But the changes LGTM, so I won't block on that.

[nixpkgs-ci]

Backport failed for release-25.05, because it was unable to cherry-pick the commit(s).

Please cherry-pick the changes locally and resolve any conflicts.

git fetch origin release-25.05
git worktree add -d .worktree/backport-410458-to-release-25.05 origin/release-25.05
cd .worktree/backport-410458-to-release-25.05
git switch --create backport-410458-to-release-25.05
git cherry-pick -x 4460656436381d1f00f43b7e3b26e3fb6b4acef0

[phanirithvij]

version update + refactor should always be split into two commits

I will follow that from now thanks