[Buildroot] [git commit branch/next] Revert "core: enable 'NDEBUG' unless BR2_ENABLE_RUNTIME_DEBUG is set"
From: Yann E. MORIN <hidden>
Date: 2021-08-03 21:48:22
Subsystem:
the rest · Maintainer:
Linus Torvalds
commit: https://git.buildroot.net/buildroot/commit/?id=a1c7cff1a081765c082c196bd9e6c1e72ceee797 branch: https://git.buildroot.net/buildroot/commit/?id=refs/heads/next Enabling -DNEBUG, although correct on the paper, causes a lot of packages to fail to build because they explicitly require not building with NDEBUG; they use assert() to check actual runtime errors and expect it to not be elidded away (sometimes with side effects in the arguments passed to assert(). This reverts commit 5a8c50fe05afacc3cbe8e7347e238da9f242fab0, as discussed on the list: http://lists.busybox.net/pipermail/buildroot/2021-July/313646.html Signed-off-by: Yann E. MORIN <redacted> --- package/Makefile.in | 3 --- 1 file changed, 3 deletions(-)
diff --git a/package/Makefile.in b/package/Makefile.in
index 955e6a8e8c..86db62ba5b 100644
--- a/package/Makefile.in
+++ b/package/Makefile.in@@ -148,9 +148,6 @@ endif ifeq ($(BR2_DEBUG_3),y) TARGET_DEBUGGING = -g3 endif -ifeq ($(BR2_ENABLE_RUNTIME_DEBUG),) -TARGET_DEBUGGING += -DNDEBUG -endif TARGET_LDFLAGS = $(call qstrip,$(BR2_TARGET_LDFLAGS))
_______________________________________________ buildroot mailing list buildroot@busybox.net http://lists.busybox.net/mailman/listinfo/buildroot