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 9 compatibility fixes #189

Closed

Conversation

utdemir
Copy link

@utdemir utdemir commented Aug 18, 2020

GHC 9 also comes with ghc-bignum library which should be preferred over integer-gmp. However since it also provides a compatibility shim for integer-gmp, so this PR doesn't change that. It would be good to address #188 at one point.

I tried to make it backwards-compatible, however; I only tested this PR on GHC HEAD, so I'm hoping for CI to test the older versions.

* Use 'Data.Kind.Type' instead of '*'. Using '*' in that position requires `StarIsType` language extension in GHC 9.
* Import `unsafeCoerce#` from `GHC.Exts`  (https://gitlab.haskell.org/ghc/ghc/-/wikis/migration/9.0#base-415)
* Relax some upper bounds.
@hvr
Copy link
Collaborator

hvr commented Aug 19, 2020 via email

@phadej
Copy link
Contributor

phadej commented Aug 23, 2020

See #188.

This is "lazy" approach, just bumping integer-gmp upper bound...

@vdukhovni
Copy link

Thanks! I'll revisit this PR once the new major GHC 9.0 release enters its final candidate release phase.

Cool. GHC 9.0.1-rc1 is now out, so perhaps this is the right time to merge this PR?

@vmchale
Copy link
Contributor

vmchale commented Feb 5, 2021

GHC 9 is out

@phadej
Copy link
Contributor

phadej commented Feb 16, 2021

I'll take a look shortly.

I'll look around at other issues as well and will make a release ASAP. Thank you for your patience.

@phadej
Copy link
Contributor

phadej commented Feb 16, 2021

I made few things differently in #195. Thanks for the base work anyway!

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.

5 participants