-
Notifications
You must be signed in to change notification settings - Fork 61
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add compatibility for GHC 9.0 and 9.2 #300
Conversation
It would be great make haskell gha download the ghc-9.2 prerelase to test it in ci (maybe it is possible? ghcup can install the ghc prerelase and gha uses ghcup under the hood) |
Related: haskell/actions#55 I would like to avoid implementing custom support, so I would wait for this feature to be implemented. |
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.
LGTM, thanks for this double support
@fendor could we get a hackage release with 9.0 support? thanks! |
#if __GLASGOW_HASKELL__ >= 902 | ||
-- Copied from GHC, do we need that? | ||
addWay' :: Way -> DynFlags -> DynFlags | ||
addWay' w dflags0 = | ||
let platform = targetPlatform dflags0 | ||
dflags1 = dflags0 { targetWays_ = Platform.addWay w (targetWays_ dflags0) } | ||
dflags2 = foldr setGeneralFlag' dflags1 | ||
(Platform.wayGeneralFlags platform w) | ||
dflags3 = foldr unSetGeneralFlag' dflags2 | ||
(Platform.wayUnsetGeneralFlags platform w) | ||
in dflags3 | ||
#endif |
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.
IIRC, we don't need this in later GHC versions, as we can use GHC.Driver.Session.setDynamicNow
directly.
* Make hie-bios compile with GHC-HEAD * Adapt hie-bios to compile with GHC HEAD * Add allow-newer to get a build plan * Bump up ghc upper bound * Update ghc head to 9.0 * Use ghc-api-compat head * Adjust some defs to 9.0.1 * Add ghc-api-compat to exe component * Remove updated libraries * Fix some definitions * Add compatibility for GHC 9.2 Co-authored-by: jneira <atreyu.bbb@gmail.com> (cherry picked from commit 3703788)
* Make hie-bios compile with GHC-HEAD * Adapt hie-bios to compile with GHC HEAD * Add allow-newer to get a build plan * Bump up ghc upper bound * Update ghc head to 9.0 * Use ghc-api-compat head * Adjust some defs to 9.0.1 * Add ghc-api-compat to exe component * Remove updated libraries * Fix some definitions * Add compatibility for GHC 9.2 Co-authored-by: jneira <atreyu.bbb@gmail.com> (cherry picked from commit 3703788)
Avoids dependency on
ghc-api-compat
so thathie-bios
can actually be uploaded to Hackage.Additionally, moves
parseTargetFiles
toGap.hs
since GHC 9.2 exposes the function we want 馃帄.supersedes #285