-
Notifications
You must be signed in to change notification settings - Fork 710
Move process >= 1.16.4
constraint from Cabal-3.8.1.0
to cabal-install-3.8.1.0
#8802
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
Conversation
CI does not seem to run here. |
The |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Thank you.
, Cabal ^>=3.8.1.0 | ||
, Cabal-syntax ^>=3.8.1.0 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We actually keep these, and everywhere else, at X.X.0.0 to avoid having to update at each minor release.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Okay!
zlib >= 0.5.3 && < 0.7, | ||
hackage-security >= 0.6.2.0 && < 0.7, | ||
text >= 1.2.3 && < 1.3, | ||
text >= 1.2.3 && < 1.3 || >=2.0 && <2.1, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't remember --- was this tested with 3.8 (as opposed to with master)?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I made this revision 2022-08-30. I strongly suspect that I did test it before revising.
But even if I hadn't, it has now passed the test of time. ;-)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hah, it did. But if so, I bet we tested it on 3.8 (or master before 3.8 has been cut).
Resolves #8554.
This PR is not to be merged, just for testing and discussion, as agreed in the Cabal developers meeting of 2023-02-23.
The plan is to revise
Cabal-3.8.1.0
andcabal-install-3.8.1.0
according to this PR, namely the last 2 commits:The first commit just capture revisions already present on hackage. The second commit makes changes so that this PR builds.
I checked builds locally:
Cabal
withmtl-2.3.1
(for 2beee0b)cabal-install
withunix-2.8.0.0
(for aa60929)