build(xsnap): don't set mxDebug in release builds

fixes Agoric#2216 only tested on lin, not mac nor win
dckc · Jan 23, 2021 · f000f0f · f000f0f
1 parent 648864c
commit f000f0f
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 5 deletions.
diff --git a/packages/xsnap/makefiles/lin/xsnap.mk b/packages/xsnap/makefiles/lin/xsnap.mk
@@ -25,7 +25,6 @@ C_OPTIONS = \
 	-fno-common \
 	-DINCLUDE_XSPLATFORM \
 	-DXSPLATFORM=\"xsnap.h\" \
-	-DmxDebug=1 \
 	-DmxMetering=1 \
 	-DmxParse=1 \
 	-DmxRun=1 \
@@ -41,7 +40,7 @@ C_OPTIONS += \
 	-Wno-misleading-indentation \
 	-Wno-implicit-fallthrough
 ifeq ($(GOAL),debug)
-	C_OPTIONS += -g -O0 -Wall -Wextra -Wno-missing-field-initializers -Wno-unused-parameter
+	C_OPTIONS += -g -O0 -Wall -Wextra -Wno-missing-field-initializers -Wno-unused-parameter -DmxDebug=1
 else
 	C_OPTIONS += -O3
 endif

diff --git a/packages/xsnap/makefiles/mac/xsnap.mk b/packages/xsnap/makefiles/mac/xsnap.mk
@@ -28,7 +28,6 @@ C_OPTIONS = \
 	$(MACOS_VERSION_MIN) \
 	-DINCLUDE_XSPLATFORM \
 	-DXSPLATFORM=\"xsnap.h\" \
-	-DmxDebug=1 \
 	-DmxMetering=1 \
 	-DmxParse=1 \
 	-DmxRun=1 \
@@ -44,7 +43,7 @@ ifneq ("x$(SDKROOT)", "x")
 	C_OPTIONS += -isysroot $(SDKROOT)
 endif
 ifeq ($(GOAL),debug)
-	C_OPTIONS += -g -O0 -Wall -Wextra -Wno-missing-field-initializers -Wno-unused-parameter
+	C_OPTIONS += -g -O0 -Wall -Wextra -Wno-missing-field-initializers -Wno-unused-parameter -DmxDebug=1
 else
 	C_OPTIONS += -O3
 endif

diff --git a/packages/xsnap/makefiles/win/xsnap.mak b/packages/xsnap/makefiles/win/xsnap.mak
@@ -20,7 +20,6 @@ C_OPTIONS = \
 	/D _CRT_SECURE_NO_DEPRECATE \
 	/D INCLUDE_XSPLATFORM \
 	/D XSPLATFORM=\"xsnap.h\" \
-	/D mxDebug=1 \
 	/D mxMetering=1 \
 	/D mxParse=1 \
 	/D mxRun=1 \
@@ -35,6 +34,7 @@ C_OPTIONS = \
 !IF "$(GOAL)"=="debug"
 C_OPTIONS = $(C_OPTIONS) \
 	/D _DEBUG \
+	/D mxDebug=1 \
 	/Fp$(TMP_DIR_DBG)\ \
 	/Od \
 	/W3 \