Skip to content

GHC 9.8 support#3727

Merged
mergify[bot] merged 3 commits into
masterfrom
wip/ghc-9.8
Oct 10, 2023
Merged

GHC 9.8 support#3727
mergify[bot] merged 3 commits into
masterfrom
wip/ghc-9.8

Conversation

@wz1000

@wz1000 wz1000 commented Jul 24, 2023

Copy link
Copy Markdown
Collaborator

Requires head.hackage to build

Support for ghcide + non-exactprint plugins.

@wz1000

wz1000 commented Oct 5, 2023

Copy link
Copy Markdown
Collaborator Author

All non exactprint plugins work and tests pass locally. I've added a CI job to test against 9.8-rc1.

@wz1000
wz1000 force-pushed the wip/ghc-9.8 branch 6 times, most recently from 366ecb1 to 6aa2a66 Compare October 6, 2023 08:37

@michaelpj michaelpj left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks reasonable. Maybe @joyfulmantis as the CPP sherrif wants to take a look at the CPP.

Comment thread cabal.project
deepseq,

if impl(ghc >= 9.7)
repository head.hackage.ghc.haskell.org

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I guess we can't really avoid this at the moment? I'm unsure how I feel about the prospect of us e.g. releasing binaries based on using packages from head.hackage 🤔 maybe it's fine

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No. I could use git dependencies instead by pushing head.hackage patches to forks, but I don't think it earns us much.

Comment thread cabal.project
26021a13b401500c8eb2761ca95c61f2d625bfef951b939a8124ed12ecf07329
7541f32a4ccca4f97aea3b22f5e593ba2c0267546016b992dfadcd2fe944e55d

active-repositories: hackage.haskell.org, head.hackage.ghc.haskell.org

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think you don't need this

Comment thread ghcide/src/Development/IDE/Core/Compile.hs
Comment thread ghcide/src/Development/IDE/Core/Compile.hs
Comment thread ghcide/src/Development/IDE/Core/Compile.hs Outdated
Comment thread ghcide/src/Development/IDE/GHC/Compat/Core.hs
Comment thread ghcide/test/exe/CompletionTests.hs
Comment thread plugins/hls-alternate-number-format-plugin/src/Ide/Plugin/Literals.hs Outdated
Comment thread plugins/hls-class-plugin/hls-class-plugin.cabal
@michaelpj

Copy link
Copy Markdown
Collaborator

Incidentally, can we delete the allow-newers for older GHC versions now?

@fendor fendor left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, modulo the comments found by michael.

@wz1000
wz1000 force-pushed the wip/ghc-9.8 branch 2 times, most recently from 6d3635c to 6d22882 Compare October 9, 2023 11:49
@wz1000
wz1000 force-pushed the wip/ghc-9.8 branch 3 times, most recently from 2af067b to 10549ff Compare October 10, 2023 04:03
@wz1000 wz1000 added the merge me Label to trigger pull request merge label Oct 10, 2023
@wz1000 wz1000 changed the title WIP: GHC 9.8 support GHC 9.8 support Oct 10, 2023
@mergify
mergify Bot merged commit 5580ffc into master Oct 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

merge me Label to trigger pull request merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants