Skip to content

haskellPackages.posix-waitpid: unbreak#478857

Merged
wolfgangwalther merged 1 commit intoNixOS:haskell-updatesfrom
danielrolls:posix-waitpid-unbreak
Jan 12, 2026
Merged

haskellPackages.posix-waitpid: unbreak#478857
wolfgangwalther merged 1 commit intoNixOS:haskell-updatesfrom
danielrolls:posix-waitpid-unbreak

Conversation

@danielrolls
Copy link
Contributor

Fix CPid constructor import (needs CPid(..) for pattern matching on foreign calls) and jailbreak for version bounds.

Package has been broken since GHC 7.4+ (2011) and marked broken in nixpkgs since 2016.

Things done

  • Built on platform:
    • x86_64-linux
    • aarch64-linux
    • 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.

Add a 👍 reaction to pull requests you find important.

@danielrolls danielrolls added the 6.topic: haskell General-purpose, statically typed, purely functional programming language label Jan 10, 2026
@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 Jan 10, 2026
@danielrolls danielrolls marked this pull request as ready for review January 11, 2026 09:45
@danielrolls
Copy link
Contributor Author

@wolfgangwalther, the original Hackage package has no repo and hasn't been updated since 2010. However, there's a fork at https://github.com/GaloisInc/posix-waitpid which already has this fix.

We could switch to the fork if preferred.

@wolfgangwalther
Copy link
Contributor

I'd be fine with the current approach, if we were to add some of that explanation to the code comment, too.

@danielrolls danielrolls force-pushed the posix-waitpid-unbreak branch from c9d58ac to 8fa7044 Compare January 11, 2026 20:46
Fix CPid constructor import (needs CPid(..) for pattern matching on
foreign calls) and jailbreak for version bounds.

Package has been broken since GHC 7.4+ (2011) and marked broken in
nixpkgs since 2016.
@danielrolls danielrolls force-pushed the posix-waitpid-unbreak branch from 8fa7044 to 3d9b17a Compare January 11, 2026 21:05
@danielrolls
Copy link
Contributor Author

@wolfgangwalther, I had a go a both. How's this?

@nixpkgs-ci nixpkgs-ci bot added 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. and removed 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 Jan 11, 2026
@wolfgangwalther
Copy link
Contributor

LGTM; I am doing a build, but this will take a while - still need to build GHC on haskell-updates right now. Will look at this tomorrow again.

@wolfgangwalther
Copy link
Contributor

nixpkgs-review result

Generated using nixpkgs-review.

Command: nixpkgs-review pr 478857
Commit: 3d9b17a5d7cddc6a63753678b0c671a783b246cb


x86_64-linux

✅ 3 packages built:
  • haskell.package-list
  • haskellPackages.posix-waitpid
  • haskellPackages.posix-waitpid.doc

@wolfgangwalther wolfgangwalther added this pull request to the merge queue Jan 12, 2026
Merged via the queue into NixOS:haskell-updates with commit 1b4d91b Jan 12, 2026
30 of 32 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

6.topic: haskell General-purpose, statically typed, purely functional programming language 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants