summaryrefslogtreecommitdiffstats
path: root/Tests
Commit message (Collapse)AuthorAgeFilesLines
* Unity Builds: Do not set SKIP_AUTOGEN to source filesCristian Adam2020-07-294-0/+23
| | | | Fixes: #21028
* Merge topic 'revert-add_test-special-chars' into release-3.18Brad King2020-07-2856-373/+6
|\ | | | | | | | | | | | | 5fc5f4d26e add_test: Revert "Allow special characters in test name" Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !5067
| * add_test: Revert "Allow special characters in test name"Brad King2020-07-2756-373/+6
| | | | | | | | | | | | | | | | | | | | | | | | Revert commit f84af8e270 (add_test: Allow special characters in test name, 2020-05-16, v3.18.0-rc1~142^2). Unfortunately the fix breaks projects that were working around the limitation with manual escaping. The fix can be re-introduced with a policy in a future version. Also add a 3.18.1 release note explaining the change. Fixes: #21017, #20965 Issue: #19391
* | Merge topic 'vs-lang-flags' into release-3.18Brad King2020-07-273-0/+23
|\ \ | |/ |/| | | | | | | | | c4109a1bc8 VS: Restore toleration of target-wide -TP flag with MSVC Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !5051
| * VS: Restore toleration of target-wide -TP flag with MSVCBrad King2020-07-233-0/+23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Since commit 3b547e2e4b (VS: Simplify logic adding source file C/C++ language flag to MSVC, 2020-05-15, v3.18.0-rc1~139^2~1) we only add a per-source language selection flag when the source file extension does not match the compiler's default. This approach breaks when a project adds a target-wide `-TP` flag. Although such projects likely did not work with non-VS generators, we did support them before in Visual Studio generators. Add a special case to tolerate such flags again. Fixes: #21005
* | Merge topic 'graphviz-restore-per-target' into release-3.18Brad King2020-07-223-0/+56
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | 1829220310 Merge branch 'backport-3.17-graphviz-restore-per-target' f3a6b4a209 Tests: Cover Graphviz support for per-target dependency graph options 93549b9224 Graphviz: Restore support for per-target dependency graph options Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !5039
| * \ Merge branch 'backport-3.17-graphviz-restore-per-target'Brad King2020-07-213-0/+56
| |\ \ | | |/ | |/|
| | * Tests: Cover Graphviz support for per-target dependency graph optionsStephan Rohmen2020-07-213-0/+56
| | | | | | | | | | | | Issue: #20928
* | | Swift: Fix regression in linking to interface librariesBrad King2020-07-171-0/+1
|/ / | | | | | | | | | | | | | | | | | | Since commit 2026915f8f (Swift: Propagate Swift_MODULE_DIRECTORY as include directory, 2020-02-03, v3.18.0-rc1~547^2) we internally call `GetAllConfigCompileLanguages` on all directly linked targets without checking if they are interface libraries that don't compile at all. That violates an internal assumption and assertion. Fixes: #20977
* | Merge topic 'source_file_props_dedup_scopes' into release-3.18Craig Scott2020-07-141-0/+19
|\ \ | | | | | | | | | | | | | | | | | | f6969b917d set_property: Deduplicate source file directory scopes Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !5009
| * | set_property: Deduplicate source file directory scopesAlexandru Croitor2020-07-131-0/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | A user could specify the same directory scope to set_property() multiple times, which in conjunction with APPEND would append the property multiple times. Make sure to deduplicate scopes across both DIRECTORY and TARGET_DIRECTORY options, so that a property is only appended once in such a scenario. Fixes: #20941
* | | Merge topic 'macos-11-file-GET_RUNTIME_DEPENDENCIES' into release-3.18Brad King2020-07-141-12/+14
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | d9af90504f Tests: Update file(G_R_D) test to make system library optional c51400033c file: Update GET_RUNTIME_DEPENDENCIES for macOS 11 dylib cache Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: Kyle Edwards <kyle.edwards@kitware.com> Merge-request: !4998
| * | | Tests: Update file(G_R_D) test to make system library optionalKyle Edwards2020-07-131-12/+14
| | | | | | | | | | | | | | | | | | | | Since the system library may not exist on the filesystem on macOS 11, skip it if it doesn't exist.
* | | | Merge topic 'profiling-case-insensitive-command-names' into release-3.18Brad King2020-07-142-1/+18
|\ \ \ \ | |_|/ / |/| | | | | | | | | | | | | | | | | | | 7cf2f7d2af cmake: Store lowercase command names in profiling output Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !5007
| * | | cmake: Store lowercase command names in profiling outputCraig Scott2020-07-132-1/+18
| |/ / | | | | | | | | | | | | | | | | | | This ensures commands can be properly aggregated by tools processing the profiling output. Fixes: #20946
* | | ALIAS targets: Non-global aliases must be propagated to sub-directoriesMarc Chevrier2020-07-114-6/+25
|/ / | | | | | | Fixes: #20942
* | set_property: Allow both DIRECTORY and TARGET_DIRECTORY togetherAlexandru Croitor2020-07-092-2/+26
| | | | | | | | | | | | | | | | | | Allow to specify both DIRECTORY and TARGET_DIRECTORY at the same time in `set_source_files_properties()` and `set_property(SOURCE)` commands. Add test cases and update the documentation. Fixes: #20932
* | set_property: Fix name of TARGET_DIRECTORY option in error messagesAlexandru Croitor2020-07-092-4/+4
| |
* | Help: Clarify search order for resource spec fileKyle Edwards2020-07-076-1/+26
| | | | | | | | | | | | | | | | Clarify the order in which --resource-spec-file, RESOURCE_SPEC_FILE, and CTEST_RESOURCE_SPEC_FILE are searched, and add tests to enforce this. Fixes: #20914
* | Restore handling of build directory inside a symlinked pathBrad King2020-07-026-0/+52
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In commit dd8365b3f1 (Merge branch 'upstream-KWSys' into update-kwsys, 2020-04-06, v3.18.0-rc1~397^2) we imported KWSys commit `019afb6ea` (SystemTools: Drop GetCurrentWorkingDirectory 'collapse' argument, 2020-04-03). That caused `GetCurrentWorkingDirectory` to no longer send paths through the KWSys translation map and broke CMake's detection of the absolute path to a build directory containing a symbolic link. Add our own `cmSystemTools::GetCurrentWorkingDirectory` wrapper around the KWSys method in order to restore that mapping. Test-case-by: Ben Boeckel <ben.boeckel@kitware.com> Issue: #16228 Fixes: #20900
* | Merge topic 'file-ARCHIVE-files-dirs' into release-3.18Brad King2020-07-012-14/+14
|\ \ | | | | | | | | | | | | | | | | | | bbcff21f71 file(ARCHIVE*): Collapse FILES and DIRECTORY options Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4955
| * | file(ARCHIVE*): Collapse FILES and DIRECTORY optionsCraig Scott2020-06-302-14/+14
| | | | | | | | | | | | | | | | | | | | | | | | The two options were concatenated internally for both ARCHIVE_CREATE and ARCHIVE_EXTRACT. The distinction between files and dirs was not meaningful. Therefore, replace them with PATHS or PATTERNS to more accurately describe the way the options are used. Fixes: #20884
* | | Merge topic 'file-CONFIGURE-subdir' into release-3.18Brad King2020-06-304-0/+7
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | e5f5eeca2f file: Fix CONFIGURE output relative path Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4951
| * | | file: Fix CONFIGURE output relative pathBrad King2020-06-294-0/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In commit a6fee09484 (file: Add CONFIGURE subcommand, 2020-03-06, v3.18.0-rc1~584^2) we accidentally treated relative path outputs with respect to the current working directory. Treat them with respect to the current binary directory instead. Fixes: #20885
* | | | Merge topic 'find_library-file-readable' into release-3.18Brad King2020-06-304-2/+22
|\ \ \ \ | |_|/ / |/| | | | | | | | | | | | | | | | | | | f2c903fb9a find_library: Check that library files are readable Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4945
| * | | find_library: Check that library files are readableBrad King2020-06-294-2/+22
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | Refactoring in commit 6b85166920 (ENH: Refactor find_library search logic, 2008-09-22, v2.8.0~1665) dropped a call to `FileExists` on the path under the assumption that the presence of a file in a directory listing means it exists. However, dropping that also dropped a check that verifies the file is readable. Restore the `FileExists` call to ensure that we only find readable libraries.
* | | file(ARCHIVE_CREATE): Rename TYPE option to COMPRESSIONCraig Scott2020-06-2719-18/+31
| | | | | | | | | Fixes: #20883
* | | Tests: Don't ask for things not required for GoogleTestCraig Scott2020-06-244-8/+4
| | | | | | | | | | | | The test cases only need C or C++, but not both. The CTest module is also not needed because we are not running a dashboard script.
* | | Merge topic 'test-all-find-modules' into release-3.18Brad King2020-06-233-35/+0
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | 6c6b227813 Tests: Drop redundant FindModulesExecuteAll test Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4930
| * | | Tests: Drop redundant FindModulesExecuteAll testBrad King2020-06-233-35/+0
| |/ / | | | | | | | | | | | | This test has been superseded by `CMakeOnly.AllFindModules`, which does much more checking anyway.
* | | Merge topic 'CUDAToolkit-cuda-11' into release-3.18Brad King2020-06-192-6/+33
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | b284a3c764 CudaToolkit: Update for new/removed libraries in CUDA 11 Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4917
| * | | CudaToolkit: Update for new/removed libraries in CUDA 11Robert Maynard2020-06-182-6/+33
| |/ /
* | | Tests: Shorten case names in RunCMake.target_link_libraries-* testsBrad King2020-06-1718-31/+31
| | | | | | | | | | | | | | | Drop the portion of the case names that repeats the name of the containing test.
* | | Tests: Split RunCMake.target_link_libraries testMarc Chevrier2020-06-1740-68/+201
|/ / | | | | | | Fixes: #20836
* | Merge topic 'test-file-GET_RUNTIME_DEPENDENCIES' into release-3.18Brad King2020-06-1768-275/+295
|\ \ | | | | | | | | | | | | | | | | | | | | | 9ae3382855 Tests: Shorten case names in RunCMake.file-GET_RUNTIME_DEPENDENCIES test 248d0570c9 Tests: Factor out RunCMake.file-GET_RUNTIME_DEPENDENCIES test Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4904
| * | Tests: Shorten case names in RunCMake.file-GET_RUNTIME_DEPENDENCIES testBrad King2020-06-1664-255/+255
| | | | | | | | | | | | | | | Now that these cases are in a dedicated test we can drop the `file-GET_RUNTIME_DEPENDENCIES-` prefix from their name.
| * | Tests: Factor out RunCMake.file-GET_RUNTIME_DEPENDENCIES testBrad King2020-06-1669-296/+316
| | | | | | | | | | | | Move the `file(GET_RUNTIME_DEPENDENCIES)` cases out of `RunCMake.install`.
* | | Merge topic 'test-CudaOnly.StaticRuntimePlusToolkit' into release-3.18Brad King2020-06-171-0/+6
|\ \ \ | |/ / |/| | | | | | | | | | | | | | cb9fab49dd Tests: Fix CudaOnly.StaticRuntimePlusToolkit build rpath Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4901
| * | Tests: Fix CudaOnly.StaticRuntimePlusToolkit build rpathRobert Maynard2020-06-161-0/+6
| | |
* | | Tests: Factor out RunCMake.GenEx-GENEX_EVAL testBrad King2020-06-1630-9/+15
| | | | | | | | | | | | Move the `GENEX_EVAL` cases and friends out of `RunCMake.GeneratorExpression`.
* | | Tests: Factor out RunCMake.GenEx-TARGET_FILE testBrad King2020-06-1675-31/+38
| | | | | | | | | | | | Move the `TARGET_FILE` cases and friends out of `RunCMake.GeneratorExpression`.
* | | Tests: Factor out RunCMake.GenEx-DEVICE_LINK testBrad King2020-06-1644-15/+21
| | | | | | | | | | | | Move the `DEVICE_LINK` cases out of `RunCMake.GeneratorExpression`.
* | | Tests: Factor out RunCMake.GenEx-HOST_LINK testBrad King2020-06-1644-15/+21
| | | | | | | | | | | | Move the `HOST_LINK` cases out of `RunCMake.GeneratorExpression`.
* | | Tests: Factor out RunCMake.GenEx-LINK_LANG_AND_ID testBrad King2020-06-1639-12/+18
| | | | | | | | | | | | Move the `LINK_LANG_AND_ID` cases out of `RunCMake.GeneratorExpression`.
* | | Tests: Factor out RunCMake.GenEx-LINK_LANGUAGE testBrad King2020-06-1642-13/+19
| | | | | | | | | | | | Move the `LINK_LANGUAGE` cases out of `RunCMake.GeneratorExpression`.
* | | Tests: Factor out RunCMake.GenEx-COMPILE_LANG_AND_ID testBrad King2020-06-1627-8/+14
| | | | | | | | | | | | Move the `COMPILE_LANG_AND_ID` cases out of `RunCMake.GeneratorExpression`.
* | | Tests: Factor out RunCMake.GenEx-COMPILE_LANGUAGE testBrad King2020-06-1627-8/+14
|/ / | | | | | | Move the `COMPILE_LANGUAGE` cases out of `RunCMake.GeneratorExpression`.
* | Merge topic 'cuda_architectures_disable' into release-3.18Brad King2020-06-153-2/+16
|\ \ | | | | | | | | | | | | | | | | | | 877a92e968 CUDA: Add support for disabling CUDA_ARCHITECTURES Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4886
| * | CUDA: Add support for disabling CUDA_ARCHITECTURESRaul Tambre2020-06-153-2/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The ability to disable adding architectures completely for packaging purposes and cases requiring passing the architectures flags explicitly has been requested. Support a false value for CUDA_ARCHITECTURES and CMAKE_CUDA_ARCHITECTURES for this purpose. Implements #20821.
* | | Merge topic 'revert-find_program-exe-no-read' into release-3.18Brad King2020-06-152-4/+10
|\ \ \ | |/ / |/| | | | | | | | | | | | | | cc02ced530 find_program: Revert "Find programs that are executable but not readable" Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4871