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

haddock-api: fix build on GHC-8.0.2 #22369

Merged
merged 1 commit into from
Feb 2, 2017

Conversation

basvandijk
Copy link
Member

@peti I discovered that ghcjsHEAD doesn't build (my bad) because its dependency haddock-api doesn't build on GHC-8.0.2. This patch should fix that.

There's another problem with the shelly library which I'm investigating right now.

@mention-bot
Copy link

@basvandijk, thanks for your PR! By analyzing the history of the files in this pull request, we identified @peti, @abbradar and @pikajude to be potential reviewers.

@basvandijk
Copy link
Member Author

This should also be cherry picked on release-16.09.

@peti
Copy link
Member

peti commented Feb 2, 2017

Is it possible to download that patch from somewhere rather than checking it into our repository?

@basvandijk
Copy link
Member Author

I do have a git commit here basvandijk/haddock@f4c5e46. Is there a nix function that turns a commit into a patch?

If so, we still have to patch the cabal file but we can also just jailbreak it.

@basvandijk
Copy link
Member Author

I see there's a fetchpatch function and that you can turn a GitHub commit into a patch by appending ".patch" to the commit URL. Let me see if I can get that to work.

@basvandijk
Copy link
Member Author

@peti I forced pushed a new commit with your recommendation.

@peti
Copy link
Member

peti commented Feb 2, 2017

Thank you very much!

@peti peti merged commit 072ebac into NixOS:master Feb 2, 2017
peti added a commit that referenced this pull request Feb 2, 2017
haddock-api: fix build on GHC-8.0.2
(cherry picked from commit 072ebac)
@dmjio dmjio mentioned this pull request Aug 7, 2017
adrianpk added a commit to adrianpk/nixpkgs that referenced this pull request May 31, 2024
haddock-api: fix build on GHC-8.0.2
(cherry picked from commit 072ebac)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants