Skip to content

Commit b9417f7

Browse files
committed
Move UPPERCASE_BUILD_TYPE definition and also rebase w/ master
1 parent ad7f882 commit b9417f7

File tree

2 files changed

+1
-3
lines changed

2 files changed

+1
-3
lines changed

cpp/CMakeLists.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,6 +28,7 @@ if(NOT GENERATOR_IS_MULTI_CONFIG AND NOT DEFINED CMAKE_BUILD_TYPE)
2828
set(CMAKE_BUILD_TYPE Release CACHE STRING "Choose the type of build.")
2929
endif()
3030

31+
string(TOUPPER ${CMAKE_BUILD_TYPE} UPPERCASE_BUILD_TYPE)
3132
project(arrow VERSION "${ARROW_BASE_VERSION}")
3233

3334
set(ARROW_VERSION_MAJOR "${arrow_VERSION_MAJOR}")
@@ -398,7 +399,6 @@ set(EXECUTABLE_OUTPUT_PATH "${BUILD_OUTPUT_ROOT_DIRECTORY}")
398399
if(CMAKE_GENERATOR STREQUAL Xcode)
399400
# Xcode projects support multi-configuration builds. This forces a single output directory
400401
# when building with Xcode that is consistent with single-configuration Makefile driven build.
401-
string(TOUPPER ${CMAKE_BUILD_TYPE} UPPERCASE_BUILD_TYPE)
402402
set(CMAKE_ARCHIVE_OUTPUT_DIRECTORY_${UPPERCASE_BUILD_TYPE}
403403
"${BUILD_OUTPUT_ROOT_DIRECTORY}")
404404
set(CMAKE_LIBRARY_OUTPUT_DIRECTORY_${UPPERCASE_BUILD_TYPE}

cpp/cmake_modules/ThirdpartyToolchain.cmake

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -417,8 +417,6 @@ endif()
417417
# ----------------------------------------------------------------------
418418
# ExternalProject options
419419

420-
string(TOUPPER ${CMAKE_BUILD_TYPE} UPPERCASE_BUILD_TYPE)
421-
422420
set(EP_CXX_FLAGS "${CMAKE_CXX_FLAGS} ${CMAKE_CXX_FLAGS_${UPPERCASE_BUILD_TYPE}}")
423421
set(EP_C_FLAGS "${CMAKE_C_FLAGS} ${CMAKE_C_FLAGS_${UPPERCASE_BUILD_TYPE}}")
424422

0 commit comments

Comments
 (0)