summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Ninja: Revert "Exclude NVHPC -Werror flags during Fortran preprocessing"Brad King2023-04-262-24/+2
| | | | | | | | | Revert commit 6b58cdd4cf (Ninja: Exclude NVHPC -Werror flags during Fortran preprocessing, 2023-04-04). Since commit 9d40f01442 (NVHPC: Use -Werror for COMPILE_OPTIONS_WARNING_AS_ERROR, 2023-04-13), `nvfortran` can handle the plain `-Werror` flag during preprocessing. Issue: #24665
* Merge topic 'iwyu_errors'Brad King2023-04-268-3/+28
|\ | | | | | | | | | | | | | | fcacc319d9 IWYU: Return error code if user enables error reporting Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: buildbot <buildbot@kitware.com> Merge-request: !8406
| * IWYU: Return error code if user enables error reportingAshish Sadanandan2023-04-258-3/+28
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously CMake ignored the return code from iwyu because old versions of the tool would exit with an error code even when no header include violations were detected. The iwyu project has since changed this behavior, so the tool no longer returns an error code unless the user enables error reporting via command line arguments. Behavior seen with iwyu version 0.19 Source file with missing includes: - Case 1: iwyu arguments: - return code: 0 - output: <report of all missing includes> - Case 2: iwyu arguments: `-Xiwyu --error` - return code: 1 - output: <report of all missing includes> Source file with no missing includes: - Case 1: iwyu arguments: - return code: 0 - output: `(/path/to/file.cc has correct #includes/fwd-decls)` - Case 2: iwyu arguments: `-Xiwyu --error` - return code: 0 - output: `(/path/to/file.cc has correct #includes/fwd-decls)` Teach CMake to return the iwyu return code if the user has invoked the tool with any of these command line arguments included: - `--error[=N]` - `--error_always[=N]` Fixes: #24066
* | Merge topic 'vs2022-android'Brad King2023-04-264-17/+16
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 16d8b65bc0 VS/Android: Use safe API level when detecting the NDK directory f7af10100c VS/Android: Detect full NDK root instead of sysroot 850ee280e0 VS/Android: Set API level explicitly during compiler identification 5d5b6c741d VS/Android: Do not specify Windows Runtime library type during compiler id e78abf94e3 VS/Android: Use ApplicationTypeRevision 3.0 in VS2022 Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !8426
| * | VS/Android: Use safe API level when detecting the NDK directoryMichael Karcher2023-04-251-0/+1
| | |
| * | VS/Android: Detect full NDK root instead of sysrootMichael Karcher2023-04-252-14/+9
| | | | | | | | | | | | | | | | | | | | | Visual Studio always uses a complete NDK and not a standalone toolchain. Let CMake handle the NDK and related logic correctly, avoid trying to find the standalone toolchain version from the unified clang toolchain in newer NDKs.
| * | VS/Android: Set API level explicitly during compiler identificationMichael Karcher2023-04-252-0/+3
| | | | | | | | | | | | | | | VS2022 defaults to API 31 in 64-bit builds. This breaks if you combine VS2022 with an older Android NDK.
| * | VS/Android: Do not specify Windows Runtime library type during compiler idMichael Karcher2023-04-251-1/+1
| | | | | | | | | | | | | | | | | | The VS2022 Clang toolchain adds some windows libraries to the linked library list if RuntimeLibrary is set, even if the Project is targeting Android. This causes an unexpected linker failure.
| * | VS/Android: Use ApplicationTypeRevision 3.0 in VS2022Michael Karcher2023-04-252-2/+2
| | | | | | | | | | | | | | | | | | Visual Studio 17 (Marketing name: Visual Studio 2022) still ships with "3.0" as most recent Variant of the Android application type. Use this revision.
* | | Merge topic 'update-libarchive'Brad King2023-04-2638-115/+354
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ba5ced1be6 libarchive: Set build options the way we need for CMake 1e2bce305f Merge branch 'upstream-LibArchive' into update-libarchive e468170731 LibArchive 2022-12-09 (ba80276c) 744a42c0ee libarchive: Update script to get 3.6.2 Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: buildbot <buildbot@kitware.com> Merge-request: !8431
| * | | libarchive: Set build options the way we need for CMakeBrad King2023-04-252-1/+5
| | | |
| * | | Merge branch 'upstream-LibArchive' into update-libarchiveBrad King2023-04-2536-113/+348
| |\ \ \ | | | | | | | | | | | | | | | | | | | | * upstream-LibArchive: LibArchive 2022-12-09 (ba80276c)
| | * | | LibArchive 2022-12-09 (ba80276c)LibArchive Upstream2023-04-2538-117/+354
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Code extracted from: https://github.com/libarchive/libarchive.git at commit ba80276ccc3c941c4918ec6e2460059f0c525c43 (v3.6.2).
| * | | | libarchive: Update script to get 3.6.2Brad King2023-04-251-1/+1
| | | | |
* | | | | Merge topic 'fortran_ninja_compile_as_warning'Brad King2023-04-262-2/+2
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 9d40f01442 NVHPC: Use -Werror for COMPILE_OPTIONS_WARNING_AS_ERROR Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !8415
| * | | | | NVHPC: Use -Werror for COMPILE_OPTIONS_WARNING_AS_ERRORRobert Maynard2023-04-252-2/+2
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The previously-used `-Werror all-warnings` is not supported by the NVHPC suite of compilers. This previously worked since `-Werror` was being used and `all-warnings` was being excluded. We thought this was the correct syntax due to incorrect documentation about `-Werror`, which stated the argument should be space-separated, while it should actually be separated with `=` or `,`. Issue: #24665
* | | | | Merge branch 'release-3.26'Brad King2023-04-260-0/+0
|\ \ \ \ \
| * \ \ \ \ Merge topic 'FindCUDAToolkit-static-deps' into release-3.26Brad King2023-04-261-1/+5
| |\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 1d6c343661 FindCUDAToolkit: Add missing static library dependencies on pthread and libdl Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !8422
* | \ \ \ \ \ Merge topic 'FindCUDAToolkit-static-deps'Brad King2023-04-261-1/+5
|\ \ \ \ \ \ \ | | |/ / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 1d6c343661 FindCUDAToolkit: Add missing static library dependencies on pthread and libdl Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !8422
| * | | | | | FindCUDAToolkit: Add missing static library dependencies on pthread and libdlRobert Maynard2023-04-261-1/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Discourse-topic: https://discourse.cmake.org/t/7946
* | | | | | | Merge branch 'release-3.26'Brad King2023-04-260-0/+0
|\ \ \ \ \ \ \ | | |/ / / / / | |/| | | | |
| * | | | | | Merge topic 'FindCUDAToolkit_nvrtc_windows_dependencies' into release-3.26Brad King2023-04-261-1/+6
| |\ \ \ \ \ \ | | |/ / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 11a9f88bdf FindCUDAToolkit: Fix nvrtc_static dependencies on Windows Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !8405
* | | | | | | Merge topic 'FindCUDAToolkit_nvrtc_windows_dependencies'Brad King2023-04-261-1/+6
|\ \ \ \ \ \ \ | | |/ / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 11a9f88bdf FindCUDAToolkit: Fix nvrtc_static dependencies on Windows Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !8405
| * | | | | | FindCUDAToolkit: Fix nvrtc_static dependencies on WindowsRobert Maynard2023-04-251-1/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Required for CUDA toolkit 11.5: * https://docs.nvidia.com/cuda/archive/11.5.0/nvrtc/index.html#build-instruction
* | | | | | | Merge topic 'test-driver-use-anonymous-namespace'Marc Chevrier2023-04-261-4/+14
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ca2a84c3d7 TestDriver: Fix 'misc-use-anonymous-namespace' warning from clang-tidy 16 Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: buildbot <buildbot@kitware.com> Merge-request: !8413
| * | | | | | | TestDriver: Fix 'misc-use-anonymous-namespace' warning from clang-tidy 16Sylvain Joubert2023-04-251-4/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | C++ best practices recommend using anonymous namespace instead of a global static qualifier.
* | | | | | | | CMake Nightly Date StampKitware Robot2023-04-261-1/+1
| |_|_|/ / / / |/| | | | | |
* | | | | | | Merge topic 'CMake-uses-cmList-class'Brad King2023-04-2578-456/+440
|\ \ \ \ \ \ \ | |/ / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | e08ba229ee CMake code rely on cmList class for CMake lists management (part. 1) 51b0d45d91 cmExpandList and cmExpandLists rely on cmList class Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !8423
| * | | | | | CMake code rely on cmList class for CMake lists management (part. 1)Marc Chevrier2023-04-2477-381/+374
| | | | | | |
| * | | | | | cmExpandList and cmExpandLists rely on cmList classMarc Chevrier2023-04-233-75/+66
| | | | | | |
* | | | | | | CMake Nightly Date StampKitware Robot2023-04-251-1/+1
| | | | | | |
* | | | | | | CMake Nightly Date StampKitware Robot2023-04-241-1/+1
| | | | | | |
* | | | | | | CMake Nightly Date StampKitware Robot2023-04-231-1/+1
| | | | | | |
* | | | | | | Merge topic 'Link_instruction_in_RSP'Marc Chevrier2023-04-222-5/+35
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | e0364eb20d Add support of CMAKE_<LANG>_USE_RESPONSE_FILE_FOR_LIBRARIES variable. Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: buildbot <buildbot@kitware.com> Merge-request: !8409
| * | | | | | | Add support of CMAKE_<LANG>_USE_RESPONSE_FILE_FOR_LIBRARIES variable.Matthieu Ribiere2023-04-132-5/+35
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We add a function in cmNinjaNormalTargetGenerator.cxx to check for the state of the option (as it was done for Makefile). This will be checked to add or not the `$LINK_PATH` and `$LINK_LIBRARIES` to the Ninja file. The default behavior is adding those libraries. Fixes: #24681
* | | | | | | | Merge topic 'emacs-mode-shell-quote-fix'Marc Chevrier2023-04-221-2/+2
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 0722569cf3 Properly handle cases where `topic` argument is defaulted Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !8424
| * | | | | | | | Properly handle cases where `topic` argument is defaultedDamien Nguyen2023-04-211-2/+2
| | |_|_|_|_|/ / | |/| | | | | |
* | | | | | | | CMake Nightly Date StampKitware Robot2023-04-221-1/+1
|/ / / / / / /
* | | | | | | CMake Nightly Date StampKitware Robot2023-04-211-1/+1
| |/ / / / / |/| | | | |
* | | | | | Merge topic 'GenEx-list'Marc Chevrier2023-04-20152-37/+2382
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 31675964e7 GenEx LIST: list operations Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !8410
| * | | | | | GenEx LIST: list operationsMarc Chevrier2023-04-16152-37/+2382
| | |_|_|_|/ | |/| | | | | | | | | | | | | | | | Fixes: #24550, #24547
* | | | | | CMake Nightly Date StampKitware Robot2023-04-201-1/+1
| | | | | |
* | | | | | CMake Nightly Date StampKitware Robot2023-04-191-1/+1
| | | | | |
* | | | | | CMake Nightly Date StampKitware Robot2023-04-181-1/+1
| | | | | |
* | | | | | Merge topic 'doc-workflow-presets-vendor'Craig Scott2023-04-171-0/+6
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | dd8ac2f5ca Help: Add missing vendor field to the workflow presets docs Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !8419
| * | | | | | Help: Add missing vendor field to the workflow presets docsCraig Scott2023-04-151-0/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The JSON schema already allows a vendor field for workflow presets, but it was missed in the documentation. There is no change in behavior or support, this just fixes an oversight in the docs when the workflow presets were originally added.
* | | | | | | Merge topic 'export-file-set-old-cmake-versions'Craig Scott2023-04-171-1/+16
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 42e5be8cef install(EXPORT): Fall back to set_property(APPEND) for FILE_SET Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: buildbot <buildbot@kitware.com> Merge-request: !8408
| * | | | | | | install(EXPORT): Fall back to set_property(APPEND) for FILE_SETKyle Edwards2023-04-121-1/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | FILE_SET was introduced in CMake 3.23, so install(EXPORT) puts it behind a version gate. However, this results in the include directories not being picked up by older versions of CMake. Fall back to set_property(APPEND) for versions of CMake older than 3.23. Fixes: #24787
* | | | | | | | Merge branch 'release-3.26'Marc Chevrier2023-04-170-0/+0
|\ \ \ \ \ \ \ \ | | |_|_|_|_|/ / | |/| | | | | |
| * | | | | | | Merge topic 'Help-CMP0105-clarifications' into release-3.26Marc Chevrier2023-04-171-2/+3
| |\ \ \ \ \ \ \ | | |_|_|_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | afd40232fa Help: CMP0105 policy: clarifications Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !8418