diff options
| author | Yu-Ping Wu <yupingso@chromium.org> | 2023-04-11 15:59:34 +0800 |
|---|---|---|
| committer | Chromeos LUCI <chromeos-scoped@luci-project-accounts.iam.gserviceaccount.com> | 2023-04-13 20:18:37 +0000 |
| commit | 090aa4131649f4a1b32e65874691fd48e3207d5f (patch) | |
| tree | 79030d355306617ee086daa10359e2016823b29c | |
| parent | f28c9d7a5b2485b7346a19763aa4db004480d08d (diff) | |
| download | vboot-090aa4131649f4a1b32e65874691fd48e3207d5f.tar.gz | |
Makefile: Disable NDEBUG by default
We decided in to enable NDEBUG by default because cryptolib would spam
the debug output [1]. Now that cryptolib is gone (CL:400903), we should
be able to turn it on by default, so that assert() calls will no longer
be no-ops.
The DISABLE_NDEBUG make variable is replaced with NDEBUG.
[1] https://codereview.chromium.org/3004001
BUG=none
TEST=make NDEBUG=1 futil -j
TEST=emerge-corsola vboot_reference
BRANCH=none
Change-Id: Ic97d85f0cc14c05a1084d02c6a150e4cc7d3393e
Signed-off-by: Yu-Ping Wu <yupingso@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/vboot_reference/+/4413341
Reviewed-by: Julius Werner <jwerner@chromium.org>
Commit-Queue: Julius Werner <jwerner@chromium.org>
| -rw-r--r-- | Makefile | 2 |
1 files changed, 1 insertions, 1 deletions
@@ -176,7 +176,7 @@ ifneq ($(filter-out 0,${DEBUG})$(filter-out 0,${TEST_PRINT}),) CFLAGS += -DVBOOT_DEBUG endif -ifeq ($(filter-out 0,${DISABLE_NDEBUG}),) +ifneq ($(filter-out 0,${NDEBUG}),) CFLAGS += -DNDEBUG endif |