-
-
Notifications
You must be signed in to change notification settings - Fork 370
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
bump lock, downgrade fourmolu to 0.11 #3797
Conversation
Sorry, I don't understand. Why would we downgrade fourmolu? |
@cab404 fourmolu 0.14 most definitely works for GHC 9.2+ 🙂 What exactly is the resolver issue? It's probably other dependencies that need to get bumped, as opposed to downgrading fourmolu |
If you try building it as is:
|
After patching in fourmolu 0.13 or 0.14:
|
And, finally, after patching in Cabal-syntax and ghc-lib-parser:
|
@michaelpj @brandonchinn178 Sorry for keeping you waiting for an answer~ |
After you try overriding
After all that you slam into the following:
|
And after you actually read the reason for halting (abort because of serious configure-time warning from Cabal), you go a bit higher in the log, and discover that your dependency tree became bodged enough to fail to please Cabal.
|
So you go ahead and override Cabal as well, hoping that nothing in API changed too much to break with all those repinnings — and perform another |
And only after that, and praying, you get a working HLS binary (and then you just need to go and fix it for other 3 versions). In other words, downgrading it to 0.11 was a quicker way to go on with my workday after encountering a wild 9.4.5→6 bump — although I should probably had kept it in a draft |
The Nix build is on a best-effort basis (for reasons like this), and frankly I'm thinking of getting rid of it because it's always broken and we really can't promise to keep it working. I certainly don't think we should force a downgrade on ourselves. The build plan with |
But 9.4 dourmolu plugin requested 0.13 in cabal? Maybe I didn't rebase since it got updated? |
I think this isn't relevant any more since #3804 has been merged. |
Would be glad on some help getting nix packages building again. This fixes build for 9.4.6 (and actually bumps nixpkgs to include 9.4.6), alas that required downgrading fourmolu — without that I cannot resolve dependencies :(