summaryrefslogtreecommitdiffstats
path: root/Tests
Commit message (Collapse)AuthorAgeFilesLines
* find_*: Add support for REQUIRED keywordSylvain Joubert2020-03-0620-0/+78
| | | | | | In the same spirit as the REQUIRED keyword on find_package, this will stop cmake execution with an error on a failed find_program, find_file, find_path or find_library.
* Merge topic 'FindPython-artifacts-scope'Brad King2020-03-063-3/+56
|\ | | | | | | | | | | | | e5b4c74238 FindPython: Add possibility to control scope of artifacts. Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4426
| * FindPython: Add possibility to control scope of artifacts.Marc Chevrier2020-03-053-3/+56
| | | | | | | | Fixes: #20362
* | Merge topic 'macos_framework_postfix'Kyle Edwards2020-03-053-0/+106
|\ \ | | | | | | | | | | | | | | | | | | ad3f69c86e Add support for FRAMEWORK_MULTI_CONFIG_POSTFIX_<CONFIG> Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4289
| * | Add support for FRAMEWORK_MULTI_CONFIG_POSTFIX_<CONFIG>Alexandru Croitor2020-03-043-0/+106
| | |
* | | Merge topic 'objlib-transitive-deps'Brad King2020-03-057-2/+57
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | a833aa1167 Fix dependencies on targets linked through object libraries 3b3de0fd17 Tests: Extend ObjectLibrary test with case for object lib dependencies Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4427
| * | | Fix dependencies on targets linked through object librariesBrad King2020-03-043-2/+33
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When an object library is used via `target_link_libraries`, any targets listed in the object library's `INTERFACE_LINK_LIBRARIES` closure should become direct dependencies of the consuming target. However, these were accidentally left out by `cmComputeTargetDepends::CollectTargetDepends` because object libraries are encountered through external object sources first and then added to the `emitted` set which blocks them from being processed as link dependencies. This was not noticed by the test case in commit bab24e782c (target_link_libraries: Propagate dependencies of object libraries, 2018-12-10, v3.14.0-rc1~260^2) because the relevant dependency appears transitively through the object library target itself. Re-order the logic to process link dependencies first, and then external object sources. That way object libraries used via `target_link_libraries` will be treated as such by dependency analysis. This also adds missing backtrace information for object libraries used via `target_link_libraries`. The missing information was mentioned in a FIXME comment in the RunCMake.FileAPI test added by commit ea0a060168 (fileapi: Add test for codemodel v2, 2018-11-09, v3.14.0-rc1~257^2~7). That comment itself was dropped by commit a0de350e2f (FileAPI test: Break gen_check_targets() into JSON files, 2020-02-07), but we can now update the corresponding location in the `.json` files to have the now-expected backtrace information. Fixes: #20421
| * | | Tests: Extend ObjectLibrary test with case for object lib dependenciesBrad King2020-03-045-0/+24
| | |/ | |/| | | | | | | | | | | | | Add the test case from commit bab24e782c (target_link_libraries: Propagate dependencies of object libraries, 2018-12-10, v3.14.0-rc1~260^2) to the main ObjectLibrary test.
* | | Merge topic 'ninja-multi-custom-command-deps'Brad King2020-03-053-6/+3
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | 081c4679f7 Ninja Multi-Config: Don't build target dependencies for custom commands Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4423
| * | | Ninja Multi-Config: Don't build target dependencies for custom commandsKyle Edwards2020-03-033-6/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | If cross-config mode is used, and a target depends on another target as well as a custom command, we don't want the custom command to also depend on the depended target, as that would build targets unnecessarily. Fix this behavior.
* | | | Merge topic 'findx11-improvement'Brad King2020-03-051-0/+11
|\ \ \ \ | |_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | 77802aab97 FindX11: Find additional libraries Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: Ben Boeckel <ben.boeckel@kitware.com> Merge-request: !4101
| * | | FindX11: Find additional librariesZsolt Parragi2020-03-041-0/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | With this change, FindLibrary(X11) learns about the following libraries: * xcb * X11-xcb * xcb-icccm * xcb-xkb * xkbcommon * xkbcommon-X11
* | | | Merge topic 'FindPython-manage-SOABI-suffix'Brad King2020-03-041-0/+10
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 0c97b73bc0 FindPython: python_add_library can now manage SOABI suffix. Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4420
| * | | | FindPython: python_add_library can now manage SOABI suffix.Marc Chevrier2020-03-031-0/+10
| | |/ / | |/| | | | | | | | | | Fixes: #20408
* | | | Merge topic 'ninja-multi-spurious-unused-variables'Brad King2020-03-042-2/+13
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 8b799f80e3 Ninja Multi-Config: Fix spurious unused variable warning Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4421
| * | | | Ninja Multi-Config: Fix spurious unused variable warningKyle Edwards2020-03-032-2/+13
| |/ / / | | | | | | | | | | | | Fixes: #20381
| * | | Merge topic 'ninja-multi-variable-shuffle-again' into release-3.17Brad King2020-02-2817-34/+70
| |\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | c794b70f19 Ninja Multi-Config: Always generate build.ninja 9590c3a400 Generator: Don't allow Ninja Multi-Config variables on other generators 7a63dafafb Ninja Multi-Config: Remove "NMC" from variable names Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4403
* | \ \ \ Merge topic 'cmake_command-eval'Brad King2020-03-0315-1/+30
|\ \ \ \ \ | |_|_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | | 598b676b5e cmake_command: Add command to EVAL a CMake script as a string Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4408
| * | | | cmake_command: Add command to EVAL a CMake script as a stringCristian Adam2020-03-0315-1/+30
| | | | |
* | | | | Merge topic 'test_output'Brad King2020-03-022-13/+14
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 1ae9ddbb9d Tests: Output the expected value in Preprocess test for debugging Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4410
| * | | | | Tests: Output the expected value in Preprocess test for debuggingFrancisco Facioni2020-02-282-13/+14
| | | | | |
* | | | | | Merge topic 'xl-fortran-module-dir-use'Brad King2020-03-021-1/+1
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 92785ed746 Tests: Enable Fortran submodule tests on XL compilers 210b0b99a9 XL: Fix using Fortran modules from their output directory Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4405
| * | | | | | Tests: Enable Fortran submodule tests on XL compilersBrad King2020-02-281-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since commit b66bc6606e (Tests: Add Fortran submodule tests, 2016-09-22, v3.7.0-rc1~55^2) we try a small test program to see if the Fortran compiler supports submodules. However, a typo in the test program caused it to fail on XL with the error: 1513-083 (E) Internal or module function id was not set within the function. Fix the typo so that the check passes and enables the submodule tests with XL compilers.
* | | | | | | Merge topic 'CheckFortranSource-def'Brad King2020-03-021-27/+23
|\ \ \ \ \ \ \ | |_|_|_|_|_|/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 1bc521416f CheckFortranSource*: Avoid passing -DVAR to linker Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4406
| * | | | | | CheckFortranSource*: Avoid passing -DVAR to linkerBrad King2020-02-281-27/+23
| |/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The test project is compiled with a `-DVAR` compiler flag where `VAR` is the result variable. Tell `try_compile` to add the flag through `add_definitions` instead of `CMAKE_Fortran_FLAGS` so that it is not used for linking. Otherwise some Fortran compilers (e.g. XL 15) do not like the flag when used to drive linking.
* | | | | | Merge topic 'csharp-out-of-source-groups'Brad King2020-02-2810-5/+77
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ac6b18cd90 CSharp: Add support for source groups with out-of-source builds Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4374
| * | | | | | CSharp: Add support for source groups with out-of-source buildsKinan Mahdi2020-02-2810-5/+77
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This also fixes support for multiple sources of the same name in different directories. Add a test for both problems. Issue: #19505
* | | | | | | Merge topic 'ninja-multi-variable-shuffle-again'Brad King2020-02-2817-34/+70
|\ \ \ \ \ \ \ | | |_|_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | c794b70f19 Ninja Multi-Config: Always generate build.ninja 9590c3a400 Generator: Don't allow Ninja Multi-Config variables on other generators 7a63dafafb Ninja Multi-Config: Remove "NMC" from variable names Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4403
| * | | | | | Ninja Multi-Config: Always generate build.ninjaKyle Edwards2020-02-274-12/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If CMAKE_DEFAULT_BUILD_TYPE is not specified, use the first item from CMAKE_CONFIGURATION_TYPES instead.
| * | | | | | Generator: Don't allow Ninja Multi-Config variables on other generatorsKyle Edwards2020-02-2710-0/+45
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We may want to enable these variables later on with specific semantics. To avoid breaking backwards compatibility, make it an error to use them for now.
| * | | | | | Ninja Multi-Config: Remove "NMC" from variable namesKyle Edwards2020-02-276-27/+27
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Also rename `..._DEFAULT_BUILD_FILE_CONFIG` to `..._DEFAULT_BUILD_TYPE`. These name changes make the variables meaningful for future use by other generators.
* | | | | | | Merge topic 'foreach-range-issues'Brad King2020-02-2822-0/+56
|\ \ \ \ \ \ \ | |_|_|_|/ / / |/| | | | | / | | |_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | 185d1aefaa foreach: Set fatal error on invalid range a33b3949e5 foreach: Fix crash when parsing invalid integer Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4407
| * | | | | foreach: Set fatal error on invalid rangeKyle Edwards2020-02-274-0/+8
| | | | | | | | | | | | | | | | | | | | | | | | Fixes: #20394
| * | | | | foreach: Fix crash when parsing invalid integerKyle Edwards2020-02-2719-0/+48
| |/ / / / | | | | | | | | | | | | | | | Fixes: #20393
| * | | | Merge topic 'RunCMake-test-lang' into release-3.17Brad King2020-02-261-0/+3
| |\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 95dbcf0598 Tests: fix RunCMake.Make test when run on systems with non-english locale Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4389
* | \ \ \ \ Merge topic 'tutorial-have-log-and-exp'Brad King2020-02-271-6/+15
|\ \ \ \ \ \ | |_|_|_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | 07223c5c27 Tutorial: Update Step 5 to work on Windows Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4375
| * | | | | Tutorial: Update Step 5 to work on WindowsBetsy McPhail2020-02-271-6/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | `log` and `exp` should be found on Windows systems now, as expected. Update tests to be more specific in looking for the expected outcome of the tutorial.
* | | | | | Merge topic 'Genex-LINK_LANGUAGE'Brad King2020-02-27170-8/+1156
|\ \ \ \ \ \ | |_|_|_|_|/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | 461efa7b51 Genex: Add $<LINK_LANGUAGE:...> and $<LINK_LANG_AND_ID:...> Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4244
| * | | | | Genex: Add $<LINK_LANGUAGE:...> and $<LINK_LANG_AND_ID:...>Marc Chevrier2020-02-26170-8/+1156
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This MR may help to solve issues #19757 and #18008 Fixes: #19965
* | | | | | Merge topic 'cmake_command-command'Brad King2020-02-2720-0/+38
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 54e4f2ad45 cmake_command: Add command to INVOKE other commands by name Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: Kyle Edwards <kyle.edwards@kitware.com> Merge-request: !4286
| * | | | | | cmake_command: Add command to INVOKE other commands by nameCristian Adam2020-02-2520-0/+38
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes: #18392
* | | | | | | Merge topic 'execute_process_exclusive_capture'Brad King2020-02-275-0/+31
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 6ec274b002 execute_process: Add ECHO_(OUTPUT|ERROR)_VARIABLE options Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: Michael Hirsch, Ph.D. <michael@scivision.dev> Merge-request: !4383
| * | | | | | | execute_process: Add ECHO_(OUTPUT|ERROR)_VARIABLE optionsCristian Adam2020-02-245-0/+31
| | |_|/ / / / | |/| | | | | | | | | | | | | | | | | | | Fixes: #20378
* | | | | | | Merge topic 'FindSWIG-components'Marc Chevrier2020-02-278-0/+28
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | e131d9f974 FindSWIG: Add COMPONENTS support for SWIG target languages Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4384
| * | | | | | | FindSWIG: Add COMPONENTS support for SWIG target languagesSeth R Johnson2020-02-268-0/+28
| |/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Newer versions of SWIG drop support for some target languages, and some forks of SWIG (such as for Fortran and MATLAB) aren't supported by the mainline version of SWIG. Swig versions as old as 1.3.6 (circa 2001) and possibly older use the same format for listing available wrappers "%-15s - Generate %s wrappers", so component detection should be quite reliable.
* | | | | | | Merge topic 'vs-sln-deploy'Brad King2020-02-263-0/+71
|\ \ \ \ \ \ \ | |_|_|/ / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 7c944da757 VS: Add target property to explicitly control solution deployment Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4357
| * | | | | | VS: Add target property to explicitly control solution deploymentAlexander Boczar2020-02-253-0/+71
| | |_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | Add a `VS_SOLUTION_DEPLOY` property to control solution deploy mark. Fixes: #20346
* | | | | | Merge topic 'CPackRPM-trans-scripts'Brad King2020-02-262-2/+18
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | c0534c4a68 CPackRPM: Add PRE_/POST_TRANS scripts Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4387
| * | | | | | CPackRPM: Add PRE_/POST_TRANS scriptsSarang Joshi2020-02-242-2/+18
| |/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add variables for PRE_TRANS and POST_TRANS scripts. Fixes: #18917
* | | | | | Merge topic 'RunCMake-test-lang'Brad King2020-02-261-0/+3
|\ \ \ \ \ \ | | |_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | 95dbcf0598 Tests: fix RunCMake.Make test when run on systems with non-english locale Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4389