[ci] Add minimal asan build#22618
Conversation
Test Results 18 files 18 suites 2d 21h 56m 33s ⏱️ Results for commit 0eb641f. |
|
this is a great initiative. To optimise, one could even turn off all other builds while in the PR with a specific commit, so that only asan can be tested, the necessary fixes added, and then all other builds re-added before merging |
ferdymercury
left a comment
ferdymercury
left a comment
There was a problem hiding this comment.
Thanks! Could you port the changes from https://github.com/root-project/root/pull/19636/changes into here?
The documentation fix for asan build option and portentially the Halton error flag to get s full overview at once
Thks!
| platform_config: alma10 | ||
| is_special: true | ||
| property: "asan" | ||
| overrides: ["asan=ON", "minimal=ON"] |
There was a problem hiding this comment.
| overrides: ["asan=ON", "minimal=ON"] | |
| overrides: ["asan=ON", "minimal=ON, "CMAKE_BUILD_TYPE=Debug"] |
Could help in debugging
| overrides: ["CMAKE_CXX_STANDARD=20"] | ||
| # Asan build | ||
| - image: alma10 | ||
| platform_config: alma10 |
There was a problem hiding this comment.
If remember correctly the correct way to actually get a minimal build is:
| platform_config: alma10 | |
| platform_config: alma10-minimal |
Otherwise the other options just get added.
4 participants
Using the alma10 image.
IMPORTANT: asan currently only works with gcc (or at least, that was the case on my machine).