summaryrefslogtreecommitdiffstats
path: root/Tests
Commit message (Collapse)AuthorAgeFilesLines
* Merge branch 'vs-pch-compile-opts' into release-3.16Brad King2020-05-111-0/+3
|\ | | | | | | 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 branch 'backport-3.16-FindPkgConfig-isystem' into release-3.16Brad King2020-05-061-1/+21
|\ \ | | | | | | | | | Merge-request: !4693
| * | FindPkgConfig: also handle "-isystem" prefixes for include directoriesRolf Eike Beer2020-05-051-1/+21
| |/ | | | | | | Fixes: #20652
* | Merge branch 'CheckLanguage-cuda-host' into release-3.16Brad King2020-05-051-9/+9
|\ \ | | | | | | | | | 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
* | 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
* 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