Skip to content

Commit 458a858

Browse files
committed
Merge branch 'PHP-8.3' into PHP-8.4
2 parents 3b6ee80 + b3296a7 commit 458a858

File tree

7 files changed

+27
-19
lines changed

7 files changed

+27
-19
lines changed

cmake/cmake/Configuration.cmake

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -133,10 +133,9 @@ mark_as_advanced(PHP_DMALLOC)
133133
option(PHP_DTRACE "Enable DTrace support" OFF)
134134
mark_as_advanced(PHP_DTRACE)
135135

136-
if(CMAKE_SYSTEM_NAME STREQUAL "Windows")
137-
set(PHP_FD_SETSIZE "256" CACHE STRING "Size of descriptor sets")
138-
else()
139-
set(PHP_FD_SETSIZE "" CACHE STRING "Size of descriptor sets")
136+
set(PHP_FD_SETSIZE "" CACHE STRING "Size of file descriptor sets")
137+
if(CMAKE_SYSTEM_NAME STREQUAL "Windows" AND PHP_FD_SETSIZE STREQUAL "")
138+
set_property(CACHE PHP_FD_SETSIZE PROPERTY VALUE "256")
140139
endif()
141140
mark_as_advanced(PHP_FD_SETSIZE)
142141

@@ -145,7 +144,7 @@ mark_as_advanced(PHP_VALGRIND)
145144

146145
option(
147146
PHP_MEMORY_SANITIZER
148-
"Enable the memory sanitizer compiler options (clang only)"
147+
"Enable the memory sanitizer compiler options (Clang only)"
149148
OFF
150149
)
151150
mark_as_advanced(PHP_MEMORY_SANITIZER)

cmake/cmake/ConfigureChecks.cmake

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -427,15 +427,19 @@ include(PHP/CheckWrite)
427427

428428
# Checking file descriptor sets.
429429
message(CHECK_START "Checking file descriptor sets size")
430-
if(PHP_FD_SETSIZE GREATER 0)
430+
if(PHP_FD_SETSIZE MATCHES "^[0-9]+$" AND PHP_FD_SETSIZE GREATER 0)
431431
message(CHECK_PASS "using FD_SETSIZE=${PHP_FD_SETSIZE}")
432432
target_compile_definitions(
433433
php_configuration
434434
INTERFACE
435435
$<$<COMPILE_LANGUAGE:ASM,C,CXX>:FD_SETSIZE=${PHP_FD_SETSIZE}>
436436
)
437-
elseif(NOT PHP_FD_SETSIZE STREQUAL "" AND NOT PHP_FD_SETSIZE GREATER 0)
438-
message(FATAL_ERROR "Invalid value PHP_FD_SETSIZE=${PHP_FD_SETSIZE}")
437+
elseif(NOT PHP_FD_SETSIZE STREQUAL "")
438+
message(
439+
FATAL_ERROR
440+
"Invalid value of PHP_FD_SETSIZE=${PHP_FD_SETSIZE}. Pass integer greater "
441+
"than 0."
442+
)
439443
else()
440444
message(CHECK_PASS "using system default")
441445
endif()

cmake/cmake/Flags.cmake

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -382,7 +382,7 @@ if(PHP_UNDEFINED_SANITIZER)
382382
NOT DEFINED PHP_HAVE_UBSAN_EXITCODE
383383
AND CMAKE_CROSSCOMPILING
384384
AND NOT CMAKE_CROSSCOMPILING_EMULATOR
385-
AND CMAKE_C_COMPILER_ID STREQUAL "Clang"
385+
AND CMAKE_C_COMPILER_ID MATCHES "AppleClang|Clang"
386386
AND CMAKE_C_COMPILER_VERSION VERSION_GREATER_EQUAL 17
387387
)
388388
# When cross-compiling without emulator and using Clang 17 and greater,

cmake/cmake/modules/PHP/CheckAttribute.cmake

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -106,7 +106,7 @@ function(_php_check_attribute what attribute result)
106106
cmake_push_check_state(RESET)
107107
# Compilers by default may not treat attribute warnings as errors, so some
108108
# error flag needs to be set to make the check certain (-Werror=attributes,
109-
# clang's -Werror=unknown-attributes, -Werror...). Use the internal CMake
109+
# Clang's -Werror=unknown-attributes, -Werror...). Use the internal CMake
110110
# compiler-agnostic variable CMAKE_C_COMPILE_OPTIONS_WARNING_AS_ERROR
111111
# (-Werror or other compiler-based option), if available.
112112
list(JOIN CMAKE_C_COMPILE_OPTIONS_WARNING_AS_ERROR " " CMAKE_REQUIRED_FLAGS)

cmake/main/CMakeLists.txt

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -183,6 +183,11 @@ if(PHP_DTRACE)
183183
message(CHECK_FAIL "no")
184184
endif()
185185
endif()
186+
add_feature_info(
187+
"DTrace"
188+
PHP_DTRACE
189+
"performance analysis and troubleshooting"
190+
)
186191

187192
# Add Dmalloc.
188193
if(PHP_DMALLOC)

cmake/sapi/cli/CMakeLists.txt

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -11,14 +11,6 @@ add_feature_info(
1111
"Command-line interface SAPI executable"
1212
)
1313

14-
message(CHECK_START "Checking cli SAPI")
15-
if(SAPI_CLI)
16-
message(CHECK_PASS "enabled")
17-
else()
18-
message(CHECK_FAIL "disabled")
19-
return()
20-
endif()
21-
2214
cmake_dependent_option(
2315
SAPI_CLI_WIN_NO_CONSOLE
2416
"Build console-less CLI SAPI"
@@ -32,6 +24,14 @@ add_feature_info(
3224
"[Windows only] Same as CLI SAPI but without console (no output is given)"
3325
)
3426

27+
message(CHECK_START "Checking cli SAPI")
28+
if(SAPI_CLI)
29+
message(CHECK_PASS "enabled")
30+
else()
31+
message(CHECK_FAIL "disabled")
32+
return()
33+
endif()
34+
3535
check_symbol_exists(setproctitle "unistd.h;stdlib.h" HAVE_SETPROCTITLE)
3636
check_include_file(sys/pstat.h HAVE_SYS_PSTAT_H)
3737

cmake/sapi/fpm/CMakeLists.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ option(SAPI_FPM "Enable the FPM SAPI module" OFF)
2121
add_feature_info(
2222
"sapi/fpm"
2323
SAPI_FPM
24-
"FPM SAPI module"
24+
"FastCGI Process Manager (FPM) SAPI module"
2525
)
2626

2727
if(SAPI_FPM)

0 commit comments

Comments
 (0)