-
Notifications
You must be signed in to change notification settings - Fork 92
Bump haskell.nix #738
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 haskell.nix #738
Conversation
Thanks @andreabedini. If this is building, this should be good to go? i.e. "ready for review". Also note, that we usually do rebase our changes on top of master (instead of merging master into our branches). |
7eb3729
to
cbe2078
Compare
Yes!
Ok, just rebased. Feel free to merge when it's green again :) |
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.
Sorry to bother you with that, but can you maybe summarize why we should bump?
"owner": "input-output-hk", | ||
"repo": "haskell.nix", | ||
"rev": "6a971848cb717d8d0c8db71bd1c1d9fd38f02851", | ||
"rev": "4883bba92116b54db92182ca24841c0aa922c9d6", |
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.
@andreabedini Is this seriously the diff of haskell.nix between these two commits? https://github.com/input-output-hk/haskell.nix/compare/6a971848cb717d8d0c8db71bd1c1d9fd38f02851..4883bba92116b54db92182ca24841c0aa922c9d6 :o
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.
Yes it is. It's ~100 of business logic change but it's right at the core of it. This makes a change in the way the cabal plan is represent in nix and, as a consequence, all materialisations have to be redone and all the compiliers and tools end up being rebuilt.
Haskell.nix had broken support for cabal file revisions in a repository other than hackage. E.g. a cabal file revision in Hackage would be picked up and work properly; while a cabal file revision in CHaP will cause haskell.nix to error out. Haskell.nix fixed that only two weeks ago. A cabal file revision appeared on CHaP today. Any project which uses a version of haskell.nix older than two week will have issues bumping CHaP, unless haskell.nix gets bumped too. I have been preventively bumping haskell.nix everywhere to find issues and warm up the caches (as you can tell it's a big change). |
cbe2078
to
822153a
Compare
The bump is to support cabal file revisions on CHaP which are necessary to get our packages to build without all those manual constraints we have in cabal.project.
I am opening a draft PR to test the waters and warm up the chaces ahead of time since it's relatively big change in haskell.nix. I'll make sure everything works and then mark it as ready.