From: Yann E. MORIN Date: Sun, 4 Jul 2021 11:33:50 +0000 (+0200) Subject: Revert "core: enable 'NDEBUG' unless BR2_ENABLE_RUNTIME_DEBUG is set" X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=a1c7cff1a081765c082c196bd9e6c1e72ceee797;p=buildroot.git Revert "core: enable 'NDEBUG' unless BR2_ENABLE_RUNTIME_DEBUG is set" 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 --- 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))