CI: Remove the broken clang installation step #1855

Merged

Jade merged 1 commit from spaetz/continuwuity:spaetz-patch-1 into main

2026-06-19 11:23:53 +00:00

spaetz commented

2026-06-18 07:20:29 +00:00

Member

tomfos.tr act-runner images removed the possibility on Feb 19, 2026 to install the latest LLVM using an installer script in git.tomfos.tr/tom/act-runner@01782867ce.
I built the Debian/Ubuntu CI with the distribution repository clang, and both Debian and Ubuntu latest built fine, so let us also remove the detection and just live with the distro's clang package.
Debian Trixie has clang 1.19 and Ubuntu resolute has Clang 1.21.

If this ever becomes a problem and we need a different version we would need to reintroduce
this step but install clang directly using the LLVM deb repo (https://apt.llvm.org/#ubuntu), installing the clang version that we actually need. Or we would have to reintroduce the installation via the installation scripts that Tom has cut out of their runner image.

This pull request would fix the broken build-Debian CI run (ubuntu-latest is broken as there is no clang-23 package available) and I am not sure that we actually need clang 23(?!)

Pull request checklist:

This pull request targets the main branch, and the branch is named something other than
main.
I have written an appropriate pull request title and my description is clear.
I understand I am responsible for the contents of this pull request.
I have followed the contributing guidelines:
- My contribution follows the code style, if applicable.
- I ran pre-commit checks before opening/drafting this pull request.
- I have tested my contribution (or proof-read it for documentation-only changes)
  myself, if applicable. This includes ensuring code compiles.
- My commit messages follow the commit message format and are descriptive.

tomfos.tr act-runner images removed the possibility on Feb 19, 2026 to install the latest LLVM using an installer script in https://git.tomfos.tr/tom/act-runner/commit/01782867ce50c4d81302f27b8bbe9a7a48e46581. I built the Debian/Ubuntu CI with the distribution repository clang, and both Debian and Ubuntu latest built fine, so let us also remove the detection and just live with the distro's clang package. Debian Trixie has clang 1.19 and Ubuntu resolute has Clang 1.21. If this ever becomes a problem and we need a different version we would need to reintroduce this step but install clang directly using the LLVM deb repo (https://apt.llvm.org/#ubuntu), installing the clang version that we actually need. Or we would have to reintroduce the installation via the installation scripts that Tom has cut out of their runner image. This pull request would fix the broken build-Debian CI run (ubuntu-latest is broken as there is no clang-23 package available) and I am not sure that we actually need clang 23(?!)  **Pull request checklist:**  - [X] This pull request targets the `main` branch, and the branch is named something other than `main`. - [X] I have written an appropriate pull request title and my description is clear. - [X] I understand I am responsible for the contents of this pull request. - I have followed the [contributing guidelines][c1]: - [X] My contribution follows the [code style][c2], if applicable. - [X] I ran [pre-commit checks][c1pc] before opening/drafting this pull request. - [X] I have [tested my contribution][c1t] (or proof-read it for documentation-only changes) myself, if applicable. This includes ensuring code compiles. - [X] My commit messages follow the [commit message format][c1cm] and are descriptive.  [c1]: https://forgejo.ellis.link/continuwuation/continuwuity/src/branch/main/CONTRIBUTING.md [c2]: https://forgejo.ellis.link/continuwuation/continuwuity/src/branch/main/docs/development/code_style.mdx [c1pc]: https://forgejo.ellis.link/continuwuation/continuwuity/src/branch/main/CONTRIBUTING.md#pre-commit-checks [c1t]: https://forgejo.ellis.link/continuwuation/continuwuity/src/branch/main/CONTRIBUTING.md#running-tests-locally [c1cm]: https://forgejo.ellis.link/continuwuation/continuwuity/src/branch/main/CONTRIBUTING.md#commit-messages

spaetz added 1 commit

2026-06-18 07:20:30 +00:00

CI: Remove the clang detection

Documentation / Build and Deploy Documentation (pull_request) Has been skipped

Details

Auto Labeler / Apply labels based on changed files (pull_request_target) Successful in 2s

Details

Checks / Changelog / Check changelog is added (pull_request_target) Successful in 7s

Details

Checks / Prek / Check changed files (pull_request) Successful in 6s

Details

Checks / Prek / Clippy and Cargo Tests (pull_request) Has been skipped

Details

Update flake hashes / update-flake-hashes (pull_request) Successful in 1m9s

Details

Checks / Prek / Pre-commit & Formatting (pull_request) Successful in 1m22s

Details

f265c836fa

tomfos.tr act-runner image removed the possibility to install the latest LLVM using an installer script, so let us also remove the detection and just live with the distro's clang image.