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

Relax bounds so compdata builds with GHC-shipped core libraries #40

Merged
merged 2 commits into from
Oct 20, 2023

Conversation

andreasabel
Copy link
Contributor

@andreasabel andreasabel commented Oct 20, 2023

This PR fixes the issue underlying

The current bounds on dependencies containers, mtl, and transformers are too strict as they do not allow building with the GHC-shipped versions of these libraries. This is a bit problematic for two reasons:

  • in a project using compdata, other dependencies may have to be rebuild as the default versions of the ubiquitous containers, mtl, and transformers can't be used
  • it does not fit the Stackage model which fixes the ghc core libraries in the snapshot

This PR relaxes the bounds so as compdata is buildable with the pre-installed versions of the libraries of GHC 9.2 - 9.8.
Also, there are revisions to the CI, mainly:

  • enforce building with the GHC-shipped libraries by adding constraints: mtl installed etc to the cabal.project.local file
  • reform the cache key to use precise ghc and cabal versions and the build-plan generated by cabal build --dry-run

The changes in the PR can be published as a Hackage revision, no new version of compdata needed.

Commits:

The aim seems to support GHC 9.2 - 9.8, so the lower bounds of the dependencies should allow to build with the libraries shipped with GHC 9.2:

- `containers-0.6.5`
- `mtl-2.2.2`
- `transformers-0.5.6`
Further:
- bump `actions/checkout`
- use `haskell-actions` instead of the deprecated `haskell/actions`
- use `plan.json` as cache key rather than the freeze file
- use full version numbers of GHC and Cabal in cache key
- single out building of dependencies in extra step
@andreasabel andreasabel changed the title loosen bounds Relax bounds so compdata builds with GHC-shipped core libraries Oct 20, 2023
@andreasabel
Copy link
Contributor Author

ATTN: @pa-ba

@pa-ba pa-ba merged commit e916a9a into pa-ba:master Oct 20, 2023
4 checks passed
@pa-ba
Copy link
Owner

pa-ba commented Oct 20, 2023

Thanks @andreasabel !

@andreasabel
Copy link
Contributor Author

Welcome!
compdata should be back in the next stackage nightly.

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.

2 participants