-
Notifications
You must be signed in to change notification settings - Fork 5
Lower cabal-version back to 1.18 #48
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
Merged
Merged
Changes from all commits
Commits
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
There is little point of going all the way to 1.18, clients with Cabal < 2.0 are not served by modern Hackage server. So even very conservatively you can use 2.0, but practically I'd put 3.10, which is the version of Cabal at Hackage build machine (https://github.com/haskell-infra/hackage-doc-builder-config/blob/master/flake.lock).
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.
Sorry I've been jumping the gun. I will continue making minor version changes.
Uh oh!
There was an error while loading. Please reload this page.
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.
Perhaps we can keep it like this (changes are already merged), and make a bump to 3.14 (so that the originally proposed
extra-filesfield can be used) once that is supported.BTW, is the spec version used by the Hackage server the effective upper-bound for supported versions?
Also, it is not clear to me what reasons could we have to be conservative about this version number. Wouldn't some consumers be affected by bumping to 3.10? Are there existing documentation on best practices?
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.
Hackage server itself at the moment uses Cabal-the-library 3.16 (you can see it in the footer of any package page such as https://hackage.haskell.org/package/Diff), so the original 3.14 was absolutely fine. If it was too new, Hackage would simply reject the upload as unparseable. It's the documentation build server which lags behind at the moment and still uses Cabal-the-executable 3.10. So to be honest I would not revert anything in the first place, just uploaded the documentation manually and leave it be; at some point the doc builder will be upgraded.
If consumers are using Cabal-the-executable 3.8 (which is 5 major versions behind the last release of 3.18), I imagine they are not exactly looking to upgrade
Diffto the latest and greatest.No active harm in having
cabal-version: 1.18, so I would not do another minor release for now.