summaryrefslogtreecommitdiffstats
path: root/Tests
Commit message (Collapse)AuthorAgeFilesLines
* Merge topic 'policy-cmp0111-iface'Brad King2020-11-245-12/+34
|\ | | | | | | | | | | | | | | | | 54ef732b0c cmVisualStudio10TargetGenerator: Avoid GetFullPath on INTERFACE library f06f4b517c cmTarget: Do not enforce CMP0111 on imported INTERFACE libraries 43c95df8fb Tests: Match RunCMake.CMP0111 stderr more strictly Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !5530
| * cmTarget: Do not enforce CMP0111 on imported INTERFACE librariesBrad King2020-11-233-5/+7
| | | | | | | | | | | | | | | | Fix logic added by commit 359c500a24 (cmTarget: Raise error if imported target location is not set, 2020-08-08, v3.19.0-rc1~273^2) to exclude INTERFACE libraries from the policy. They have no location. Fixes: #21470
| * Tests: Match RunCMake.CMP0111 stderr more strictlyBrad King2020-11-232-7/+27
| |
| * Merge topic 'fix-compile-db-crash' into release-3.19Brad King2020-11-232-0/+6
| |\ | | | | | | | | | | | | | | | | | | 67e2130c96 Makefiles: Fix CMAKE_EXPORT_COMPILE_COMMANDS crash with custom compile rule Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !5521
* | \ Merge topic 'xcode-cc-work-dir'Brad King2020-11-245-0/+27
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | 36921d2d23 Xcode: Fix custom command work-dir placeholders in "new build system" Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !5527
| * | | Xcode: Fix custom command work-dir placeholders in "new build system"Brad King2020-11-235-0/+27
| |/ / | | | | | | | | | | | | | | | | | | | | | The placeholders for `CONFIGURATION` and `EFFECTIVE_PLATFORM_NAME` need to be handled in the `WORKING_DIRECTORY` of custom commands just as we already do for the `COMMAND`. Fixes: #21483
* | | Merge topic 'fix-compile-db-crash'Brad King2020-11-232-0/+6
|\ \ \ | | |/ | |/| | | | | | | | | | | | | 67e2130c96 Makefiles: Fix CMAKE_EXPORT_COMPILE_COMMANDS crash with custom compile rule Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !5521
| * | Makefiles: Fix CMAKE_EXPORT_COMPILE_COMMANDS crash with custom compile ruleBrad King2020-11-202-0/+6
| | | | | | | | | | | | Fixes: #21471
* | | Merge topic 'improve_cmakes_error_handling_of_bad_parameters'Brad King2020-11-208-7/+12
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 7302a23a1f cmake: Simplify -W recognition of no- and error= prefixes 98290782b6 cmake: redesign command-line argument parsing d800c26ce9 cmake: Fix processing of -Wno-error= flags Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: Axel Huebl <axel.huebl@plasma.ninja> Merge-request: !5458
| * | | cmake: redesign command-line argument parsingRobert Maynard2020-11-197-7/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Make handling more consistent: "-S" -> invalid "-S/path/to/source" -> valid "-S /path/to/source" -> valid "-S=/path/to/source" -> now valid "-D" -> invalid "-DStr" -> valid "-D Str" -> valid "-D=Str" -> now valid "--log-level=" -> invalid "--log-level" -> invalid "--log-level=2" -> valid "--log-level 2" -> now valid
| * | | cmake: Fix processing of -Wno-error= flagsBrad King2020-11-171-0/+4
| | | | | | | | | | | | | | | | | | | | Fix two bugs that happened to cancel each other out for cases covered by our test suite. Add a test case that distinguishes them.
* | | | Merge topic 'revert-explicit-LANGUAGE-flag'Brad King2020-11-203-27/+1
|\ \ \ \ | | |_|/ | |/| | | | | | | | | | | | | | | | | | 30aa715fac Revert "specify language flag when source LANGUAGE property is set" Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !5519
| * | | Revert "specify language flag when source LANGUAGE property is set"Brad King2020-11-193-27/+1
| | |/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Revert commit 74b1c9fc8e (Explicitly specify language flag when source LANGUAGE property is set, 2020-06-01, v3.19.0-rc1~722^2) and the lookup tables from its two immediate ancestors. The purpose of that change was to convert an explicit `LANGUAGE` source file property into an explicit language specification compiler flag like `-x c`. This seems reasonable since the property is documented as meaning "indicate what programming language the source file is". It is also needed to help compilers deal with non-standard source file extensions they don't recognize. However, some projects have been setting `LANGUAGE C` on `.S` assembler source files to mean "use the C compiler". Passing `-x c` for them breaks the build because the `.S` sources are not written in C. These projects should be updated to use `enable_language(ASM)`, for which CMake often chooses the C compiler as the assembler when using toolchains that support it (which would have to be the case for projects using the approach). Revert the change for now to preserve the old behavior for such projects. We can re-introduce it with a policy in a future version of CMake. Fixes: #21469 Issue: #14516, #20716
* | | Merge topic 'rename_cuda_memcheck'Brad King2020-11-186-3/+3
|\ \ \ | |/ / | | / | |/ |/| | | | | fea49b2df0 CTest: Rename CudaMemcheck to CudaSanitizer Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !5514
| * CTest: Rename CudaMemcheck to CudaSanitizerTobias Ribizel2020-11-176-3/+3
| |
* | Merge topic 'cmake-build-specify-makefile-name'Brad King2020-11-172-0/+13
|\ \ | | | | | | | | | | | | | | | | | | | | | cfce067b3f Tests: Add RunCMake.Make case for conflicting Makefile names 378c047ae3 Makefiles: Explicitly tell 'make' tool what Makefile name to use Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !5509
| * | Tests: Add RunCMake.Make case for conflicting Makefile namesBrad King2020-11-162-0/+13
| | | | | | | | | | | | Issue: #21418
* | | CPack/DEB: Do not crash when asked for debug symbols when there are noneAndrew Fuller2020-11-162-1/+9
|/ / | | | | | | Fixes: #21356
* | Merge topic 'msvc-no-GR'Brad King2020-11-167-0/+27
|\ \ | | | | | | | | | | | | | | | | | | | | | c00a6d3967 MSVC: Do not add /GR to CMAKE_CXX_FLAGS by default 6114c8e994 MSVC: Factor out initialization of /GR flag Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !5508
| * | MSVC: Do not add /GR to CMAKE_CXX_FLAGS by defaultBrad King2020-11-137-0/+27
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The `/GR` flag has been on by default since MSVC cl 14.0 from VS 2005. Remove it from the default flags to make it easier for projects to pass `/GR-` themselves to turn it off. Projects may be using string processing to replace `/GR` with another flag, so we cannot simply drop it. Add a policy to drop it in a compatible way. Fixes: #21428
* | | Merge topic 'ci-fedora33'Brad King2020-11-161-2/+2
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 3d1183c457 Merge branch 'ci-fedora33-release' into ci-fedora33 6e35919e15 gitlab-ci: use Fedora 33 to build sources 061f7d5ca3 gitlab-ci: update Linux CI to use Fedora 33 bd62d2ecf3 ci: update to Fedora 33 for Linux builds ada3170625 ci: install gmock in the Fedora 31 image 86f196f634 Tests/FindBoost/TestPython: support finding 3.8 and 3.9 6a83417908 clang-tidy: ignore new warnings from newer versions d2b648ec4c clang-tidy: ignore `misc-no-recursion` Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !5504
| * \ \ Merge branch 'ci-fedora33-release' into ci-fedora33Ben Boeckel2020-11-131-2/+2
| |\ \ \ | | |/ / | |/| / | | |/ | | | | | | | | | | | | | | | | | | | | | * ci-fedora33-release: gitlab-ci: use Fedora 33 to build sources gitlab-ci: update Linux CI to use Fedora 33 ci: update to Fedora 33 for Linux builds ci: install gmock in the Fedora 31 image Tests/FindBoost/TestPython: support finding 3.8 and 3.9 clang-tidy: ignore new warnings from newer versions clang-tidy: ignore `misc-no-recursion`
| | * Tests/FindBoost/TestPython: support finding 3.8 and 3.9Ben Boeckel2020-11-121-2/+2
| | |
* | | Merge topic 'cmake_path-enhancements'Craig Scott2020-11-1415-80/+28
|\ \ \ | |/ / |/| | | | | | | | | | | | | | ed21ad181e cmake_path: various enhancements to subcommands/keywords Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !5457
| * | cmake_path: various enhancements to subcommands/keywordsMarc Chevrier2020-11-1115-80/+28
| | | | | | | | | | | | This change address partly the remarks done in issue #21385
* | | Merge topic 'FindPython-version-range-fix-tests'Brad King2020-11-121-16/+25
|\ \ \ | |/ / |/| / | |/ | | | | | | a00603af0b FindPython: Fix version range tests Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !5497
| * FindPython: Fix version range testsMarc Chevrier2020-11-111-16/+25
| | | | | | | | Tests must be successful when multiple versions are installed.
| * cmake_path: remove new command from 3.19Marc Chevrier2020-11-033-3/+3
| | | | | | | | | | | | | | Defer adding this command until post-3.19 development so that it has more time to mature before being included in a release. Issue: #21385
* | Merge topic 'android-root'Brad King2020-11-102-0/+24
|\ \ | | | | | | | | | | | | | | | | | | | | | cbc51a8be3 Android: restructure android search paths Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: huangqinjin <huangqinjin@gmail.com> Merge-request: !5479
| * | Android: restructure android search pathsHaibo Huang2020-11-092-0/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 1. Set CMAKE_FIND_ROOT_PATH unconditionally. Revise the implementation from commit a7f41a7ee4 (Android: Fix find_* search order within NDK for unified toolchains, 2020-10-13). In the old implementation, if people set CMAKE_FIND_ROOT_PATH, CMAKE_ANDROID_NDK won't be added to find root. And all paths added to CMAKE_SYSTEM_*_PATH below will be rerooted to the user specified root. 2. Add api level specific library path to CMAKE_SYSTEM_PREFIX_PATH. As the discussion in [1], some people want the paths added by UnixPaths.cmake. They install their libraries according to GNUInstallDirs [2]. As a result, we cannot clear CMAKE_SYSTEM_PREFIX_PATH. It includes /usr so no matter what we specify in CMAKE_SYSTEM_LIBRARY_PATH, /usr/lib/<arch> will be searched first. The author also pointed out a way to solve this issue [3]. In addition to other paths, CMake also searches <root>/<prefix> [4]. So we can add the API specific lib path to the beginning of CMAKE_SYSTEM_PREFIX_PATH, to have it searched first. [1] https://android-review.googlesource.com/c/platform/ndk/+/1486800 [2] https://cmake.org/cmake/help/latest/module/GNUInstallDirs.html [3] https://github.com/android/ndk/issues/1179#issuecomment-613435081 [4] https://gitlab.kitware.com/cmake/cmake/-/blob/11425041f04fd0945480b8f9e9933d1549b93981/Source/cmSearchPath.cxx#L202
* | | TestBigEndian: Re-implement using byte order detected from ABI checkBrad King2020-11-0611-0/+63
| | | | | | | | | | | | Document the module as deprecated in favor of the ABI check results.
* | | Tests: Prepare RunCMake.ABI cases to run more checksBrad King2020-11-065-5/+20
|/ /
* | Merge topic 'clang-tidy-for-objc'Brad King2020-11-0611-0/+32
|\ \ | | | | | | | | | | | | | | | | | | 1134064e22 clang-tidy: allow OBJC and OBJCXX Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !5467
| * | clang-tidy: allow OBJC and OBJCXXAndrew Fuller2020-11-0511-0/+32
| | |
* | | Merge topic 'GNUInstallDirs-dir'Brad King2020-11-053-0/+24
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | a61007b3b0 Tests: Add RunCMake.GNUInstallDirs case for get_absolute_install_dir 229b5ee994 GNUInstallDirs: Add dir argument to GNUInstallDirs_get_absolute_install_dir Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !5450
| * | | Tests: Add RunCMake.GNUInstallDirs case for get_absolute_install_dirBrad King2020-11-043-0/+24
| | | |
* | | | Merge topic 'abi-byte-order'Craig Scott2020-11-058-0/+40
|\ \ \ \ | |_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | f511a1c009 CMakeDetermineCompilerABI: Detect byte order as part of check 606b34b3a6 CMakeDetermineCompilerABI: Look for more INFO strings in test binary Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: Juan Ramos <juan.ramos@amd.com> Acked-by: John Palmer <juanr0911@gmail.com> Merge-request: !5460
| * | | CMakeDetermineCompilerABI: Detect byte order as part of checkBrad King2020-11-048-0/+40
| | | | | | | | | | | | | | | | | | | | | | | | | | | | We already detect `sizeof(void*)`. Detect the byte order as part of the same check. Issue: #21392
* | | | FindPython: Adds control over artifact names to searchMarc Chevrier2020-11-032-0/+78
|/ / / | | | | | | | | | Fixes: #21371
* | | Merge topic 'nmc-test-fix'Brad King2020-11-032-1/+1
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | e0a7c0d40d Tests: Fix one RunCMake.NinjaMultiConfig case name Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !5447
| * | | Tests: Fix one RunCMake.NinjaMultiConfig case nameBrad King2020-11-022-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | The CustomCommandGenerator `debug-in-release-graph-clean` case actually cleans using the Debug graph. This is intentional in order to test cleaning only the Debug parts, so fix the test name rather than changing what graph it uses.
* | | | Merge topic 'cmake-presets-no-path-arg-warning'Brad King2020-11-033-0/+15
|\ \ \ \ | | |_|/ | |/| | | | | | | | | | | | | | | | | | cb2d01c182 CMakePresets.json: Don't warn if no path argument is given Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !5455
| * | | CMakePresets.json: Don't warn if no path argument is givenKyle Edwards2020-11-023-0/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If --preset is specified with no path argument, use the current directory as the source directory, the preset's binaryDir as the binary directory, and don't issue the standard warning for no path specified. Fixes: #21386
| * | | Merge topic 'CheckSourceRuns-compatibility' into release-3.19Brad King2020-10-302-2/+17
| |\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | a524bf3aa7 CheckSourceRuns: do not fail on unrecognized arguments Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !5426
| * \ \ \ Merge topic 'cuda_vs_skip_computation' into release-3.19Brad King2020-10-272-0/+7
| |\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | dd77dec18d VS: Don't compute CUDA options unless necessary e9109dec36 Merge branch 'ninja-multi-per-config-sources' into release-3.18 7c0de4175b Merge branch 'cmake-E-cat-binary' into release-3.18 Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !5422
| * \ \ \ \ Merge topic 'cmake-presets-path-arg' into release-3.19Craig Scott2020-10-276-7/+31
| |\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | b7d7eca66d CMakePresets.json: Rework how --preset argument is handled Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !5416
* | \ \ \ \ \ Merge topic 'fix-test-for-qemu'Kyle Edwards2020-11-021-0/+4
|\ \ \ \ \ \ \ | |_|_|_|_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 8c42cfb2e8 test/CMakeLib: make testUVProcessChain work with qemu Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !5442
| * | | | | | test/CMakeLib: make testUVProcessChain work with qemuEicke Herbertz2020-10-291-0/+4
| | |_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When building and testing CMake in a container with qemu user mode emulation, the expected termination with std::abort() in testUVProcessChainHelper leads qemu to emit an additional message about an uncaught signal. There appears to be no way to make qemu shut up, so any qemu message will be removed from the output during validation.
* | | | | | Merge topic 'CheckSourceRuns-compatibility'Brad King2020-10-302-2/+17
|\ \ \ \ \ \ | |/ / / / / |/| | | | / | | |_|_|/ | |/| | | | | | | | | | | | | a524bf3aa7 CheckSourceRuns: do not fail on unrecognized arguments Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !5426
| * | | | CheckSourceRuns: do not fail on unrecognized argumentsBen Boeckel2020-10-282-2/+17
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | The old `check_X_source_runs` modules did not verify the arguments, so we cannot start doing it now. Downgrade the hard error introduced in commit 357e2ef429 (CheckSoureRuns: Add a unified way to check if a source runs, 2020-09-14) via !5223 into a noisy warning.