10000 hard drop GHC 8.4 by andreasabel · Pull Request #6917 · agda/agda · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

hard drop GHC 8.4 #6917

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 3 commits into from
Oct 13, 2023
Merged

hard drop GHC 8.4 #6917

merged 3 commits into from
Oct 13, 2023

Conversation

andreasabel
Copy link
Member
  • Remove #ifs for GHC < 8.6
  • Remove #ifs for versions of dependencies we no longer support.
  • More precise lower bound for vector-hashtables

These conditionals are no longer needed since we dropped GHC < 8.6.

Genesis: Use the `unifdef` tool to fix the GHC version to 8.6.
```sh
src/full/Agda$ for i in $(find . -name "*.hs"); do unifdef -t -D__GLASGOW_HASKELL__=806 -o $i $i; done
```
This throws away too many conditionals
(like `#if __GLASGOW_HASKELL > 900` which evaluates to true then),
so one has to go through the changes manually and keep only the
correct ones.
Includes `Cabal-2.3`, `base-4.11` (GHC 8.4), `containers-0.5.8`.
@andreasabel andreasabel added type: task Concerning the development of Agda (not in changelog) dependencies Pull requests that update a dependency file labels Oct 13, 2023
@andreasabel andreasabel added this to the 2.6.4.1 milestone Oct 13, 2023
@andreasabel andreasabel self-assigned this Oct 13, 2023
@andreasabel andreasabel merged commit 4e02112 into master Oct 13, 2023
@andreasabel andreasabel deleted the hard-drop-GHC-8.4 branch November 30, 2023 18:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file type: task Concerning the development of Agda (not in changelog)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant
0