Skip to content

Conversation

@yutotakano
Copy link
Owner

@yutotakano yutotakano commented Jul 10, 2025

Fixed the macOS 8.10.7 CI being broken because of 8.10.7 requiring llvm < 13 when Homebrew have disabled llvm@12 since the first of July this year. The solution is to override the disabled status in a local copy of the formula, then install from that (hinting to Homebrew that it should prioritise local copies over the Homebrew web API).

Ubuntu 20.04 was also phased out this May, so this PR also bumps the standard Ubuntu cases to run on 24.04. In addition, GHC versions up to Stackage LTS (9.8.4) was added for the standard Ubuntu CI matrix. Once #59 is merged, we can also add 9.10 and 9.12 in the CI at least for the standard Ubuntu matrix.

For Windows and Mac CI (which are supplementary to the ubuntu matrix which tests all supported GHC versions), this PR also adds LTS compilation tests because only running 8.10.7 felt a little outdated.

@yutotakano yutotakano changed the title Try force-installing disabled llvm@12 on macOS CI Fix macOS CI broken due to disabled llvm@12, bump Ubuntu to 24 Jul 10, 2025
@yutotakano yutotakano enabled auto-merge July 10, 2025 11:45
@yutotakano yutotakano disabled auto-merge July 10, 2025 11:59
@yutotakano
Copy link
Owner Author

yutotakano commented Jul 10, 2025

This PR introduces an issue where on GHC 8.10.7, the cabal resolver finds a local maxima that doesn't involve the newest supported discord-haskell version (1.17.1) and instead choose 1.13.0. This is because the version restrictions on 1.17.1 lead to more negative scoring of the solver solution compared to 1.13.0 which has no version restrictions on dependencies.

The issue will be fixed when #59 is merged.

@yutotakano yutotakano merged commit 1d349c9 into main Jul 10, 2025
11 checks passed
@yutotakano yutotakano deleted the try-fix-llvm-12 branch July 10, 2025 12:48
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