Skip to content

Commit 8668707

Browse files
committed
Fix nits
1 parent f7f225f commit 8668707

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

cmake/Zend/CMakeLists.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -505,7 +505,7 @@ add_feature_info(
505505

506506
include(cmake/MaxExecutionTimers.cmake)
507507
if(TARGET Zend::MaxExecutionTimers)
508-
target_link_libraries(zend PRIVATE Zend::MaxExecutionTimers)
508+
target_link_libraries(zend PUBLIC Zend::MaxExecutionTimers)
509509
endif()
510510

511511
################################################################################

cmake/Zend/cmake/MaxExecutionTimers.cmake

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -37,8 +37,8 @@ if(CMAKE_SYSTEM_NAME STREQUAL "Windows")
3737
return()
3838
endif()
3939

40-
include(PHP/SearchLibraries)
4140
include(FeatureSummary)
41+
include(PHP/SearchLibraries)
4242

4343
set(
4444
ZEND_MAX_EXECUTION_TIMERS "auto"
@@ -102,8 +102,8 @@ if(libraryForTimerCreate)
102102
)
103103
endif()
104104

105-
# The configuration header with ZEND_MAX_EXECUTION_TIMERS might not be included
106-
# in some source files, therefore also compilation definitions are added.
105+
# zend_config.h (or its parent php_config.h) isn't included in some zend_*
106+
# files, therefore also compilation definition is added.
107107
if(ZEND_MAX_EXECUTION_TIMERS)
108108
target_compile_definitions(
109109
Zend::MaxExecutionTimers

0 commit comments

Comments
 (0)