Skip to content

Commit d3ba986

Browse files
authored
Remove dead CMake options (#7847)
1 parent ec03274 commit d3ba986

4 files changed

Lines changed: 8 additions & 35 deletions

File tree

CMakeLists.txt

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -60,8 +60,6 @@ set(CMAKE_EXPORT_COMPILE_COMMANDS ON)
6060

6161
find_package(Threads REQUIRED)
6262

63-
option(PROFILE_TESTS "Profile tests" OFF)
64-
6563
if("$ENV{CI}" STREQUAL "")
6664
set(PYTHON unbuffer python3)
6765
else()

cmake/common.cmake

Lines changed: 1 addition & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -97,21 +97,7 @@ function(add_e2e_test)
9797
endif()
9898

9999
if(BUILD_END_TO_END_TESTS)
100-
if(PROFILE_TESTS)
101-
set(
102-
PYTHON_WRAPPER
103-
py-spy
104-
record
105-
--format
106-
speedscope
107-
-o
108-
${PARSED_ARGS_NAME}.trace
109-
--
110-
python3
111-
)
112-
else()
113-
set(PYTHON_WRAPPER ${PYTHON})
114-
endif()
100+
set(PYTHON_WRAPPER ${PYTHON})
115101

116102
# For fast e2e runs, tick node faster than default value (except for
117103
# instrumented builds which may process ticks slower).

cmake/preproject.cmake

Lines changed: 6 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -15,13 +15,6 @@ if(
1515
find_program(FOUND_CMAKE_C_COMPILER NAMES clang)
1616
find_program(FOUND_CMAKE_CXX_COMPILER NAMES clang++)
1717

18-
# vvvvv Ubuntu-20.04, to be removed after support dropped. vvvvv #
19-
if(NOT (FOUND_CMAKE_C_COMPILER AND FOUND_CMAKE_CXX_COMPILER))
20-
find_program(FOUND_CMAKE_C_COMPILER NAMES clang-15)
21-
find_program(FOUND_CMAKE_CXX_COMPILER NAMES clang++-15)
22-
endif()
23-
# ^^^^^ Ubuntu-20.04, to be removed after support dropped. ^^^^^ #
24-
2518
if(NOT (FOUND_CMAKE_C_COMPILER AND FOUND_CMAKE_CXX_COMPILER))
2619
message(
2720
WARNING
@@ -63,11 +56,12 @@ if(FUZZING AND TSAN)
6356
message(FATAL_ERROR "FUZZING and TSAN cannot be enabled together")
6457
endif()
6558

66-
option(COLORED_OUTPUT "Always produce ANSI-colored output." ON)
67-
68-
if(${COLORED_OUTPUT})
69-
add_compile_options(-fcolor-diagnostics)
70-
endif()
59+
add_compile_options(
60+
$<$<COMPILE_LANG_AND_ID:C,Clang>:-fcolor-diagnostics>
61+
$<$<COMPILE_LANG_AND_ID:CXX,Clang>:-fcolor-diagnostics>
62+
$<$<COMPILE_LANG_AND_ID:C,GNU>:-fdiagnostics-color=always>
63+
$<$<COMPILE_LANG_AND_ID:CXX,GNU>:-fdiagnostics-color=always>
64+
)
7165

7266
function(add_warning_checks name)
7367
target_compile_options(

cmake/quickjs.cmake

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,7 @@
11
# Copyright (c) Microsoft Corporation. All rights reserved.
22
# Licensed under the Apache 2.0 License.
33

4-
set(
5-
QUICKJS_PREFIX
6-
${CCF_3RD_PARTY_EXPORTED_DIR}/quickjs
7-
CACHE PATH
8-
"Prefix to the QuickJS library"
9-
)
4+
set(QUICKJS_PREFIX ${CCF_3RD_PARTY_EXPORTED_DIR}/quickjs)
105

116
set(QUICKJS_INC ${QUICKJS_PREFIX})
127

0 commit comments

Comments
 (0)