Skip to content

Drop GHC support for <9.6.7 #534

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

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

Conversation

aniketd
Copy link
Contributor

@aniketd aniketd commented May 16, 2025

Description

Closes #530

Checklist

  • Commit sequence broadly makes sense and commits have useful messages
  • New tests are added if needed and existing tests are updated
  • All visible changes are prepended to the latest section of a CHANGELOG.md for the affected packages.
    New section is never added with the code changes. (See RELEASING.md)
  • When applicable, versions are updated in .cabal and CHANGELOG.md files according to the
    versioning process.
  • The version bounds in .cabal files for all affected packages are updated.
    If you change the bounds in a cabal file, that package itself must have a version increase. (See RELEASING.md)
  • Self-reviewed the diff

@aniketd aniketd force-pushed the aniketd/drop-support-upto-ghc967 branch from 2f11df5 to 629a312 Compare May 16, 2025 14:44
@aniketd aniketd changed the title Update flake.nix and GHA Drop GHC upto 9.6.7 May 16, 2025
@aniketd aniketd changed the title Drop GHC upto 9.6.7 Drop GHC support upto 9.6.7 May 16, 2025
@aniketd aniketd changed the title Drop GHC support upto 9.6.7 Drop GHC support for <9.6.7 May 16, 2025
@aniketd aniketd force-pushed the aniketd/drop-support-upto-ghc967 branch from 629a312 to 017b916 Compare May 20, 2025 06:01
@aniketd aniketd marked this pull request as ready for review May 20, 2025 06:01
@aniketd aniketd requested review from erikd, lehins, nfrisby and a team as code owners May 20, 2025 06:01
index-state: cardano-haskell-packages 2025-05-14T07:44:24Z
index-state: cardano-haskell-packages 2025-04-08T17:22:48Z
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This change was necessary in order to be able to reuse the flake inputs that do not cause a problem on MacOS.

Copy link
Collaborator

Choose a reason for hiding this comment

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

You should not be downgrading index-state

@aniketd aniketd force-pushed the aniketd/drop-support-upto-ghc967 branch from 017b916 to 0b06f22 Compare May 20, 2025 06:39
@aniketd aniketd force-pushed the aniketd/drop-support-upto-ghc967 branch from 0b06f22 to f3a4da3 Compare May 20, 2025 08:08
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.

@aniketd I recommend you upgrade to latest haskellNix and if it gives you issues then reach out to someone on #dx-pub

Hacking around it by using an older state only delays the problem.

index-state: cardano-haskell-packages 2025-05-14T07:44:24Z
index-state: cardano-haskell-packages 2025-04-08T17:22:48Z
Copy link
Collaborator

Choose a reason for hiding this comment

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

You should not be downgrading index-state

@@ -207,7 +201,7 @@
};
in
mkDevShells cabalProject
# Additional shells for every GHC version supported by haskell.nix, eg. `nix develop .#ghc8107`
# Additional shells for every GHC version supported by haskell.nix, eg. `nix develop .#ghc8122`
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
# Additional shells for every GHC version supported by haskell.nix, eg. `nix develop .#ghc8122`
# Additional shells for every GHC version supported by haskell.nix, eg. `nix develop .#ghc9122`

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.

Drop GHC-8.10 - GHC-9.4 support
3 participants