Skip to content

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

Merged
merged 1 commit into from
Feb 23, 2023
Merged

Conversation

andreabedini
Copy link
Contributor

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.

@ch1bo
Copy link
Member

ch1bo commented Feb 22, 2023

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).

@andreabedini andreabedini marked this pull request as ready for review February 22, 2023 09:38
@andreabedini
Copy link
Contributor Author

Thanks @andreabedini. If this is building, this should be good to go? i.e. "ready for review".

Yes!

Also note, that we usually do rebase our changes on top of master (instead of merging master into our branches).

Ok, just rebased. Feel free to merge when it's green again :)

Copy link
Member

@ch1bo ch1bo left a 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",
Copy link
Member

Choose a reason for hiding this comment

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

Copy link
Contributor Author

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.

@andreabedini
Copy link
Contributor Author

Sorry to bother you with that, but can you maybe summarize why we should bump?
@ch1bo I should apologies, I am failing at communication here 😂

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).

@pgrange pgrange merged commit 8f3e97c into cardano-scaling:master Feb 23, 2023
@ch1bo ch1bo mentioned this pull request Feb 23, 2023
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.

3 participants