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

Build against sbv-10.2 #1550

Merged
merged 5 commits into from
Jul 13, 2023
Merged

Build against sbv-10.2 #1550

merged 5 commits into from
Jul 13, 2023

Conversation

RyanGlScott
Copy link
Contributor

The primary motivation here is to have the Docker images and binary artifacts build against sbv-10.2, which contains a fix for #1548. This requires a fair bit of CI rejigging, however, as our Docker images and binary artifacts were previously being built against GHC 8.10.7, which doesn't support building sbv-10.*. As such, we now adjust the CI to use GHC 9.2.8 (the version that ghcup recommends at the time of writing) instead of GHC 8.10.7.

@RyanGlScott RyanGlScott temporarily deployed to github-pages July 12, 2023 12:18 — with GitHub Actions Inactive
@RyanGlScott RyanGlScott changed the title Building against sbv-10.2 Build against sbv-10.2 Jul 12, 2023
@RyanGlScott RyanGlScott temporarily deployed to github-pages July 12, 2023 13:34 — with GitHub Actions Inactive
There are two motivations for doing so:

* `sbv-10.0` and later no longer build against GHC 8.10 (see
  LeventErkok/sbv#655), but we want to use a new `sbv`
  version to come to a resolution to #1548. As such, we need a newer GHC.
* `ghcup` now recommends GHC 9.2.8 for most usage, so it's time we switched
  anyway.
@RyanGlScott RyanGlScott temporarily deployed to github-pages July 12, 2023 14:26 — with GitHub Actions Inactive
@RyanGlScott RyanGlScott marked this pull request as draft July 12, 2023 16:23
@RyanGlScott RyanGlScott temporarily deployed to github-pages July 13, 2023 12:51 — with GitHub Actions Inactive
@RyanGlScott RyanGlScott marked this pull request as ready for review July 13, 2023 15:06
@RyanGlScott RyanGlScott requested a review from kquick July 13, 2023 15:06
@RyanGlScott
Copy link
Contributor Author

Successful CI workflow that demonstrates that the Docker images now build after these changes: https://github.com/GaloisInc/cryptol/actions/runs/5543303142

Copy link
Member

@kquick kquick left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There's an unfortunate amount of duplication between these files, but I don't see a clear way to avoid it at this stage. :-(

README.md Outdated Show resolved Hide resolved
@RyanGlScott RyanGlScott temporarily deployed to github-pages July 13, 2023 15:37 — with GitHub Actions Inactive
@RyanGlScott RyanGlScott merged commit 383a597 into master Jul 13, 2023
43 checks passed
@RyanGlScott RyanGlScott deleted the T1548-part-two branch July 13, 2023 16:46
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.

None yet

2 participants