Skip to content

Commit 3842fd1

Browse files
authored
PCBC-1031: Do not fiddle with PHP_DEBUG and NDEBUG (#229)
1 parent 70d00ac commit 3842fd1

File tree

3 files changed

+13
-2
lines changed

3 files changed

+13
-2
lines changed

Makefile.frag

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ $(COUCHBASE_CMAKE_BUILD_DIRECTORY)/CMakeCache.txt: $(COUCHBASE_CMAKE_SOURCE_DIRE
22
$(CMAKE) \
33
-S $(COUCHBASE_CMAKE_SOURCE_DIRECTORY) \
44
-B $(COUCHBASE_CMAKE_BUILD_DIRECTORY) \
5-
-DCMAKE_BUILD_TYPE=$(if $(COUCHBASE_CMAKE_BUILD_TYPE),$(COUCHBASE_CMAKE_BUILD_TYPE),RelWithDebInfo) \
5+
-DCMAKE_BUILD_TYPE=$(if $(COUCHBASE_CMAKE_BUILD_TYPE),$(COUCHBASE_CMAKE_BUILD_TYPE),$(COUCHBASE_DEFAULT_CMAKE_BUILD_TYPE)) \
66
-DCMAKE_C_COMPILER="$(CC)" \
77
-DCMAKE_CXX_COMPILER="$(CXX)" \
88
-DCMAKE_C_FLAGS="$(COMMON_FLAGS)" \

config.m4

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -34,10 +34,16 @@ if test "$PHP_COUCHBASE" != "no"; then
3434
CXXFLAGS="${CXXFLAGS} -std=c++17"
3535
COUCHBASE_CMAKE_SOURCE_DIRECTORY="$srcdir/src"
3636
COUCHBASE_CMAKE_BUILD_DIRECTORY="$ac_pwd/cmake-build"
37+
COUCHBASE_DEFAULT_CMAKE_BUILD_TYPE=RelWithDebInfo
38+
39+
if test "$PHP_DEBUG" = "yes" -o "$PHP_DEBUG" = "1"; then
40+
COUCHBASE_DEFAULT_CMAKE_BUILD_TYPE=Debug
41+
fi
3742

3843
PHP_SUBST([CMAKE])
3944
PHP_SUBST([COUCHBASE_CMAKE_SOURCE_DIRECTORY])
4045
PHP_SUBST([COUCHBASE_CMAKE_BUILD_DIRECTORY])
46+
PHP_SUBST([COUCHBASE_DEFAULT_CMAKE_BUILD_TYPE])
4147

4248
PHP_NEW_EXTENSION(couchbase,, $ext_shared,,, cxx)
4349
PHP_ADD_EXTENSION_DEP(couchbase, json)

config.w32

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -65,12 +65,17 @@ if (PHP_COUCHBASE != "no") {
6565

6666
EXTENSION("couchbase", "dummy.c", true);
6767

68+
COUCHBASE_DEFAULT_CMAKE_BUILD_TYPE="RelWithDebInfo"
69+
if (PHP_DEBUG == "yes" || PHP_DEBUG == "1") {
70+
COUCHBASE_DEFAULT_CMAKE_BUILD_TYPE="Debug"
71+
}
72+
6873
cmake_command =
6974
'cmd /c ""' + CMAKE + '"' +
7075
' -S "' + COUCHBASE_CMAKE_SOURCE_DIRECTORY + '"' +
7176
' -B "' + COUCHBASE_CMAKE_BUILD_DIRECTORY + '"' +
7277
' -G "NMake Makefiles"' +
73-
' -D CMAKE_BUILD_TYPE=RelWithDebInfo' +
78+
' -D CMAKE_BUILD_TYPE=' + COUCHBASE_DEFAULT_CMAKE_BUILD_TYPE +
7479
' -D CMAKE_ASM_NASM_COMPILER:PATH="' + NASM + '"' +
7580
' -D COUCHBASE_PHP_CFLAGS="' + COUCHBASE_PHP_CFLAGS + '"' +
7681
' -D COUCHBASE_PHP_INCLUDES="' + COUCHBASE_PHP_INCLUDES + '"' +

0 commit comments

Comments
 (0)