haskellPackages: remove unneeded jailbreaks

[alexfmpe]

New year, new automated sweep.

Tested with

nix-build -E 'with (import ./. {}).haskell.packages.ghc910; [ arbtt arch-web cabal-cargs chimera commonmark-wikilink data-array-byte decimal-literals dual-game fused-effects granite hashable heist-extra hs-speedscope http-client-tls kind-generics-th large-generics lattices matrix-client morpheus-graphql morpheus-graphql-app morpheus-graphql-client morpheus-graphql-core morpheus-graphql-server morpheus-graphql-tests nothunks patat puresat snappy-hs tasty-checklist tasty-sugar unionmount vector wild-bind ]'

9.8: Only cabal-cargs fails to build, with a haskell source error.
9.12: They all build per-se, but patat & commonmark-wikilin are blocked on test failures for deps pandoc & unicode-data.
9.14: Didn't bother with it as it's very recent and lots of things don't build with it

Things done

• Built on platform:
  • x86_64-linux
  • aarch64-linux
  • x86_64-darwin
  • aarch64-darwin
• Tested, as applicable:
  • NixOS tests in nixos/tests.
  • Package tests at passthru.tests.
  • Tests in lib/tests or pkgs/test for functions and "core" functionality.
• 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.

[maralorn]

Already merge-conflict. 😬

[alexfmpe]

Rebased

[maralorn]

I haven’t yet succeeded in running nixpkgs-review on this. Building the review-shell needs 30k downloads and 12k builds. Am I holding it wrong?

[alexfmpe]

Do we want to run nixpkgs-review here? Removing these jailbreaks technically triggers rebuild of all packages depending on any of these, but jailbreak can only affect whether the targeted package builds or not right?

[maralorn]

Yeah, probably unnecessary. It just was a way to give this due diligence.

[maralorn]

Even to build your example nix-build command I need to compile ghc. Whatever, I trust your testing.

The change looks fine and if this really breaks anything maintained we will notice.

[sternenseemann]

It would be better to merge these kinds of changes directly into staging in the future when we have a fully built haskell-updates branch already.