summaryrefslogtreecommitdiffstats
path: root/Tests
Commit message (Collapse)AuthorAgeFilesLines
* 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.
* | | | Merge topic 'stl-support'Brad King2020-05-012-4/+2
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 8d4a9ee398 Refactoring: rename "cm_static_string_view.hxx" as <cmext/string_view> Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4689
| * | | | Refactoring: rename "cm_static_string_view.hxx" as <cmext/string_view>Marc Chevrier2020-04-302-4/+2
| | |/ / | |/| |
* | | | Merge topic 'ctest-nightly-start-time'Brad King2020-05-014-1/+10
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 4f2fa2459a Tests: Add tests to demonstrate when CTEST_NIGHTLY_START_TIME is needed c1397ace1d Help: Clarify why CTEST_NIGHTLY_START_TIME is needed Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4688
| * | | | Tests: Add tests to demonstrate when CTEST_NIGHTLY_START_TIME is neededKyle Edwards2020-04-294-1/+10
| |/ / /
* | | | IWYU: mark <cstddef> as neededBen Boeckel2020-04-291-1/+1
| | | | | | | | | | | | | | | | Newer IWYU is not seeing this as necessary for `_GLIBCXX_USE_CXX11_ABI`.
* | | | IWYU: mark <cstddef> as neededBen Boeckel2020-04-293-3/+3
|/ / / | | | | | | | | | Newer IWYU is not seeing them as needed for `size_t`.
* | | Merge topic 'vs_platform_toolset'Brad King2020-04-293-0/+43
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | 197b4cbe18 VS: Add option for per-target PlatformToolset Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4682
| * | | VS: Add option for per-target PlatformToolsetJulien Jemine2020-04-293-0/+43
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add a `VS_PLATFORM_TOOLSET` target property to set `PlatformToolset` in the `.vcxproj` file for specific targets. Document that this is safe only when the named toolset uses the same underlying compiler as the primary toolset. Fixes: #17429
* | | | Merge topic 'FindLibXslt-targets'Brad King2020-04-295-0/+96
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 8a90ef5a44 FindLibXslt: use LIBXSLT_LIBRARY as cache variable 1aa7df4114 FindLibXslt: provide imported targets dd506714f4 FindLibXslt: identify libexslt include directory Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4650
| * | | | FindLibXslt: provide imported targetsMarkus Rickert2020-04-285-0/+96
| | |/ / | |/| |
* | | | Merge topic 'FindPython-find-implementations'Brad King2020-04-294-2/+80
|\ \ \ \ | |_|/ / |/| | | | | | | | | | | | | | | | | | | d2c47c822b FindPython: Add capability to specify Python implementations Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4676
| * | | FindPython: Add capability to specify Python implementationsMarc Chevrier2020-04-284-2/+80
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Through hint Python_FIND_IMPLEMENTATIONS it is possble to specify, as an ordered list, which implementations must be searched for. Currently possible values are: * CPython * IronPython
* | | | Merge topic 'FindLAPACK-target'Brad King2020-04-284-0/+44
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 4ed936d1b8 FindLAPACK: Provide the LAPACK::LAPACK import target Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4677
| * | | | FindLAPACK: Provide the LAPACK::LAPACK import targetRobert Maynard2020-04-274-0/+44
| |/ / /
* | | | Merge topic 'pch-genex-absolute'Brad 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
| * | | target_precompile_headers: Fix documented example using genexBrad King2020-04-274-2/+6
| | |/ | |/| | | | | | | | | | | | | | | | | | | When the path to a header file is specified using a generator expression, evaluation of the genex must produce an absolute path. Update our documented example and add a test covering the case. Fixes: #20617
| * | target_link_libraries: Fix regression in case of $<CONFIG> genexBrad King2020-03-304-0/+28
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since commit b8626261e9 (Precompile headers: Add methods to generate PCH sources, 2019-07-13, v3.16.0-rc1~182^2~4) we look up source files for a target using an upper-case configuration even though an original-case name is sufficient. Since commit 36ded610af (PCH: Generate sources during Compute step, 2019-10-05, v3.16.0-rc1~2^2) the source file lookup is the first time we compute many on-demand structures that depend on the configuration name. This caused the `$<CONFIG>` generator expression to evaluate to the upper-case configuration name in some cases where we used original-case before. Fix this by switching the source file lookup to the original-case config name. Add a test covering the symptom that led to the discovery of this problem. Fixes: #20517
* | | Merge topic 'FindBLAS-target'Brad King2020-04-274-0/+38
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | 85a9813a76 BLAS: Provide the BLAS::BLAS import target Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4657
| * | | BLAS: Provide the BLAS::BLAS import targetRobert Maynard2020-04-234-0/+38
| | | |
* | | | Merge topic 'FindPython-fix-python-compiler-validation'Brad King2020-04-244-4/+113
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | f39da773ee FindPython: fix python compiler validation Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4655
| * | | | FindPython: fix python compiler validationMarc Chevrier2020-04-234-4/+113
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Ensure also the cache of properties is erased in case of multiple searches. Fixes: #20626, #20627
* | | | | Merge branch 'master' into ninja-order-only-fixKyle Edwards2020-04-23692-4962/+9364
|\ \ \ \ \
| * \ \ \ \ Merge topic 'remove-documentation-module'Brad King2020-04-2315-0/+110
| |\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 306a1ba960 Modules/Documentation: remove ad4487a96a cmIncludeCommand: add infrastructure for deprecated modules Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4617
| | * | | | | Modules/Documentation: removeBen Boeckel2020-04-2215-0/+110
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is an old module from when VTK and other Kitware projects drove the addition of commands and modules into CMake itself. Modern VTK doesn't need this module and it can be ignored. This module is kept around so that the ancient VTK versions which use it are not broken by it. VTK itself stopped using the module in 2012 and the last usage by an example in VTK was removed in 2020. Fixes: #20591
| * | | | | | Merge topic 'ninja-compiler-PATH-windows'Brad King2020-04-239-0/+30
| |\ \ \ \ \ \ | | |_|_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | e01990999a Ninja: On Windows, select the compiler occurring first in PATH Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4625
| | * | | | | Ninja: On Windows, select the compiler occurring first in PATHBrad King2020-04-229-0/+30
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | On Windows, toolchains like MSVC require a set of environment variables to be configured for the compiler to work correctly. The scripts that prepare these environments typically put the compiler's directory ahead of others in the `PATH`. Teach the Ninja generators to use this as a heuristic to select the compiler when none is explicitly specified. This is not necessary with Makefile generators because each toolchain's environment comes with its own make tool variant, and the corresponding Makefile generator (e.g. "NMake Makefiles") automatically implies the matching compiler. Fixes: #20585
| * | | | | | Merge topic 'win-user-manifest-rc'Brad King2020-04-229-5/+75
| |\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 7ca13e04fa Windows: Merge user-provided manifests into compiled resource Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4484
| | * | | | | | Windows: Merge user-provided manifests into compiled resourceKenney Phillis2020-04-219-5/+75
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously we merged user-provided manifests only into the manifest file given to the linker. Merge them into the manifest file that is compiled as a resource too.
| * | | | | | | Merge topic 'ctest_stop_on_failure_16628'Brad King2020-04-226-0/+51
| |\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | e89aeba5c4 ctest: add option --stop-on-failure Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4630
| | * | | | | | | ctest: add option --stop-on-failureJohnny Jazeix2020-04-206-0/+51
| | | |/ / / / / | | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | To stop the tests once one has failed Fixes: #16628
| * | | | | | | Merge topic 'external-project-mkdir-step-fix'Brad King2020-04-211-1/+1
| |\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | e5b33871ed ExternalProject: Fix step dependency on mkdir step Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4640
| | * | | | | | | ExternalProject: Fix step dependency on mkdir stepVyacheslav Yurkov2020-04-201-1/+1
| | | |_|/ / / / | | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When looking at `list(FIND)` result, zero index is ignored due to incorrect error handling, and users can't set dependencies on mkdir step. Fixes: #20605
| * | | | | | | Merge topic 'test-vtk'Brad King2020-04-211-2/+2
| |\ \ \ \ \ \ \ | | |/ / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | cc56bf89d9 Tests/Contracts: Update VTK release branch build configuration Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: Ben Boeckel <ben.boeckel@kitware.com> Merge-request: !4638