Skip to content

[ci] fix ASAN build - #22726

Closed
jblomer wants to merge 2 commits into
root-project:masterfrom
jblomer:asan-rdf
Closed

[ci] fix ASAN build#22726
jblomer wants to merge 2 commits into
root-project:masterfrom
jblomer:asan-rdf

Conversation

@jblomer

@jblomer jblomer commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

Turns out that we weren't running ASAN builds so far. This PR should fix it. Also change the build type to RelWithDebugInfo.

@jblomer
jblomer requested a review from vepadulano June 29, 2026 09:00
@jblomer jblomer self-assigned this Jun 29, 2026
@jblomer
jblomer requested a review from dpiparo as a code owner June 29, 2026 09:00
@github-actions

github-actions Bot commented Jun 29, 2026

Copy link
Copy Markdown

Test Results

    23 files      23 suites   3d 22h 21m 1s ⏱️
 3 874 tests  3 858 ✅   0 💤 16 ❌
79 712 runs  79 444 ✅ 252 💤 16 ❌

For more details on these failures, see this check.

Results for commit c98b2fa.

♻️ This comment has been updated with latest results.

@jblomer jblomer added the clean build Ask CI to do non-incremental build on PR label Jun 29, 2026
@jblomer jblomer closed this Jun 29, 2026
@jblomer jblomer reopened this Jun 29, 2026
@jblomer
jblomer force-pushed the asan-rdf branch 3 times, most recently from 120a8c9 to 83f377e Compare July 9, 2026 07:26
@jblomer jblomer changed the title [ci] add rdf to asan build [ci] fix ASAN build Jul 9, 2026
@jblomer
jblomer requested a review from silverweed July 9, 2026 07:26
@jblomer jblomer removed the clean build Ask CI to do non-incremental build on PR label Jul 9, 2026
@jblomer
jblomer force-pushed the asan-rdf branch 3 times, most recently from 3edcb4b to c98b2fa Compare July 9, 2026 21:39
@@ -1,11 +1,10 @@
CMAKE_BUILD_TYPE=Debug
CMAKE_CXX_FLAGS_DEBUG="-Og -g"
CMAKE_BUILD_TYPE=RelWithDebugInfo

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I believe this should be called RelWithDebInfo if it's supposed to have an effect.
Currently, this leads to a build without any optimisation flags or debug options.

@jblomer

jblomer commented Sep 4, 2026

Copy link
Copy Markdown
Contributor Author

Replaced by #22943

@jblomer jblomer closed this Sep 4, 2026
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.

4 participants