Skip to content
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

revert haskell.nix update #503

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

revert haskell.nix update #503

wants to merge 8 commits into from

Conversation

angerman
Copy link
Collaborator

This hopefully unbreaks the CI issues we see.

@angerman angerman marked this pull request as ready for review October 17, 2024 07:47
@angerman angerman requested review from lehins and a team as code owners October 17, 2024 07:47
@angerman
Copy link
Collaborator Author

@lehins this is as green as we can get it right now, I believe. There are only some 9.8.2 windows cross build failures, everything else is green.

Copy link
Collaborator

@lehins lehins left a comment

Choose a reason for hiding this comment

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

Looking at the changes in the PR makes me believe that something is broken on haskellNix and we are waiting for it to be fixed. Only after it is fixed then we restore using newer ghc and cabal. Is that correct?
If my understanding is correct, could you please link to a github issue that tracks this, so we know when we can get rid of the git pin for haskellNix.

Also, could I ask you to cleanup the git history in this PR a little bit. If you won't get it to it, not the end of the world, I'll squash all them together when merging.

@@ -48,7 +56,7 @@
inherit (nixpkgs) lib;

# see flake `variants` below for alternative compilers
defaultCompiler = "ghc966";
defaultCompiler = "ghc965";
Copy link
Collaborator

Choose a reason for hiding this comment

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

Why is the downgrade?

@@ -84,7 +92,7 @@
# tools we want in our shell, from hackage
tools =
{
cabal = "3.12.1.0";
cabal = "3.10.3.0";
Copy link
Collaborator

Choose a reason for hiding this comment

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

Same here, how come we need to downgrade cabal?

-- Windows cross with bitvec/simd is broken.
package bitvec
-- Workaround for windows cross-compilation
flags: -simd
Copy link
Collaborator

Choose a reason for hiding this comment

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

Do you know if this is a valid syntax for cabal.project? Never tried it before, but if it works it would be much better to turn it off only on platforms that are problematic. The concern I have is that if we do use simd on linux and mac for the actual cardano-node then all of the tests in this repo are not tested with the same flags

Suggested change
flags: -simd
if os(windows)
flags: -simd

Copy link
Collaborator

@lehins lehins left a comment

Choose a reason for hiding this comment

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

Looks like there are still failures on hydra. We do need to fix all of them

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