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
More GHC8.8.1 fixes #68935
More GHC8.8.1 fixes #68935
Conversation
…t_1_0_7" This reverts commit 5e91bfa.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Aside from the one thing with alex
, this looks good to me.
Thanks for fixing the broken dhall-related stuff as well!
@luc-tielen Also, could you edit the message for this PR (this) and put in the text "Fix #68879" somewhere so that #68879 is automatically closed when this PR is merged in? |
pkgs/development/haskell-modules/patches/alex-3.2.4-ghc8.8.x.patch
Outdated
Show resolved
Hide resolved
@alexarice Good catch with the |
@GrahamcOfBorg eval |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@luc-tielen Thanks for working on this.
I tried compiling all the packages touched in this PR with the following command:
$ nix-build -E 'with (import ./. {}); haskell.packages.ghc881.ghcWithPackages (p: with p; [cabal-doctest yaml alex haskell-src-meta happy haskell-src-exts vector-th-unbox socks attoparsec vault tasty-hedgehog])'
Everything compiled successfully. I also confirmed that ats-pkg
could be built.
@peti This should be ready to merge.
Thanks! 😃 |
Motivation for this change
I couldn't get one of my haskell projects to work with ghc881 with hoogle, so I tried fixing what I came across.
NOTE: I have never pushed to this repo before and I still consider myself to be fairly new at using nix. So feel free to point out if I did some silly stuff. 😄
Fix #68879
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)Notify maintainers
cc @