Skip to content

ci: add support for asan with cmake #1025

Draft
OverOrion wants to merge 8 commits into
axoflow:mainfrom
OverOrion:ci_cmake_asan_fixed
Draft

ci: add support for asan with cmake #1025
OverOrion wants to merge 8 commits into
axoflow:mainfrom
OverOrion:ci_cmake_asan_fixed

Conversation

@OverOrion
Copy link
Copy Markdown
Contributor

I don't think it necessitates a news file.

To try it out (assuming build build directory and install_dir directory for prefix, currently in repository root):

$ cmake -DSANITIZER=address -DCMAKE_INSTALL_PREFIX=`realpath ./install_dir` -DCMAKE_C_COMPILER=clang -DCMAKE_CXX_COMPILER=clang++ -B build .
$ make -j 8 install
$ nm -na ../install_dir/sbin/syslog-ng | grep -i asan_ -c
365

@OverOrion OverOrion self-assigned this Apr 2, 2026
@OverOrion OverOrion force-pushed the ci_cmake_asan_fixed branch from 0583098 to ef6063b Compare April 2, 2026 12:01
Comment thread modules/systemd-journal/tests/CMakeLists.txt Outdated
@OverOrion OverOrion force-pushed the ci_cmake_asan_fixed branch 4 times, most recently from f6d52d9 to 4d41288 Compare April 2, 2026 14:35
Signed-off-by: Szilard Parrag <szilard.parrag@axoflow.com>
Signed-off-by: Szilard Parrag <szilard.parrag@axoflow.com>
Signed-off-by: Szilard Parrag <szilard.parrag@axoflow.com>
Signed-off-by: Szilard Parrag <szilard.parrag@axoflow.com>
Signed-off-by: Szilard Parrag <szilard.parrag@axoflow.com>
Signed-off-by: Szilard Parrag <szilard.parrag@axoflow.com>
@OverOrion OverOrion force-pushed the ci_cmake_asan_fixed branch from 4d41288 to 5dfd4aa Compare April 2, 2026 14:43
Signed-off-by: Szilard Parrag <szilard.parrag@axoflow.com>
Signed-off-by: Szilard Parrag <szilard.parrag@axoflow.com>
@OverOrion OverOrion force-pushed the ci_cmake_asan_fixed branch from 5dfd4aa to 0762581 Compare April 2, 2026 15:11
@bazsi
Copy link
Copy Markdown
Member

bazsi commented Apr 23, 2026

the changes so far look good to me. it still is not green in cmake/clang

@OverOrion
Copy link
Copy Markdown
Contributor Author

Yeah, once I find a little time to tinker with it I'll make sure it's green.

@OverOrion OverOrion marked this pull request as draft April 28, 2026 14:29
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.

3 participants