Remove -UNDEBUG from tests
The explicit -UNDEBUG was introduced in the CMake to meson migration (62900cd1) but as far as I can tell it was not present in the CMake tests definition.
First of all, I don't see any reason why we would undef NDEBUG for builds where we explicitly define it.
Secondly, it makes tests less reliable because it enables only some debug messages but not all. For example if emper is build with NDEBUG a test using a Semaphore will output debug messages from the Semaphore but not from the Runtime.
Merge request reports
Activity
added 5 commits
-
32defcb5...118a4dcb - 4 commits from branch
master
- b33ac93c - [tests] don't undef NDEBUG for test executables
-
32defcb5...118a4dcb - 4 commits from branch
enabled an automatic merge when the pipeline for b33ac93c succeeds
mentioned in commit 39d147fb
Please register or sign in to reply