summaryrefslogtreecommitdiffstats
path: root/Tests
Commit message (Collapse)AuthorAgeFilesLines
* Merge topic 'clang-scan-deps-failed-scan'Brad King2023-11-221-2/+2
|\ | | | | | | | | | | | | | | 77a7edb73f Clang-CXX: copy into the dyndep output on success Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: Namniav W <namniav@gmail.com> Merge-request: !8991
| * Clang-CXX: copy into the dyndep output on successBen Boeckel2023-11-211-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When `clang-scan-deps` fails to scan (e.g., bad source syntax, junk flags, etc.), the redirection unconditionally updates the file. If this fails, the `.ddi` file timestamp is updated. If the state is then reverted (e.g., the command line returns to the state of the last successful build), the updated file is not useful, but `ninja` does not rerun because: - the command hash matches the last successful run - the output file is newer than its inputs However, since the `.ddi` file has been updated with bogus contents from a failed scan, collation fails as the `rules` array is empty (or incomplete from a batch scan). If `clang-scan-deps` were properly aware of its output file, it could use this to not write the file if any inner scan fails. Requested in https://github.com/llvm/llvm-project/issues/72875. See: https://github.com/llvm/llvm-project/issues/72875 Fixes: #25429
* | cmUVHandlePtr: Add uv_idle_ptr::{start,stop} methodsBrad King2023-11-201-1/+12
| | | | | | | | These were missing w.r.t. the pattern established for other handle wrappers.
* | cmUVHandlePtr: Add uv_timer_ptr::stop methodBrad King2023-11-201-0/+10
| | | | | | | | This was missing w.r.t. the pattern established for other handle wrappers.
* | Tests: Add dedicated test case for uv_timer_ptrBrad King2023-11-201-0/+27
| |
* | Tests: Factor out callback in uv_idle_ptr test caseBrad King2023-11-201-4/+6
| |
* | Tests: Add dedicated test to cover cmUVHandlePtr typesBrad King2023-11-203-25/+37
| | | | | | | | | | Move the case added by commit 70d88a5361 (cmUVHandlePtr: Add uv_idle_ptr, 2023-11-06) to a dedicated test.
* | Merge topic 'fileapi-file-sets-base-dirs-relative'Brad King2023-11-202-5/+5
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | a3a85524cd fileapi: Fix file sets' base directories relative to top source Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: buildbot <buildbot@kitware.com> Reviewed-by: Ben Boeckel <ben.boeckel@kitware.com> Merge-request: !8977
| * | fileapi: Fix file sets' base directories relative to top sourceKyle Edwards2023-11-172-5/+5
| | | | | | | | | | | | | | | | | | | | | | | | This field was added by commit b3e9fb67bb (file-api: support exporting file set information, 2022-11-03, v3.26.0-rc1~389^2) but the relative path convention used elsewhere was accidentally left out. Fixes: #25422
* | | Merge topic 'ctest-cleanup'Brad King2023-11-208-15/+76
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 5d2e93f9e8 cmCTestMultiProcessHandler: Simplify logic on unavailable resources a4b061a035 cmCTestMultiProcessHandler: Clarify resource availability error member names 1487e540aa cmCTestMultiProcessHandler: Reduce repeat test property map lookups b02b628ad9 cmCTestMultiProcessHandler: Simplify loop termination on serial test 8f1e8af0cc cmCTestMultiProcessHandler: Stop searching for tests when limit is reached bd0b4ca867 cmCTestMultiProcessHandler: Invert spare load condition 9b548139fd cmCTestMultiProcessHandler: Clarify search for tests <= concurrency limit ee321dc85f cmCTestMultiProcessHandler: Clarify search for tests <= spare load ... Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: buildbot <buildbot@kitware.com> Merge-request: !8981
| * | | cmCTestMultiProcessHandler: Stop searching for tests when limit is reachedBrad King2023-11-175-0/+55
| | | | | | | | | | | | | | | | Avoid an extra loop iteration if we have no room for more tests.
| * | | Tests: Cover ctest waiting on insufficient non-zero spare test-loadBrad King2023-11-175-15/+21
| | | |
* | | | Merge topic 'ci-fedora-39'Brad King2023-11-2011-15/+19
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | cdd741ebf9 Merge branch 'backport-ci-fedora-39' into ci-fedora-39 9283b20659 ci: Suppress CPack/RPM tests pending fix for Fedora 39 18145e8745 ci: Update FindMPI test environment for mpich on Fedora 39 a8be80ccf2 ci: Drop now-unnecessary Clang rules for CXXModules tests 99238b23e9 ci: use Fedora 39 images and environments 57eadec617 ci: update Linux image to Fedora 39 653262162c clang-tidy module: Update to build against LLVM/Clang 17 2cf9a65835 clang-tidy: ignore warnings new in version 17 ... Acked-by: Kitware Robot <kwrobot@kitware.com> Tested-by: buildbot <buildbot@kitware.com> Merge-request: !8983
| * \ \ \ Merge branch 'backport-ci-fedora-39' into ci-fedora-39Brad King2023-11-1711-15/+19
| |\ \ \ \ | | |/ / / | |/| | / | | | |/ | | |/|
| | * | codespell: Fix typosBrad King2023-11-1710-11/+11
| | | |
| | * | Tests: Update HIP.MathFunctions case for nodiscard enforcementBrad King2023-11-171-4/+8
| | | |
* | | | Merge topic 'uv-idle-ptr'Brad King2023-11-191-3/+29
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 70d88a5361 cmUVHandlePtr: Add uv_idle_ptr 17690558c3 cmUVHandlePtr: Add explicit conversion to bool cd2894a089 cmUVHandlePtr: Conversions to raw pointers are const Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: buildbot <buildbot@kitware.com> Merge-request: !8980
| * | | | cmUVHandlePtr: Add uv_idle_ptrBrad King2023-11-171-0/+26
| | | | | | | | | | | | | | | | | | | | Wrap a `uv_idle_t` handle.
| * | | | cmUVHandlePtr: Add explicit conversion to boolBrad King2023-11-171-3/+3
| |/ / /
* | | | Merge topic 'swift-compilation-mode'Brad King2023-11-1914-8/+77
|\ \ \ \ | |/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 0f80101b73 Tests: Update Swift tests to use CMP0157 NEW behavior c1d787e473 Swift: Add abstraction for compilation mode c39384f540 Tests: Simplify RunCMake.Swift conditions to enable use of Swift Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: buildbot <buildbot@kitware.com> Merge-request: !8918
| * | | Tests: Update Swift tests to use CMP0157 NEW behaviorEvan Wilde2023-11-175-0/+5
| | | | | | | | | | | | | | | | | | | | Fixing failing tests caused by introduction of new policy + warning when the policy is not set.
| * | | Swift: Add abstraction for compilation modeEvan Wilde2023-11-178-0/+66
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add a `CMAKE_Swift_COMPILATION_MODE` variable and corresponding `Swift_COMPILATION_MODE` target property to control the compilation mode. Select among `wholemodule`, `singlefile`, and `incremental`. Add policy CMP0157 to remove the default `-wmo` flags in favor of the abstract setting. Issue: #25366
| * | | Tests: Simplify RunCMake.Swift conditions to enable use of SwiftBrad King2023-11-172-8/+6
| | | |
* | | | Merge topic 'install-export-xcframework'Brad King2023-11-1619-0/+203
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 37bc3400cd CMakePackageConfigHelpers: Add generate_apple_platform_selection_file() 256bb0cc40 install(EXPORT): Add ability to point to .xcframework file a90968e044 cmExportBuildFileGenerator: Add structs for target exports Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: Jason Juang <jasjuang@gmail.com> Merge-request: !8969
| * | | | CMakePackageConfigHelpers: Add generate_apple_platform_selection_file()Kyle Edwards2023-11-156-0/+39
| | | | | | | | | | | | | | | | | | | | Issue: #25262
| * | | | install(EXPORT): Add ability to point to .xcframework fileKyle Edwards2023-11-1515-0/+164
| | | | | | | | | | | | | | | | | | | | Issue: #25262
* | | | | Merge topic 'target-objects'Brad King2023-11-155-0/+33
|\ \ \ \ \ | |/ / / / |/| | / / | | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | d7988ff6b8 Merge branch 'backport-target-objects' into target-objects 1814853081 cmCommonTargetGenerator: Drop unused local variable 377b78aef9 cmComputeLinkInformation: Simplify recording OBJECT libraries as link items 50fdaf8f1f cmComputeLinkInformation: Track targets named by TARGET_OBJECTS sources 0e26bd334d cmCommonTargetGenerator: Factor out GetLinkedTargetDirectories loop body Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: buildbot <buildbot@kitware.com> Reviewed-by: Ben Boeckel <ben.boeckel@kitware.com> Merge-request: !8974
| * | | Merge branch 'backport-target-objects' into target-objectsBrad King2023-11-145-0/+33
| |\ \ \
| | * | | cmComputeLinkInformation: Track targets named by TARGET_OBJECTS sourcesBrad King2023-11-145-0/+33
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since commit b6a5382217 (Ninja: depend on language module information files directly, 2023-02-10, v3.27.0-rc1~502^2), the return value of `cmCommonTargetGenerator::GetLinkedTargetDirectories` must account for linked object libraries because they may provide modules (#25112). These were added by commit b665966933 (cmComputeLinkInformation: track OBJECT library dependencies, 2023-07-22, v3.27.1~5^2). However, targets named by `$<TARGET_OBJECTS:...>` sources are also needed (#25365). The latter were added by commit 22da18b995 (Fortran: Restore support for TARGET_OBJECTS providing modules, 2023-10-27, v3.28.0-rc4~9^2) and commit 035302b7e3 (cmComputeLinkDepends: also copy the target from object link items, 2023-10-27, v3.28.0-rc4~9^2~2). However, their approach added link entries not actually specified by projects. It also incorrectly re-used `cmComputeLinkDepends::AddLinkObject` for object library targets when it is meant for their individual object files. These problems caused additional regressions (#25417). Revert the implementation parts of those commits and leave behind an assertion and comment to help avoid the mistake in the future. Instead, track targets named by `$<TARGET_OBJECTS:...>` sources with a dedicated member. Issue: #25112 Issue: #25365 Fixes: #25417 Co-authored-by: Ben Boeckel <ben.boeckel@kitware.com>
* | | | | Merge topic 'export-find_dependency-calls'Brad King2023-11-1431-1/+340
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 0ad8fc5a63 FetchContent: Set CMAKE_EXPORT_FIND_PACKAGE_NAME c6e6861e63 install(EXPORT): Export find_dependency() calls 2837f592ab cmExportFileGenerator: Add function to set required CMake version e62a3b8625 Help/export: Add signature directives Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: Brad King <brad.king@kitware.com> Acked-by: Jason Juang <jasjuang@gmail.com> Merge-request: !8957
| * | | | | FetchContent: Set CMAKE_EXPORT_FIND_PACKAGE_NAMEKyle Edwards2023-11-136-0/+110
| | | | | |
| * | | | | install(EXPORT): Export find_dependency() callsKyle Edwards2023-11-1325-1/+230
| | |_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | Issue: #20511 Co-Authored-by: Brad King <brad.king@kitware.com> Co-Authored-by: Robert Maynard <rmaynard@nvidia.com>
* | | | | Merge topic 'sources-per-config'Brad King2023-11-142-0/+16
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 16c5977504 Fix per-config sources in multi-config generators when first config adds none Acked-by: Kitware Robot <kwrobot@kitware.com> Tested-by: buildbot <buildbot@kitware.com> Merge-request: !8971
| * | | | | Fix per-config sources in multi-config generators when first config adds noneBrad King2023-11-132-0/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since commit b1c3ae33ea (cmTarget: Short-circuit language computation if context independent., 2014-04-09, v3.1.0-rc1~669^2~1) we've tried to avoid repeating computation of the list of sources for a target for every configuration in the case that a per-config source (or object library) contributes zero sources. However, it is possible that an entry contributes zero sources in the first configuration processed but at least one source in other configurations. Fixes: #25400
* | | | | | Merge topic 'genex-fix-short-circuit'Brad King2023-11-142-0/+5
|\ \ \ \ \ \ | |_|/ / / / |/| | / / / | | |/ / / | |/| | | | | | | | | | | | | | | | | | 27244a8f73 cmGeneratorExpressionNode: Fix short-circuit logic Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: buildbot <buildbot@kitware.com> Merge-request: !8966
| * | | | cmGeneratorExpressionNode: Fix short-circuit logicMartin Duffy2023-11-132-0/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix logic added by commit 634079b86d (cmGeneratorExpressionEvaluator: Short-circuit boolean operators, 2023-09-11, v3.28.0-rc1~47^2) and add missing test cases. Fixes: #25412
| * | | | Merge topic 'cxx23' into release-3.28Brad King2023-11-082-3/+3
| |\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | a0fabc4769 cmGlobalGeneratorFactory: Provide complete cmGlobalGenerator to deleter 85627a93c9 cmCTestBuildCommand: Avoid requiring complete cmGlobalGenerator type publicly 641c02a3ce cmList: Avoid using operator-> on input iterator e4483b8871 Tests: Avoid compiling call to dap::optional<dap::string>(nullptr) Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: buildbot <buildbot@kitware.com> Acked-by: scivision <michael@scivision.dev> Merge-request: !8950
* | \ \ \ \ Merge topic 'make-test-depend-on-all'Brad King2023-11-1310-0/+38
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 5e0c1777a3 Optionally make `test` target depend on `all` eaa00d4dfa Tests: Add case to verify that 'make test' does not 'make all' Acked-by: Kitware Robot <kwrobot@kitware.com> Tested-by: buildbot <buildbot@kitware.com> Acked-by: Jason Juang <jasjuang@gmail.com> Merge-request: !8956
| * | | | | | Optionally make `test` target depend on `all`William Sciaroni2023-11-104-0/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes: #8774
| * | | | | | Tests: Add case to verify that 'make test' does not 'make all'Brad King2023-11-107-0/+29
| | |_|/ / / | |/| | | | | | | | | | | | | | | | | | | | | | Place it in a new `RunCMake.BuiltinTargets` umbrella test meant to cover behavior of targets builtin to CMake.
* | | | | | Merge topic 'link-deduplicate-libs'Brad King2023-11-13115-24/+480
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 7b99c42e57 Link step: Enable to configure deduplication of libraries 07501c1678 Link Step: Introduce EntriesProcessing class Acked-by: Kitware Robot <kwrobot@kitware.com> Tested-by: buildbot <buildbot@kitware.com> Acked-by: scivision <michael@scivision.dev> Merge-request: !8946
| * | | | | | Link step: Enable to configure deduplication of librariesMarc Chevrier2023-11-09115-24/+480
| |/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Some platforms, Apple or Windows for instance, do not require to duplicate static libraries to resolve mutual dependencies. Moreover, Xcode version 15 emits a warning if a library is duplicated. On Windows, enable a better control of libraries order. Fixes: #20722, #25297
* | | | | | ctest_empty_binary_directory: Report more detail in failure messageBrad King2023-11-096-0/+24
|/ / / / /
* | | | | Merge topic 'cuda_support_CMAKE_TRY_COMPILE_TARGET_TYPE'Brad King2023-11-083-0/+12
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | d7642a7d42 CUDA: Support CMAKE_TRY_COMPILE_TARGET_TYPE of STATIC_LIBRARY Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !8953
| * | | | | CUDA: Support CMAKE_TRY_COMPILE_TARGET_TYPE of STATIC_LIBRARYRobert Maynard2023-11-073-0/+12
| | | | | |
* | | | | | Merge topic 'cxx23'Brad King2023-11-082-3/+3
|\ \ \ \ \ \ | | |_|/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | a0fabc4769 cmGlobalGeneratorFactory: Provide complete cmGlobalGenerator to deleter 85627a93c9 cmCTestBuildCommand: Avoid requiring complete cmGlobalGenerator type publicly 641c02a3ce cmList: Avoid using operator-> on input iterator e4483b8871 Tests: Avoid compiling call to dap::optional<dap::string>(nullptr) Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: buildbot <buildbot@kitware.com> Acked-by: scivision <michael@scivision.dev> Merge-request: !8950
| * | | | | Tests: Avoid compiling call to dap::optional<dap::string>(nullptr)Brad King2023-11-072-3/+3
| | | | | |
* | | | | | Merge topic 'static-library-link-xcframework'Brad King2023-11-073-0/+12
|\ \ \ \ \ \ | |_|/ / / / |/| | / / / | | |/ / / | |/| | | | | | | | | | | | | 7d19246138 Xcode: Fix linking against .xcframework from static libraries Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !8949
| * | | | Xcode: Fix linking against .xcframework from static librariesKyle Edwards2023-11-063-0/+12
| |/ / / | | | | | | | | | | | | Issue: #21752
* | | | Merge topic 'vs-CharacterSet'Brad King2023-11-073-0/+69
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | cbddc66277 VS: Consider macros with values when determining CharacterSet Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !8940
generated by cgit v0.12 at 2025-03-20 14:16:21 (GMT)