Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ghc: apply fix to Haddock in relation to typechecker plugins #59317

Closed

Conversation

Taneb
Copy link
Contributor

@Taneb Taneb commented Apr 11, 2019

Motivation for this change

The version of Haddock bundled with GHC 8.6 fails with typechecker modules. This applies a patch which fixes that.

This is the main issue preventing me from moving away from nixpkgs 18.09 and GHC 8.4 at work.

Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
  • Built on platform(s)
    • NixOS
    • macOS
    • other Linux distributions
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Determined the impact on package closure size (by running nix path-info -S before and after)
  • Assured whether relevant documentation is up to date
  • Fits CONTRIBUTING.md.

peti and others added 6 commits April 11, 2019 10:42
This update was generated by hackage2nix v2.14.2-5-g11c68fd from Hackage revision
commercialhaskell/all-cabal-hashes@3d5bbe3.
This commit disables the library-for-ghci flag passed to
`Setup configure` in the Haskell generic-builder.nix file.

This stops the HSfoo.o file from being built.  Building this
HSfoo.o file caused doctest to take an extremely long time
to load dependencies when running.

This is a follow-up from NixOS#58743.
@peti
Copy link
Member

peti commented Apr 11, 2019

Merged in 2de3d6a.

@peti peti closed this Apr 11, 2019
peti pushed a commit that referenced this pull request Apr 12, 2019
peti pushed a commit that referenced this pull request Apr 13, 2019
Taneb pushed a commit to Taneb/nixpkgs that referenced this pull request Jun 11, 2019
Closes NixOS#59317.

(cherry picked from commit a184604)

Reason: this should be strictly a bug fix, to allow using Haddock and
doctest with type-checker plugins.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants