summaryrefslogtreecommitdiffstats
path: root/Tests
Commit message (Collapse)AuthorAgeFilesLines
* Merge topic 'source_file_scopes'Brad King2020-05-1510-0/+274
|\ | | | | | | | | | | | | 3d4b70ea64 set_source_files_properties: Allow specification of directory scope Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4661
| * set_source_files_properties: Allow specification of directory scopeAlexandru Croitor2020-05-1410-0/+274
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Both set_source_files_properties() and set_property(SOURCE) now accept two new optional arguments: DIRECTORY and TARGET_DIRECTORY. The DIRECTORY option takes a list of relative or absolute paths pointing to processed source directories (add_subdirectory was already called on them). These paths specify directory scopes where the source file properties will be set. Previously the scope was always the currently processed source directory. Similarly TARGET_DIRECTORY takes a list of targets, whose source directories will be used as the list of scopes where to set the source file properties. get_property() and get_source_file_property() also get the same new arguments, except only one value can be specified instead of a list. Fixes: #20128
* | Merge topic 'tests-Wstrict-prototypes'Brad King2020-05-143-3/+3
|\ \ | | | | | | | | | | | | | | | | | | 4150a18910 Tests: Fix -Wstrict-prototypes warnings in some C sources Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4744
| * | Tests: Fix -Wstrict-prototypes warnings in some C sourcesBrad King2020-05-133-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | Some RunCMake tests fail with this warning due to extra stderr content: warning: this old-style function definition is not preceded by a prototype Convert `foo()` to `foo(void)` in `.c` sources of affected tests.
* | | Merge topic 'cmake_command_preserve_args'Brad King2020-05-143-0/+19
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | 549599bf32 cmake_command: Preserve arguments to INVOKE function Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4741
| * | | cmake_command: Preserve arguments to INVOKE functionCristian Adam2020-05-123-0/+19
| | |/ | |/| | | | | | | Fixes: #20630
* | | Merge topic 'interface-sources-multi-config'Brad King2020-05-132-1/+46
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 6c5d4522bc INTERFACE_SOURCES: Fix per-config link libs on multi-config generators 8daa140c6a cmGeneratorTarget: Factor evaluated target prop entries into struct fcd1a1a920 cmGeneratorTarget: Track when the set of link libs is config-dependent Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4740
| * | | INTERFACE_SOURCES: Fix per-config link libs on multi-config generatorsBrad King2020-05-122-1/+46
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In multi-config generators we memoize the computed set of source files for a target to avoid repeating the computation when the set does not depend on the configuration. We already track whether generator expressions in `SOURCES` or `INTERFACE_SOURCES` reference the configuration (`$<CONFIG:...>`). However, we previously forgot to track whether the set of libraries whose `INTERFACE_SOURCES` are considered depends on the configuration. This caused multi-config generators to use the first configuration's set of sources for all configurations in cases such as target_link_libraries(tgt PRIVATE $<$<CONFIG:Debug>:iface_debug>) where the `iface_debug` target has `INTERFACE_SOURCES`. Fix this by also tracking config-dependence of the list of libraries for evaluation of the list of source files. Fixes: #20683
* | | Merge topic 'fix-cache-args-file-to-dep-list'Brad King2020-05-137-0/+48
|\ \ \ | |_|/ |/| | | | | | | | | | | | | | 0199dd9f1b ExternalProject: expose _ep_cache_args_script to the caller Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4710
| * | ExternalProject: expose _ep_cache_args_script to the callerBen Boeckel2020-05-127-0/+48
| | | | | | | | | | | | | | | | | | | | | This is needed so that the caller can add a dependency on the cache file. Fixes: #20668
* | | Initialize CMAKE_EXPORT_COMPILE_COMMANDS only if not set alreadyBrad King2020-05-127-0/+17
| |/ |/| | | | | | | | | Also de-duplicate its initialization code. Fixes: #16588
* | Merge topic 'vs-pch-compile-opts'Brad King2020-05-121-0/+3
|\ \ | | | | | | | | | | | | | | | | | | 6b2fb4ffd2 VS: Fix using PCH from source with COMPILE_OPTIONS Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4734
| * | VS: Fix using PCH from source with COMPILE_OPTIONSBrad King2020-05-111-0/+3
| | | | | | | | | | | | | | | | | | | | | If a source file gets per-source flags from both PCH and custom `COMPILE_OPTIONS`, combine them correctly. Fixes: #20694, #20456
* | | Merge topic 'doxygen-generated-sources'Brad King2020-05-111-5/+24
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | 5ec9690802 FindDoxygen: Allow generated files with USE_STAMP_FILE 683abe4366 FindDoxygen: Formatting-only indentation cleanup for test case Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4726
| * | | FindDoxygen: Allow generated files with USE_STAMP_FILECraig Scott2020-05-091-1/+20
| | | | | | | | | | | | | | | | | | | | Fixes: #20597
| * | | FindDoxygen: Formatting-only indentation cleanup for test caseCraig Scott2020-05-091-4/+4
| | | |
* | | | Merge topic 'source_group_forward_slashes'Brad King2020-05-113-1/+13
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | faf44a8cdb source_group: Support forward slashes in group hierarchy Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4727
| * | | | source_group: Support forward slashes in group hierarchyCraig Scott2020-05-103-1/+13
| |/ / / | | | | | | | | Fixes: #18076
* | | | Merge topic 'unity_explicit_groups'Brad King2020-05-1120-0/+389
|\ \ \ \ | |/ / / |/| | | | | | | | | | | | | | | | | | | | | | | 9f4eb352fe Unity Builds: Support explicit specification of sources to groups b00585adcc Unity: Refactor implementation to make it easier to extend Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4716
| * | | Unity Builds: Support explicit specification of sources to groupsRobert Maynard2020-05-0720-0/+389
| | | | | | | | | | | | | | | | | | | | Instead of having CMake determine which files should go into each unity file, the user can now use explicitly state the mapping.
* | | | Merge topic 'third-parties-layout'Brad King2020-05-086-6/+5
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 2faa3f6c55 Refactoring: Third-parties public headers are under cm3p prefix 75e87e3db4 bootstrap: update list of problematic files Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4705
| * | | | Refactoring: Third-parties public headers are under cm3p prefixMarc Chevrier2020-05-076-6/+5
| |/ / / | | | | | | | | | | | | Fixes: #20666
* | | | Merge topic 'docker-rel-win'Brad King2020-05-081-5/+0
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 7670ba8b0a Utilities/Release: Drop win{32,64} scripts in favor of docker build ff929badb3 Utilities/Release: Add docker specs to build and test Windows binaries 5f4f7e637a Utilities/Release: Generalize OS in README docker instructions 13467f1dbd Utilities/Release: Make README docker examples more readable 593c3fa1d4 Utilities/Release: Remove unused linux/x86_64/test/cache-ninja.txt Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4719
| * | | | Utilities/Release: Drop win{32,64} scripts in favor of docker buildBrad King2020-05-051-5/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Also drop the `CMakeNightlyWin{32,64}` tests that use the scripts. Follow the pattern from commit 689fdbfc61 (Utilities/Release: Drop linux64 script in favor of docker build, 2019-08-27, v3.16.0-rc1~184^2).
* | | | | Merge topic 'FindPython-check-specified-version'Brad King2020-05-082-0/+149
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 71229f2c85 Merge branch 'backport-3.17-FindPython-check-specified-version' into FindPython-check-specified-version e1c869906c FindPython: ensure any specified version is correctly handled 25978c7a00 FindPython: ensure any specified version is correctly handled Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4718
| * \ \ \ \ Merge branch 'backport-3.17-FindPython-check-specified-version' into ↵Marc Chevrier2020-05-072-0/+149
| |\ \ \ \ \ | | |_|/ / / | |/| | | | | | | | | | FindPython-check-specified-version
| | * | | | FindPython: ensure any specified version is correctly handledMarc Chevrier2020-05-072-0/+149
| | | | | | | | | | | | | | | | | | | | | | | | Fixes: #20674
| | * | | | Merge topic 'FindPkgConfig-isystem' into release-3.17Brad King2020-05-061-1/+21
| | |\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | e9d93c3731 Help: Add 3.17 release note for FindPkgConfig '-isystem' fix dfaaae67a3 Merge branch 'backport-3.16-FindPkgConfig-isystem' 4d446c68d1 FindPkgConfig: also handle "-isystem" prefixes for include directories Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4693
| | | * \ \ \ Merge branch 'backport-3.16-FindPkgConfig-isystem'Brad King2020-05-061-1/+21
| | | |\ \ \ \ | | | | |_|_|/ | | | |/| | |
| | | | * | | FindPkgConfig: also handle "-isystem" prefixes for include directoriesRolf Eike Beer2020-05-051-1/+21
| | | | | |/ | | | | |/| | | | | | | | | | | | | Fixes: #20652
| | * | | | Merge topic 'CheckLanguage-cuda-host' into release-3.17Brad King2020-05-061-9/+9
| | |\ \ \ \ | | | |/ / / | | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 5b304a7503 CheckLanguage: Fix forwarding of CMAKE_CUDA_HOST_COMPILER Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: Robert Maynard <robert.maynard@kitware.com> Merge-request: !4711
| | * | | | Merge topic 'pch-genex-absolute' into release-3.17Brad King2020-04-284-2/+6
| | |\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | b204bae261 target_precompile_headers: Fix documented example using genex Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: Cristian Adam <cristian.adam@gmail.com> Merge-request: !4678
| | * \ \ \ \ Merge topic 'FindPython-fix-python-compiler-validation' into release-3.17Brad King2020-04-244-4/+113
| | |\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | f39da773ee FindPython: fix python compiler validation Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4655
* | | \ \ \ \ \ Merge topic 'script-mode-and-arbitrary-args'Craig Scott2020-05-072-0/+4
|\ \ \ \ \ \ \ \ | |/ / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | e4f1b301fe cmake: Allow arbitrary args passed to CMake script Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4707
| * | | | | | | cmake: Allow arbitrary args passed to CMake scriptAlex Turbov2020-05-062-0/+4
| | | | | | | |
* | | | | | | | Merge topic 'gtest-space-in-parameter'Brad King2020-05-063-36/+52
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 839a1010a3 GoogleTestAddTests: Fix output processing Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4694
| * | | | | | | | GoogleTestAddTests: Fix output processingStefan Floeren2020-05-053-36/+52
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The function gtest_discover_tests calls the passed test executable with the parameter --gtest_list_tests and parses the output to find all tests. In case of value-parameterized tests ([1]), the test values are included in the output. While test names are alphanumeric, the values can contain arbitrary content. First, the output is separated into lines with `foreach`. Included semi-colons breaks this and need to get escaped. Afterwards, the testname is passed on to the `add_command` helper. This helper was converted into a macro in commit dac201442d (GoogleTest: Optimize gtest_discover_tests, 2020-02-18). As a macro, its arguments are re-evaluated. Therefore we need to escape `\`, `;` and to prevent unwanted variable expansion `$`. Fixes: #20661 [1] <https://github.com/google/googletest/blob/0eea2e9/googletest/docs/advanced.md#value-parameterized-tests>
* | | | | | | | | Merge topic 'FindPkgConfig-isystem'Brad King2020-05-061-1/+21
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 502e562940 Merge branch 'backport-3.17-FindPkgConfig-isystem' into FindPkgConfig-isystem e9d93c3731 Help: Add 3.17 release note for FindPkgConfig '-isystem' fix dfaaae67a3 Merge branch 'backport-3.16-FindPkgConfig-isystem' cd31a8acef FindPkgConfig: also handle "-isystem" prefixes for include directories 4d446c68d1 FindPkgConfig: also handle "-isystem" prefixes for include directories Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4693
| * | | | | | | | | FindPkgConfig: also handle "-isystem" prefixes for include directoriesRolf Eike Beer2020-05-051-1/+21
| | |_|_|_|_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | Fixes: #20652
* | | | | | | | | Merge topic 'CheckLanguage-cuda-host'Brad King2020-05-061-9/+9
|\ \ \ \ \ \ \ \ \ | |/ / / / / / / / |/| | | | | | / / | | |_|_|_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 5b304a7503 CheckLanguage: Fix forwarding of CMAKE_CUDA_HOST_COMPILER Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: Robert Maynard <robert.maynard@kitware.com> Merge-request: !4711
| * | | | | | | CheckLanguage: Fix forwarding of CMAKE_CUDA_HOST_COMPILERBrad King2020-05-051-9/+9
| | |_|_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix the condition added by commit fada8cbfd6 (CheckLanguage: Report CMAKE_CUDA_HOST_COMPILER if needed for compilation, 2019-05-31, v3.15.0-rc1~12^2) to activate CUDA-specific logic. The old condition had worked in our test suite only by accident because the loop variable used in the test happened to be the name and value that the old condition used! Update the test to use a different name. Fixes: #19013
* | | | | | | Merge topic 'remove_cli__symbols'Brad King2020-05-053-2/+34
|\ \ \ \ \ \ \ | |_|/ / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | be75622e49 bindexplib: Do not export symbols from managed code Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4692
| * | | | | | bindexplib: Do not export symbols from managed codeMarkus Mayer2020-05-043-2/+34
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes: #20653
* | | | | | | FindPython: Add support for 'PyPy'Marc Chevrier2020-05-015-1/+217
| |_|/ / / / |/| | | | |
* | | | | | Merge topic 'FPHSA-handle-components'Brad King2020-05-016-0/+63
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 0b6332af60 FPHSA: REQUIRED_VARS is optional if HANDLE_COMPONENTS is specified Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4690
| * | | | | | FPHSA: REQUIRED_VARS is optional if HANDLE_COMPONENTS is specifiedMarc Chevrier2020-04-306-0/+63
| |/ / / / / | | | | | | | | | | | | | | | | | | Fixes: #20655
* | | | | | Merge topic 'CTestCoverageCollectGCOV-compress-opts'Brad King2020-05-012-13/+136
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 40ca6ef125 CTestCoverageCollectGCOV: Add TARBALL_COMPRESSION option 81e83510a0 Tests: Fix re-running CTestCoverageCollectGCOV on in-source build ab7eda2591 Tests: Remove unnecessary pass regex on CTestCoverageCollectGCOV Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4649
| * | | | | | CTestCoverageCollectGCOV: Add TARBALL_COMPRESSION optionKelly (KT) Thompson2020-04-301-8/+136
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | For large projects, the default bzip2 compression of gcov data is prohibitively expensively (could take several hours). Introduce options that allow the consumer to choose between file size and compression time. Add a new optional argument `TARBALL_COMPRESSION` for the macro `ctest_coverage_collect_gcov`. This option accepts the values `GZIP`, `BZIP2`, `XZ`, `FROM_EXT`, or an expression that evaluates to `FALSE`. The default value is `BZIP2` to preserve prior behavior. Fixes: #20593
| * | | | | | Tests: Fix re-running CTestCoverageCollectGCOV on in-source buildBrad King2020-04-301-2/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Do not remove the test binary directory if it contains `CMakeCache.txt`. Otherwise in an in-source build we remove the source directory too.
| * | | | | | Tests: Remove unnecessary pass regex on CTestCoverageCollectGCOVBrad King2020-04-291-3/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The test script uses `message(FATAL_ERROR)` on failure, so we can just let the `ctest` exit code determine the result.