summaryrefslogtreecommitdiffstats
path: root/Tests
Commit message (Collapse)AuthorAgeFilesLines
* 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
* install: Fix regression when using default destinationsKyle Edwards2020-02-116-12/+12
| | | | | | | | | | | In commit 9fc20a4f3e (install: Add sane set of defaults for DESTINATION and file type parameters, 2018-11-02, v3.14.0-rc1~410^2~1), a regression was introduced, in which an `install(TARGETS)` with a RUNTIME/LIBRARY/ARCHIVE DESTINATION but no PUBLIC_HEADER/PRIVATE_HEADER DESTINATION would then install the headers. The old behavior did not do this. Restore the old behavior. Fixes: #20326
* CPack: Fix regression in Deb descriptionKyle Edwards2020-01-244-7/+16
| | | | Fixes: #20254
* file(GET_RUNTIME_DEPENDENCIES): Tolerate empty list argumentsKyle Edwards2020-01-222-2/+20
| | | | | Also list all keywords that were missing values instead of only the first one.
* Unity Build: include language in generated source file nameCristian Adam2020-01-1411-15/+15
| | | | Fixes: #20206
* CPack: Fix regression in DEB generator descriptionKyle Edwards2020-01-104-5/+9
| | | | Fixes: #20102
* Merge topic 'cuda-device-link-only-cuda' into release-3.16Brad King2020-01-103-0/+9
|\ | | | | | | | | | | | | | | 7ffa42288e Merge branch 'backport-3.15-cuda-device-link-only-cuda' 071c4f1a2a CUDA: Do not device link if target has no CUDA usage Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4191
| * Merge branch 'backport-3.15-cuda-device-link-only-cuda'Brad King2020-01-093-0/+9
| |\
| | * CUDA: Do not device link if target has no CUDA usageRobert Maynard2020-01-093-0/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When CUDA is enabled, and a pure non-CUDA target has CMAKE_CUDA_SEPARABLE_COMPILATION enabled, don't actually perform the device linking step, as it will fail. A target that has CMAKE_CUDA_SEPARABLE_COMPILATION enabled must also have CUDA usage (either itself, or something it links to). Fixes: #20182
| | * Merge branch 'backport-findpostgres-10-and-older' into release-3.15Brad King2019-11-071-3/+12
| | |\ | | | | | | | | | | | | Merge-request: !4007
| | | * FindPostgreSQL: support version encoding used in pre-10 releasesBen Boeckel2019-11-071-3/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | With the 10.x release, PostgreSQL upstream started encoding the version as `MMmmmm` where `M` is major and `m` is minor. Prior to that, `MMmmPP` was used where `P` was the patch number. Detect this difference and decode it based on the used encoding. Fixes: #19912
| | * | Xcode: Restore CMAKE_XCODE_GENERATE_SCHEME for custom targetsBrad King2019-10-022-0/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The target property introduced by commit 413b71485a (Xcode: Create Xcode schemes per target, 2019-03-11, v3.15.0-rc1~347^2) was accidentally not initialized by `CMAKE_XCODE_GENERATE_SCHEME` for custom targets. Fix it and update the test. Fixes: #19759
* | | | ObjC: Add _COMPILE_LAUNCHER supportCristian Adam2020-01-0912-0/+32
|/ / / | | | | | | | | | Fixes: #20178
* | | CTest: Improve error handling when reading resource spec fileKyle Edwards2019-12-272-53/+72
| | | | | | | | | | | | Fixes: #20079
* | | Merge topic 'autogen-no-hh' into release-3.16Brad King2019-12-193-16/+0
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | 7fa7f55230 Autogen: Revert processing of .hh files for compatibility Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4146
| * | | Autogen: Revert processing of .hh files for compatibilityBrad King2019-12-183-16/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since commit 4a9154537c (Autogen: Use cmake::IsHeader/SourceExtension for file type detection, 2019-07-02, v3.16.0-rc1~470^2~4) we process `.hh` files with `AUTOMOC`. However, this change can break existing projects that do not expect the behavior. Revert it for now. It can be restored later via a policy. Fixes: #20101
* | | | Merge topic 'test-resource-spec-init-lists' into release-3.16Brad King2019-12-191-37/+36
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | e8dbe4bba1 Tests: Fix testCTestResourceSpec struct initialization for some compilers Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: Michael Dickens <michael.dickens@ettus.com> Merge-request: !4141
| * | | | Tests: Fix testCTestResourceSpec struct initialization for some compilersMichael Dickens2019-12-181-37/+36
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | Some compilers do not like the nested `{{{}}}` member initialization syntax, so use just `{}`. GCC 4.8 warns about the latter, so add a suppression. Fixes: #20097
* | | | Merge topic 'pch-force-include' into release-3.16Brad King2019-12-161-1/+1
|\ \ \ \ | |/ / / |/| | | | | | | | | | | | | | | | | | | c5c218fa0d PCH: Append pch header file to list of forced include files Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4135
| * | | PCH: Append pch header file to list of forced include filesCristian Adam2019-12-131-1/+1
| | | | | | | | | | | | | | | | Fixes: #20088
* | | | Merge topic 'unity-object-libraries' into release-3.16Brad King2019-12-093-0/+20
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | fa93b4a59b Unity: Proper handling of object libraries 5ae07e7166 Unity: Generic source file handling for all generators f742f7ac1f Unity/PCH: Skip more target types when adding automatic sources Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4110
| * | | | Unity: Proper handling of object librariesCristian Adam2019-12-073-0/+20
| |/ / / | | | | | | | | | | | | Fixes: #20051
* | | | Merge topic 'pch-reuse-importlib' into release-3.16Brad King2019-12-093-0/+32
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 93becd61d1 PCH: Use the target's PREFIX for building the pdb file name Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4114
| * | | | PCH: Use the target's PREFIX for building the pdb file nameCristian Adam2019-12-083-0/+32
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | Also copy the REUSE_FROM pdb file only if the file is newer than the existing one. Fixes: #19731 Fixes: #20068
* | | | CTest Resource Allocation: Add test for spec file with no versionKyle Edwards2019-11-262-0/+5
|/ / /
* | | CTest: Clarify that resource requirements can be splitKyle Edwards2019-11-259-2/+25
| | | | | | | | | | | | | | | | | | | | | Add a note to the documentation to clarify this, and add test cases to ensure it. Fixes: #19987
* | | CTest: Add version field to resource spec fileKyle Edwards2019-11-2535-0/+189
| | | | | | | | | | | | Fixes: #20007
* | | Merge topic 'pch-reuse-no-prop' into release-3.16Brad King2019-11-182-2/+15
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | bb4c2781ce PCH: Do not issue an error on duplicate target_precompile_headers call Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4046
| * | | PCH: Do not issue an error on duplicate target_precompile_headers callCristian Adam2019-11-152-2/+15
| | | | | | | | | | | | | | | | Fixes: #19970
* | | | Merge topic 'objc-tests' into release-3.16Brad King2019-11-1523-87/+10
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | d2e5e6ff1a Tests: Organize Objective C/C++ test directories Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4051
| * | | | Tests: Organize Objective C/C++ test directoriesBrad King2019-11-1423-87/+10
| |/ / / | | | | | | | | | | | | Move them all under `Tests/ObjC` and `Tests/ObjCXX`.
* | | | Tests: Fix reliance on undefined behavior of cm::optionalKyle Edwards2019-11-131-36/+48
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The test for cm::optional assumed that *opt would return a reference to a non-constructed object if opt.has_value() is false. However, on certain systems that build CMake with flags to harden the build, this condition aborts rather than returning the invalid reference. Fix this by getting the reference only when the cm::optional is known to actually have a value. Fixes: #19964
* | | Merge topic 'unity-HEADER_FILE_ONLY' into release-3.16Brad King2019-11-123-4/+8
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | 9a5418320e Unity: Don't include sources with HEADER_FILE_ONLY property set Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4023
| * | | Unity: Don't include sources with HEADER_FILE_ONLY property setCristian Adam2019-11-113-4/+8
| | | | | | | | | | | | | | | | | | | | | | | | Fixes: #19946 Fixes: #19947 Co-authored-by: Craig Scott <craig.scott@crascit.com>
* | | | Merge topic 'objc-xcode-flags' into release-3.16Brad King2019-11-125-0/+44
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | bb42e1ed43 ObjC: Add OBJC/OBJCXX flags to Xcode projects Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4020
| * | | | ObjC: Add OBJC/OBJCXX flags to Xcode projectsCristian Adam2019-11-115-0/+44
| | | | | | | | | | | | | | | | | | | | Fixes: #19936
* | | | | Merge topic 'pch-cray-classic' into release-3.16Brad King2019-11-121-1/+1
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 77a8f9ecbc Tests: Fix ExportImport PCH expectation on Cray Classic compiler Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4033
| * | | | | Tests: Fix ExportImport PCH expectation on Cray Classic compilerBrad King2019-11-111-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | Do not expect PCH-included definitions for this compiler.
* | | | | | Merge topic 'pch-no-duplicates' into release-3.16Craig Scott2019-11-124-26/+7
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | e01935ac9d PCH: No repeated path for internal generated PCH files Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4030
| * | | | | | PCH: No repeated path for internal generated PCH filesCristian Adam2019-11-114-26/+7
| | |_|/ / / | |/| | | | | | | | | | | | | | | | Fixes: #19952
* | | | | | Merge topic 'remove-CPACK_INSTALL_CMAKE_CONFIGURATIONS' into release-3.16Craig Scott2019-11-1210-18/+51
|\ \ \ \ \ \ | |_|_|/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 1d1fa5d3e4 Tests: Add RunCMake.CPackCommandLine case for multi-config package 108207cc84 CPack: Remove CPACK_INSTALL_CMAKE_CONFIGURATIONS Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4025
| * | | | | Tests: Add RunCMake.CPackCommandLine case for multi-config packageBrad King2019-11-116-0/+51
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In commit da5ac4bb60 (cpack: Add `CPACK_INSTALL_CMAKE_CONFIGURATIONS` variable, 2019-07-10, v3.16.0-rc1~165^2) we added support for running the `cpack -C ...` command-line with multiple configurations. Add an explicit test for this. Fixes: #19918
| * | | | | CPack: Remove CPACK_INSTALL_CMAKE_CONFIGURATIONSAlex Turbov2019-11-114-18/+0
| | |/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In commit da5ac4bb60 (cpack: Add `CPACK_INSTALL_CMAKE_CONFIGURATIONS` variable, 2019-07-10, v3.16.0-rc1~165^2) we added both the variable and support for `cpack -C` with multiple configurations. Drop the variable because the `package` target cannot ensure that all of the configurations are built. Keep the command-line interface so that it can be used manually in scripts. Fixes: #19918
* | | | | Merge topic 'objc-trycompile' into release-3.16Brad King2019-11-1113-0/+70
|\ \ \ \ \ | |_|/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | 7447aa4b34 ObjC: Add try_compile support Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !4006
| * | | | ObjC: Add try_compile supportCristian Adam2019-11-0913-0/+70
| | | | | | | | | | | | | | | | | | | | Fixes: #19920
* | | | | Merge topic 'ctest-resource-groups' into release-3.16Kyle Edwards2019-11-07109-398/+402
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | c1435d9812 Help: Fix error in resource allocation example eb9d945f14 CTest: Rename hardware -> resources for RunCMake tests c544cb6698 CTest: Rename hardware -> resources for source code 6ce27d3a2e cmCTestMultiProcessHandler: Rename resource locking functions a7c1e15cc4 CTest: Rename hardware -> resources for CMake variables, command options 73a40b19ff CTest: Rename "Processes" lexer to "ResourceGroups" af9ed543b0 CTest: Rename PROCESSES test property to RESOURCE_GROUPS Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !3994
| * | | | | CTest: Rename hardware -> resources for RunCMake testsCraig Scott2019-11-0584-287/+287
| | | | | | | | | | | | | | | | | | Also includes variants like hw -> res
| * | | | | CTest: Rename hardware -> resources for source codeCraig Scott2019-11-0523-70/+70
| | | | | |
| * | | | | CTest: Rename hardware -> resources for CMake variables, command optionsCraig Scott2019-11-054-5/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Only changes the user-visible effects of renaming hardware allocation to resource allocation. Code changes are the minimum needed to achieve that.
| * | | | | CTest: Rename PROCESSES test property to RESOURCE_GROUPSBrad King2019-11-057-84/+88
| | |/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The `PROCESSES` test property name added for CMake 3.16 is too close to the existing `PROCESSORS` test property. Furthermore, the property in principle specifies groups of resources organized in a way that is meaningful to a particular test. The groups may often correspond to processes but they could have other meanings. Since the property name `PROCESSES` has not been in a final 3.16 release yet, simply rename it to `RESOURCE_GROUPS`. Fixes: #19914