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

Update deps for ghc 9.2/GHC 9.4 #40

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

Conversation

csasarak
Copy link

This PR makes codec-rpm compile with both GHC 9.2 and GHC 9.4. Some other changes:

  1. Newer versions of Conduit warn against using Coduit over ConduitT. I've edited the code to turn off those warnings. ConduitT is available since 2015's 0.2.0 which wouldn't compile with the current version of codec-rpm anyways because of base version constraints.
  2. The <> function was made a class method of Semigroup in base > 4.11 and on newer version of base emits a warning about that if you try to import it via Data.Monoid. I've added a preprocessor directive to import it from the right place based on base's version.
  3. I added a cabal.project. The newer versions of base16-bytestring are incompatible with the versions of cpio-conduit available on hackage so I pull in the new version via a git sha.

The motiviation for this is that newer versions of the compiler use a version of
bytestring than this package and its deps allow.

Fixup

fixup
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