Skip to content

Commit 1a85f33

Browse files
authored
Remove unnecessary -O2 flag
Make style more consistent
1 parent b10d71e commit 1a85f33

File tree

1 file changed

+9
-10
lines changed

1 file changed

+9
-10
lines changed

CMakeLists.txt

+9-10
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
CMAKE_MINIMUM_REQUIRED(VERSION 3.0.2)
1+
CMAKE_MINIMUM_REQUIRED (VERSION 3.0.2)
22

33
INCLUDE (cmake/cpp17.cmake)
44
INCLUDE (cmake/subdirs.cmake)
@@ -10,20 +10,19 @@ OPTION (WITH_OPENSSL "Use OpenSSL for TLS connections" OFF)
1010

1111
PROJECT (CLICKHOUSE-CLIENT)
1212

13-
USE_CXX17()
14-
USE_OPENSSL()
13+
USE_CXX17 ()
14+
USE_OPENSSL ()
1515

1616
IF ("${CMAKE_BUILD_TYPE}" STREQUAL "")
1717
SET (CMAKE_BUILD_TYPE "Debug")
18-
ENDIF()
18+
ENDIF ()
1919

2020
IF (UNIX)
21-
IF (APPLE)
22-
SET (CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -O2 -Wall -Wextra -Werror")
23-
ELSE ()
24-
SET (CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -pthread -Wall -Wextra -Werror")
21+
IF (NOT APPLE)
22+
SET (CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -pthread")
2523
ENDIF ()
2624
SET (CMAKE_EXE_LINKER_FLAGS, "${CMAKE_EXE_LINKER_FLAGS} -lpthread")
25+
SET (CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wall -Wextra -Werror")
2726
ENDIF ()
2827

2928
INCLUDE_DIRECTORIES (.)
@@ -37,11 +36,11 @@ PROJECT (CLICKHOUSE-CLIENT)
3736
)
3837

3938
IF (BUILD_BENCHMARK)
40-
SUBDIRS(bench)
39+
SUBDIRS (bench)
4140
ENDIF (BUILD_BENCHMARK)
4241

4342
IF (BUILD_TESTS)
44-
SUBDIRS(
43+
SUBDIRS (
4544
contrib/gtest
4645
tests/simple
4746
ut

0 commit comments

Comments
 (0)