From 27075111ed93f5290fbb5d801964aa20ed7a9d35 Mon Sep 17 00:00:00 2001 From: Richard Yao Date: Fri, 11 Dec 2015 18:40:05 -0500 Subject: Unconditionally build zdb and ztest with -DDEBUG Illumos unconditionally builds zdb and ztest with -DDEBUG. This helps catch bugs and eliminates the need for commits like 202619623022722f30c2ee49931a4fa6896421c7, which changed ASSERTs to VERIFYs. The following files in the illumos tree show this: usr/src/cmd/zdb/Makefile.com usr/src/cmd/ztest/Makefile.com Given the usefulness of having early failure in these tools, we should do it too. Signed-off-by: Richard Yao Signed-off-by: Brian Behlendorf Issue #4095 --- cmd/zdb/Makefile.am | 2 ++ cmd/ztest/Makefile.am | 1 + 2 files changed, 3 insertions(+) diff --git a/cmd/zdb/Makefile.am b/cmd/zdb/Makefile.am index bc47cffba..c22186a10 100644 --- a/cmd/zdb/Makefile.am +++ b/cmd/zdb/Makefile.am @@ -1,5 +1,7 @@ include $(top_srcdir)/config/Rules.am +AM_CPPFLAGS += -DDEBUG + DEFAULT_INCLUDES += \ -I$(top_srcdir)/include \ -I$(top_srcdir)/lib/libspl/include diff --git a/cmd/ztest/Makefile.am b/cmd/ztest/Makefile.am index ef4d99f28..e3be39a0c 100644 --- a/cmd/ztest/Makefile.am +++ b/cmd/ztest/Makefile.am @@ -1,6 +1,7 @@ include $(top_srcdir)/config/Rules.am AM_CFLAGS += $(DEBUG_STACKFLAGS) $(FRAME_LARGER_THAN) +AM_CPPFLAGS += -DDEBUG DEFAULT_INCLUDES += \ -I$(top_srcdir)/include \ -- cgit v1.2.3