From d421c984615210863ebe4c6304be6ca782975ee9 Mon Sep 17 00:00:00 2001 From: Fabio Alessandrelli Date: Sat, 24 Aug 2024 11:56:27 +0200 Subject: [PATCH] [Web/SCons] Use CCFLAGS for SIDE_MODULE option Was using CPPFLAGS, but should use the explicit scons CCFLAGS which makes it clear they are applied to both the C and C++ compiler. CPPFLAGS was also fine (they are preprocessor flags, also applied to both C and C++), but we should try to stay consistent with what we do in Godot. (cherry picked from commit f36acd8e312c916c7e53364e1b0bd8eec3e4410e) --- tools/web.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/web.py b/tools/web.py index dea42b2..c8f07c5 100644 --- a/tools/web.py +++ b/tools/web.py @@ -39,7 +39,7 @@ def generate(env): env.Append(LINKFLAGS=["-sUSE_PTHREADS=1"]) # Build as side module (shared library). - env.Append(CPPFLAGS=["-sSIDE_MODULE=1"]) + env.Append(CCFLAGS=["-sSIDE_MODULE=1"]) env.Append(LINKFLAGS=["-sSIDE_MODULE=1"]) # Force wasm longjmp mode.