Skip to content

[ci] Use alma10 for the clang special build #19357

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
Jul 15, 2025

Conversation

guitargeek
Copy link
Contributor

Use alma10 for the clang special build so that we use a newer clang compiler that will hopefully give us new relevant warnings, and also try to use the stock alma10 image so that we don't have the duplication like with alma9 and alma9-clang. Installing the clang package in the regular image and then setting the correct CMake flags should do the trick.

Remove `requirements.txt` from roottest, since it got merged with the
main `requirements.txt` file.
@guitargeek guitargeek self-assigned this Jul 14, 2025
@guitargeek guitargeek requested a review from dpiparo as a code owner July 14, 2025 09:55
Copy link

github-actions bot commented Jul 14, 2025

Test Results

    21 files      21 suites   3d 9h 59m 16s ⏱️
 3 203 tests  3 203 ✅ 0 💤 0 ❌
65 603 runs  65 603 ✅ 0 💤 0 ❌

Results for commit c8567e1.

♻️ This comment has been updated with latest results.

Fixes the following test failure on AlmaLinux 10 with Clang:
```txt
Test 55: Divided pads and TLatex................................... OK
         PDF output................................................ OK
         JPG output................................................ OK
         PNG output......................................... 55 FAILED
         Result    = 24855
         Reference = 25303
         Error     = 448 (was 300)
         C file result............................................. OK
```
Use alma10 for the clang special build so that we use a newer clang
compiler that will hopefully give us new relevant warnings, and also try
to use the stock `alma10` image so that we don't have the duplication
like with `alma9` and `alma9-clang`. Installing the `clang` package in
the regular image and then setting the correct CMake flags should do the
trick.
Copy link
Member

@dpiparo dpiparo left a comment

Choose a reason for hiding this comment

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

thanks for these changes!

@guitargeek guitargeek merged commit 10cee61 into root-project:master Jul 15, 2025
46 of 48 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants