kolan
/
CMake
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

prefer list(APPEND) over string(APPEND) where appropriate

This commit is contained in:
Daniel Pfeifer 2016-08-08 21:25:41 +02:00
parent ee9d4feb67
commit 3a7be4f394
5 changed files with 8 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
Modules/ExternalProject.cmake
View File

@ -1133,7 +1133,7 @@ function(_ep_command_line_to_initial_cache var args force)
endif() endif()
else() else()
# Assume this is a list to append to the last var # Assume this is a list to append to the last var
string(APPEND accumulator ";${line}") list(APPEND accumulator "${line}")
endif() endif()
endforeach() endforeach()
# Catch the final line of the args # Catch the final line of the args

2
Modules/FindQt4.cmake
View File

@ -764,7 +764,7 @@ if (QT_QMAKE_EXECUTABLE AND
############################################# #############################################
cmake_push_check_state() cmake_push_check_state()
# Add QT_INCLUDE_DIR to CMAKE_REQUIRED_INCLUDES # Add QT_INCLUDE_DIR to CMAKE_REQUIRED_INCLUDES
string(APPEND CMAKE_REQUIRED_INCLUDES ";${QT_INCLUDE_DIR}") list(APPEND CMAKE_REQUIRED_INCLUDES "${QT_INCLUDE_DIR}")
set(CMAKE_REQUIRED_QUIET ${Qt4_FIND_QUIETLY}) set(CMAKE_REQUIRED_QUIET ${Qt4_FIND_QUIETLY})
# Check for Window system symbols (note: only one should end up being set) # Check for Window system symbols (note: only one should end up being set)
CHECK_CXX_SYMBOL_EXISTS(Q_WS_X11 "QtCore/qglobal.h" Q_WS_X11) CHECK_CXX_SYMBOL_EXISTS(Q_WS_X11 "QtCore/qglobal.h" Q_WS_X11)

4
Tests/RunCMake/CPack/VerifyResult.cmake
View File

@ -16,7 +16,7 @@ include("${src_dir}/${GENERATOR_TYPE}/${RunCMake_TEST}-ExpectedFiles.cmake")
if(NOT EXPECTED_FILES_COUNT EQUAL 0) if(NOT EXPECTED_FILES_COUNT EQUAL 0)
foreach(file_no_ RANGE 1 ${EXPECTED_FILES_COUNT}) foreach(file_no_ RANGE 1 ${EXPECTED_FILES_COUNT})
file(GLOB FOUND_FILE_${file_no_} RELATIVE "${bin_dir}" "${EXPECTED_FILE_${file_no_}}") file(GLOB FOUND_FILE_${file_no_} RELATIVE "${bin_dir}" "${EXPECTED_FILE_${file_no_}}")
string(APPEND foundFiles_ ";${FOUND_FILE_${file_no_}}") list(APPEND foundFiles_ "${FOUND_FILE_${file_no_}}")
list(LENGTH FOUND_FILE_${file_no_} foundFilesCount_) list(LENGTH FOUND_FILE_${file_no_} foundFilesCount_)
if(foundFilesCount_ EQUAL 1) if(foundFilesCount_ EQUAL 1)
@ -45,7 +45,7 @@ if(NOT EXPECTED_FILES_COUNT EQUAL 0)
# check that there were no extra files generated # check that there were no extra files generated
foreach(all_files_glob_ IN LISTS ALL_FILES_GLOB) foreach(all_files_glob_ IN LISTS ALL_FILES_GLOB)
file(GLOB foundAll_ RELATIVE "${bin_dir}" "${all_files_glob_}") file(GLOB foundAll_ RELATIVE "${bin_dir}" "${all_files_glob_}")
string(APPEND allFoundFiles_ ";${foundAll_}") list(APPEND allFoundFiles_ "${foundAll_}")
endforeach() endforeach()
list(LENGTH foundFiles_ foundFilesCount_) list(LENGTH foundFiles_ foundFilesCount_)

4
Tests/SimpleInstall/CMakeLists.txt
View File

@ -348,7 +348,7 @@ if(UNIX AND NOT APPLE)
# if(NOT SKIP_TZ) # if(NOT SKIP_TZ)
# message("compress found and it was not a script") # message("compress found and it was not a script")
# message("output from file command: [${output}]") # message("output from file command: [${output}]")
# string(APPEND CPACK_GENERATOR ";TZ") # list(APPEND CPACK_GENERATOR "TZ")
# else() # else()
# message("compress found, but it was a script so dont use it") # message("compress found, but it was a script so dont use it")
# message("output from file command: [${output}]") # message("output from file command: [${output}]")
@ -357,7 +357,7 @@ if(UNIX AND NOT APPLE)
find_program(found_bz2 find_program(found_bz2
NAMES bzip2) NAMES bzip2)
if(found_bz2) if(found_bz2)
string(APPEND CPACK_GENERATOR ";TBZ2") list(APPEND CPACK_GENERATOR "TBZ2")
endif() endif()
endif() endif()

4
Tests/SimpleInstallS2/CMakeLists.txt
View File

@ -348,7 +348,7 @@ if(UNIX AND NOT APPLE)
# if(NOT SKIP_TZ) # if(NOT SKIP_TZ)
# message("compress found and it was not a script") # message("compress found and it was not a script")
# message("output from file command: [${output}]") # message("output from file command: [${output}]")
# string(APPEND CPACK_GENERATOR ";TZ") # list(APPEND CPACK_GENERATOR "TZ")
# else() # else()
# message("compress found, but it was a script so dont use it") # message("compress found, but it was a script so dont use it")
# message("output from file command: [${output}]") # message("output from file command: [${output}]")
@ -357,7 +357,7 @@ if(UNIX AND NOT APPLE)
find_program(found_bz2 find_program(found_bz2
NAMES bzip2) NAMES bzip2)
if(found_bz2) if(found_bz2)
string(APPEND CPACK_GENERATOR ";TBZ2") list(APPEND CPACK_GENERATOR "TBZ2")
endif() endif()
endif() endif()