summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Merge topic 'fix-co-compile'Brad King2017-10-3016-87/+175
|\ | | | | | | | | | | | | | | | | | | 197b4e18 Merge branch 'backport-fix-co-compile' into fix-co-compile 992962c7 cmcmd: Restore support for running multiple lint tools a5197eea cmcmd: Convert lint handlers to file-static functions 1c075ffd cmcmd: Rename loop iteration variable for clarity Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1418
| * Merge branch 'backport-fix-co-compile' into fix-co-compileBrad King2017-10-2716-87/+175
| |\ | | | | | | | | | | | | Resolve a logical conflict by replacing `cmArray{Begin,End}` from the their side with `cm::{cbegin,cend}` from our side.
| | * cmcmd: Restore support for running multiple lint toolsBrad King2017-10-2715-61/+166
| | | | | | | | | | | | | | | | | | | | | Refactoring in commit v3.10.0-rc1~115^2 (Clean up iwyu code to not be one big if statement, 2017-08-28) incorrectly changed the logic to run only one lint tool at a time. Restore support for running all tools specified on the command-line.
| | * cmcmd: Convert lint handlers to file-static functionsBrad King2017-10-272-29/+12
| | | | | | | | | | | | These do not need to be declared in the header.
| | * cmcmd: Rename loop iteration variable for clarityBrad King2017-10-271-2/+2
| | |
* | | Merge topic 'autogen-static-library-cycles'Brad King2017-10-3010-17/+172
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | 3a4db861 Autogen: Tests: Add test for STATIC_LIBRARY cycles 798e4f2f Autogen: Don't add STATIC_LIBRARY cycle targets to the _autogen dependencies Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1408
| * | | Autogen: Tests: Add test for STATIC_LIBRARY cyclesSebastian Holtermann2017-10-279-0/+89
| | | |
| * | | Autogen: Don't add STATIC_LIBRARY cycle targets to the _autogen dependenciesSebastian Holtermann2017-10-271-17/+83
| | | | | | | | | | | | | | | | | | | | | | | | | | | | When a STATIC_LIBRARY cycle is detected we don't add any STATIC_LIBRARY target from the cycle to the `_autogen` target dependencies. Closes #17389
* | | | CMake Nightly Date StampKitware Robot2017-10-301-1/+1
| | | |
* | | | CMake Nightly Date StampKitware Robot2017-10-291-1/+1
| | | |
* | | | CMake Nightly Date StampKitware Robot2017-10-281-1/+1
| | | |
* | | | Merge topic 'generalize-importedtargets-behavior'Brad King2017-10-2740-129/+236
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | fb88d2b5 Help: Add notes for topic 'generalize-importedtargets-behavior' 494906a8 Add support for IMPORTED GLOBAL targets to be aliased fe4b25ec Teach target_* commands to set INTERFACE properties of IMPORTED targets e40fd9fd cmTargetPropCommandBase: Fix typo in error message Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: Henry Schreiner <henryschreineriii@gmail.com> Merge-request: !1264
| * | | | Help: Add notes for topic 'generalize-importedtargets-behavior'Brad King2017-10-261-0/+25
| | | | | | | | | | | | | | | | | | | | Inspired-by: Deniz Bahadir <dbahadir@benocs.com>
| * | | | Add support for IMPORTED GLOBAL targets to be aliasedDeniz Bahadir2017-10-269-22/+109
| | | | | | | | | | | | | | | | | | | | | | | | | Issue: #15569 Issue: #17197
| * | | | Teach target_* commands to set INTERFACE properties of IMPORTED targetsDeniz Bahadir2017-10-2628-95/+91
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Now, several `INTERFACE_*` properties can be set on `IMPORTED` targets, not only via `set_property` and `set_target_properties` but also via `target_compile_definitions`, `target_compile_features`, `target_compile_options`, `target_include_directories`, `target_sources` and `target_link_libraries`. Fixes: #15689 Issue: #17197
| * | | | cmTargetPropCommandBase: Fix typo in error messageDeniz Bahadir2017-10-263-12/+11
| | | | |
* | | | | Merge topic 'findICU_update'Brad King2017-10-271-1/+20
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 20696a53 FindICU: Add support for static libraries on Windows 606ece67 FindICU: Search in 'sbin' directories for icu tools Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1396
| * | | | | FindICU: Add support for static libraries on WindowsFrancois Budin2017-10-251-0/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When compiling ICU statically on Windows with Visual Studio, the library file names are prefixed with 's' [1]. [1] http://source.icu-project.org/repos/icu/trunk/icu4c/source/icudefs.mk.in `STATIC_PREFIX_WHEN_USED = s`
| * | | | | FindICU: Search in 'sbin' directories for icu toolsFrancois Budin2017-10-251-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | On Ubuntu, several binaries that FindICU.cmake searches for are installed in 'sbin' instead of 'bin' [1]. [1] https://packages.ubuntu.com/xenial/amd64/icu-devtools/filelist
* | | | | | Merge topic 'cmdparser-strcpy'Brad King2017-10-274-49/+51
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 576c079f cmCommandArgumentParser: remove the variable tokens from the class 98cb9f3e cmCommandArgumentParserHelper: constify some char* values Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1410
| * | | | | | cmCommandArgumentParser: remove the variable tokens from the classRolf Eike Beer2017-10-254-32/+32
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Make them local constants in the lexer file.
| * | | | | | cmCommandArgumentParserHelper: constify some char* valuesRolf Eike Beer2017-10-252-17/+19
| | |/ / / / | |/| | | |
* | | | | | Merge topic 'cmp0040-wording'Brad King2017-10-273-5/+5
|\ \ \ \ \ \ | |_|_|_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | a1b1f1a2 CMP0040: Clarify policy warning to match documentation Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1415
| * | | | | CMP0040: Clarify policy warning to match documentationBrad King2017-10-263-5/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In commit v3.5.0-rc1~8^2~2 (Help: Clarify policy `CMP0040` documentation, 2016-01-28) the documentation was clarified to indicate that the target must be defined in the current directory. Do the same for the text of the policy warning itself. Fixes: #17399
* | | | | | CMake Nightly Date StampKitware Robot2017-10-271-1/+1
| | | | | |
* | | | | | Merge branch 'release-3.10'Brad King2017-10-260-0/+0
|\ \ \ \ \ \
| * \ \ \ \ \ Merge branch 'clang-cl-TP' into release-3.10Brad King2017-10-252-0/+2
| |\ \ \ \ \ \ | | |_|_|_|/ / | |/| | | | | | | | | | | | Merge-request: !1407
* | | | | | | Merge topic 'cm-array-begins-size'Brad King2017-10-269-105/+147
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 57132765 Replace cmArray{Begin,End,Size} by their standard counterparts Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1352
| * | | | | | | Replace cmArray{Begin,End,Size} by their standard counterpartsMatthias Maennich2017-10-239-105/+147
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | std::{begin,end} are part of C++11, std::{cbegin,cend} are part of C++14 and an standard compliant implementation has been introduced within the 'cm' namespace: cm::{cbegin,cend}. std::size is only part of C++17, hence exposing a compliant implementation within namespace cm (cm::size). where possible, the standard implementations are reused.
* | | | | | | | Merge topic 'clang-cl-TP'Brad King2017-10-262-0/+2
|\ \ \ \ \ \ \ \ | | |_|/ / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 7077a554 Clang: Use -TP flag for C++ sources with clang-cl Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1407
| * | | | | | | Clang: Use -TP flag for C++ sources with clang-clBrad King2017-10-252-0/+2
| | |_|/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We do this for `cl`, so we should do it for `clang-cl`. Fixes: #17394
* | | | | | | CMake Nightly Date StampKitware Robot2017-10-261-1/+1
| |_|_|/ / / |/| | | | |
* | | | | | Merge branch 'release-3.10'Brad King2017-10-250-0/+0
|\ \ \ \ \ \ | | |_|/ / / | |/| | | |
| * | | | | Merge branch 'omp-oacc-werror-return-type' into release-3.10Brad King2017-10-242-3/+11
| |\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | Merge-request: !1406
| * \ \ \ \ \ Merge branch 'findmpi-core-count' into release-3.10Brad King2017-10-241-7/+2
| |\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Merge-request: !1405
* | \ \ \ \ \ \ Merge topic 'server-codemodel-install-dest'Brad King2017-10-254-1/+45
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 296eb9ad cmake-server: Add target install destinations to codemodel Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1382
| * | | | | | | | cmake-server: Add target install destinations to codemodelJustin Goshi2017-10-244-1/+45
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Protocol version is updated to 1.2 since this is a change to what was released in cmake version 3.10.
* | | | | | | | | Merge topic 'flang-windows'Brad King2017-10-251-0/+2
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | b2c6c357 Flang: Add partial support on Windows Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1404
| * | | | | | | | | Flang: Add partial support on WindowsIsuru Fernando2017-10-241-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add minimal changes needed to help Flang itself build on Windows using CMake. Additional work will be required for full support. Issue: #17384
* | | | | | | | | | Merge topic 'windows_symlinks'Brad King2017-10-254-7/+48
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 393e0fbe cmTimestamp: For symlinks switch to timestamp of resolved path 640709e7 cmSystemTools: Implement GetRealPath on Windows d41582fc Call GetRealPath through cmSystemTools instead of KWSys directly Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1202
| * | | | | | | | | | cmTimestamp: For symlinks switch to timestamp of resolved pathManuel Núñez2017-10-241-2/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ModifiedTime uses stat on UNIX which does resolve symlinks, however Windows implementation relies on GetFileAttributesExW which does not. Getting real file path before calling ModifiedTime will not change UNIX semantic and will fix Windows behavior. Fixes: #17206
| * | | | | | | | | | cmSystemTools: Implement GetRealPath on WindowsManuel Núñez2017-10-242-0/+39
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Override the KWSys GetRealPath on Windows and use uv_fs_realpath first to resolve symbolic links. Fixes: #17206
| * | | | | | | | | | Call GetRealPath through cmSystemTools instead of KWSys directlyManuel Núñez2017-10-241-5/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This will allow a wrapper to be added to extend functionality.
* | | | | | | | | | | Merge topic 'omp-oacc-werror-return-type'Brad King2017-10-252-3/+11
|\ \ \ \ \ \ \ \ \ \ \ | | |_|_|_|_|/ / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 15da0ba3 Find{OpenMP,OpenACC}: Fix detection with -Werror=return-type Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1406
| * | | | | | | | | | Find{OpenMP,OpenACC}: Fix detection with -Werror=return-typeChristian Pfeiffer2017-10-242-3/+11
| | |_|_|_|_|/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Explicitly return a value from `main` in our test sources. Fixes: #17391
* | | | | | | | | | Merge topic 'findmpi-core-count'Brad King2017-10-251-7/+2
|\ \ \ \ \ \ \ \ \ \ | | |_|_|_|_|/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 1610f757 FindMPI: Use physical cores for MPIEXEC_MAX_NUMPROCS Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1405
| * | | | | | | | | FindMPI: Use physical cores for MPIEXEC_MAX_NUMPROCSChristian Pfeiffer2017-10-241-7/+2
| |/ / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since commit v3.9.0-rc1~224^2~1 (FindMPI: MPIEXEC handling improvements, 2017-04-21) the `ProcessorCount` module is being used to initialize `MPIEXEC_MAX_NUMPROCS`. However, this leads to the logical cores being counted rather than the physical ones, and some MPI implementations like OpenMPI will error if mpiexec is called with that number. Switch it to the number of physical cores using `cmake_host_system_information`. This ensures that if `MPIEXEC_MAX_NUMPROCS` is being used to set up MPI tests with CTest or similar that the tests won't spuriously fail due to OpenMPI refusing to start the application.
* | | | | | | | | Merge topic 'defer-target-source-check'Brad King2017-10-2525-76/+100
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 4e7f6738 Defer check for sources within a target until generation. 6e4e7c65 Tests: Exclude bad RunCMake.add_executable case on multi-arch Xcode Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1242
| * | | | | | | | | Defer check for sources within a target until generation.Deniz Bahadir2017-10-2424-75/+97
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The `add_library` and `add_executable` commands can now be called with no source-files and won't generate a warning or error message, as long as source-files will be added later via the `target_sources` command. If during the generation step still no sources are associated with targets created by such calls a useful error message will be generated and generation fails. Targets of type `INTERFACE_LIBRARY`, `UTILITY` or `GLOBAL_TARGET` are excluded from this check because we do not need sources for these target types during generation. Fixes: #16872
| * | | | | | | | | Tests: Exclude bad RunCMake.add_executable case on multi-arch XcodeBrad King2017-10-241-1/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The `NoSourcesButLinkObjects` case would not be expected to work under Xcode with multiple architectures even if the target objects were listed directly as sources. Exclude it. We already exclude similar cases in `RunCMake.add_library`.