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

fix for GHC 9.10 #139

Merged
merged 3 commits into from
May 15, 2024
Merged

fix for GHC 9.10 #139

merged 3 commits into from
May 15, 2024

Conversation

guibou
Copy link
Owner

@guibou guibou commented May 15, 2024

  • New stack builder for 9.8
  • nixpkgs bump
  • Disabled the "buildFromSdist". I don't remember why this was there and looks like it adds some dependencies to the build.
  • Removed nix ghc 8.8 support, not in nixpkgs anymore. This is still tested by stack.
  • Added builder pyf_910, works when using nix develop .#pyf_910, but build is broken.
  • Fixed the CPP code for GHC 9.8. I wrongly tested for GHC 9.8 instead of GHC 9.10 in previous support commit for 9.10, resulting in broken 9.8. This is fixed and 9.8 was tested again.

guibou added 3 commits May 15, 2024 20:31
- New stack builder for 9.8
- nixpkgs bump
- Disabled the "buildFromSdist". I don't remember why this was there and
  looks like it adds some dependencies to the build.
- Removed nix ghc 8.8 support, not in nixpkgs anymore. This is still
  tested by stack.
- Added builder `pyf_910`, works when using `nix develop .#pyf_910`, but
  build is broken.
- Fixed the CPP code for GHC 9.8. I wrongly tested for GHC 9.8 instead
  of GHC 9.10 in previous support commit for 9.10, resulting in broken
  9.8. This is fixed and 9.8 was tested again.
Instead of running one job building ALL GHC version, we use a matrix of
job.

That's more duplication (compared to just follows what's in flake.nix),
but should workaround the limitation.
@guibou guibou merged commit 5ff10be into main May 15, 2024
15 checks passed
@guibou guibou deleted the clean_910 branch May 15, 2024 17:28
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.

1 participant