summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Makefile: Improve const correctnessOrkun Tokdemir2024-01-091-15/+16
|
* Makefile: Remove no-op callOrkun Tokdemir2024-01-091-2/+0
|
* CMake Nightly Date StampKitware Robot2024-01-091-1/+1
|
* Merge topic 'link-stubs-transitively'Brad King2024-01-0812-11/+112
|\ | | | | | | | | | | | | | | | | | | 2c6ec6de15 Link to transitive dependencies on stub libraries only on some linkers dd4a6dff92 Link explicitly to private transitive dependencies on stub libraries 5f1bbdb3b3 Tests: Enable RunCMake.RuntimePath test on more platforms Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: buildbot <buildbot@kitware.com> Merge-request: !9050
| * Link to transitive dependencies on stub libraries only on some linkersBrad King2024-01-043-1/+19
| | | | | | | | | | Only linkers that enforce `--no-allow-shlib-undefined` recursively need to link private transitive dependencies on stub libraries explicitly.
| * Link explicitly to private transitive dependencies on stub librariesBrad King2024-01-0410-3/+84
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We represent stub libraries, e.g., for CUDA, using imported `SHARED` library targets with only `IMPORTED_IMPLIB`, and no `IMPORTED_LOCATION`, to indicate that the stub file is meant only for linkers and not dynamic loaders. See commit 7351d590ee (cmTarget: Add a way to represent imported shared library stubs, 2023-07-17, v3.28.0-rc1~344^2) and commit fc6508921c (cmComputeLinkInformation: Restore soname lookup for non-imported targets, 2023-12-05, v3.28.0~4^2). If a shared library is linked to a stub, it has a `NEEDED` field populated with the `SONAME` found in the stub. When a dependent target links to such a shared library, some linkers want to find a library file on disk and load it to see what symbols it provides. This is necessary for linkers that enforce `--no-allow-shlib-undefined`. On hosts with only the stub library installed, e.g., with only the CUDA toolkit development package, the real runtime library corresponding to the stub's `SONAME` may not even exist, so no `-rpath-link` flag can help linkers find it. Pass the stub library to linkers explicitly so they can find it without searching.
| * Tests: Enable RunCMake.RuntimePath test on more platformsBrad King2024-01-042-9/+11
| | | | | | | | Move the "ELF" check to specific cases inside the test.
* | Merge topic 'fix-isspace-usage-for-signed-char'Brad King2024-01-081-1/+5
|\ \ | | | | | | | | | | | | | | | | | | | | | 5e8c176e2a cmExecuteProcessCommand: Cast c to unsigned char before cast to int Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: buildbot <buildbot@kitware.com> Merge-request: !9132
| * | cmExecuteProcessCommand: Cast c to unsigned char before cast to intleha-bot2024-01-051-1/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | As the 'char' type may be either signed, or unsigned, there are some clashes between C Standard library functions and actual characters while casting it to int directly. In case the 'char' type was signed, the casted to int result value may be extended to full negative digit which may be out of range of isspace() function (e.g. , for MSVC implementation, which checks it for '> -1', and throwing an assertion failure on fail). Fixes: #25561
* | | Merge topic 'fix-tutorial'Brad King2024-01-083-6/+6
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 6d48bd21ac Tutorial: Clarify the expected change to target_include_directories 4d56db038d Tutorial: Make `INSTALL_DESTINATION` consistent with installed destination da22b269b2 Tutorial: Add missing `$` in generator expression Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: buildbot <buildbot@kitware.com> Merge-request: !9120
| * | | Tutorial: Clarify the expected change to target_include_directoriesKevin Albertson2024-01-051-4/+4
| | | | | | | | | | | | | | | | | | | | The expectation is to modify the existing `target_include_directories`, not add add a new `target_include_directories` command.
| * | | Tutorial: Make `INSTALL_DESTINATION` consistent with installed destinationKevin Albertson2024-01-051-1/+1
| | | |
| * | | Tutorial: Add missing `$` in generator expressionKevin Albertson2024-01-051-1/+1
| | | |
* | | | Merge topic 'add_test-CROSSCOMPILING_EMULATOR-exclusive'Brad King2024-01-082-1/+3
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 63e20653f8 add_test: Fix version of CMake introducing CMP0158 (to 3.29) Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: buildbot <buildbot@kitware.com> Merge-request: !9134
| * | | | add_test: Fix version of CMake introducing CMP0158 (to 3.29)Robert Maynard2024-01-052-1/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This was missed in commit ca5a300d7f (add_test: Honor CROSSCOMPILING_EMULATOR only when cross-compiling, 2023-11-02) when it slipped from the 3.28 release.
* | | | | Merge topic 'cmake-gui_better_text_with_preset'Brad King2024-01-083-1/+6
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 9d13674337 cmake-gui: Hint that the preset compilers will be used Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: buildbot <buildbot@kitware.com> Acked-by: Alex <leha-bot@yandex.ru> Merge-request: !9125
| * | | | | cmake-gui: Hint that the preset compilers will be usedRobert Maynard2024-01-053-1/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Instead of stating that the default is the native compilers, say we will use the compilers from the preset. This makes it more clear that the preset is working as expected.
* | | | | | Merge topic 'cmake-gui_support_preset_toolchain'Brad King2024-01-081-37/+59
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 370370364b cmake-gui: Add support for preset toolchain entry Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: buildbot <buildbot@kitware.com> Acked-by: Alex <leha-bot@yandex.ru> Merge-request: !9126
| * | | | | | cmake-gui: Add support for preset toolchain entryRobert Maynard2024-01-051-37/+59
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously the toolchain field was ignored when passing a preset to cmake-gui. Fixes: #24034
* | | | | | | Merge branch 'release-3.28'Brad King2024-01-080-0/+0
|\ \ \ \ \ \ \
| * \ \ \ \ \ \ Merge topic 'fix-openbsd' into release-3.28Brad King2024-01-083-7/+7
| |\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 1fafe35e81 Source: Restore compilation on OpenBSD Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !9085
* | \ \ \ \ \ \ \ Merge topic 'fix-openbsd'Brad King2024-01-083-7/+7
|\ \ \ \ \ \ \ \ \ | | |/ / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 1fafe35e81 Source: Restore compilation on OpenBSD Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !9085
| * | | | | | | | Source: Restore compilation on OpenBSDRafael Sadowski2024-01-053-7/+7
| |/ / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | We cannot use `OpenBSD` as a name, it is defined in `sys/param.h`.
* | | | | | | | CMake Nightly Date StampKitware Robot2024-01-081-1/+1
| | | | | | | |
* | | | | | | | CMake Nightly Date StampKitware Robot2024-01-071-1/+1
| | | | | | | |
* | | | | | | | CMake Nightly Date StampKitware Robot2024-01-061-1/+1
| |_|_|_|_|/ / |/| | | | | |
* | | | | | | Merge topic 'FindPkgConfig-caching'Brad King2024-01-051-0/+6
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | e50aaf8547 FindPkgConfig: Update cache variables after isystem extraction 8fe6196714 FindPkgConfig: Update cache variables after framework extraction Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: buildbot <buildbot@kitware.com> Merge-request: !9109
| * | | | | | | FindPkgConfig: Update cache variables after isystem extractionKai Pastor2024-01-041-0/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes: #25377
| * | | | | | | FindPkgConfig: Update cache variables after framework extractionKai Pastor2023-12-281-0/+3
| | | | | | | |
* | | | | | | | Merge topic 'swift-always-restat'Brad King2024-01-054-2/+14
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 1161ad76ac Swift/Ninja: Always restat swift build commands Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !9076
| * | | | | | | | Swift/Ninja: Always restat swift build commandsJeremy Day2024-01-044-2/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The swift toolchain leaves output files untouched if there are no meaningful input changes; without restat, this causes ninja to needlessly rebuild targets that are not actually out-of-date Fixes: #25496
* | | | | | | | | Merge topic 'swift-compile-commands'Brad King2024-01-055-20/+128
|\ \ \ \ \ \ \ \ \ | |_|_|_|_|_|/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 44f29a4291 Swift/Ninja: Fix multifile module compile commands Acked-by: Kitware Robot <kwrobot@kitware.com> Tested-by: buildbot <buildbot@kitware.com> Merge-request: !9095
| * | | | | | | | Swift/Ninja: Fix multifile module compile commandsEvan Wilde2024-01-035-20/+128
| | |_|_|_|/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Swift compile commands need to have all source files in the module specified in the compile command or LSP systems will report errors on missing types that are defined in other source files in the same module. Issue: #25491
* | | | | | | | Merge topic 'ccmake_support_preset_toolchain'Brad King2024-01-051-0/+5
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | d20c21a678 ccmake: Now supports the preset toolchain entry Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: buildbot <buildbot@kitware.com> Acked-by: Alex <leha-bot@yandex.ru> Merge-request: !9127
| * | | | | | | | ccmake: Now supports the preset toolchain entryRobert Maynard2024-01-031-0/+5
| | |_|_|_|/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously the toolchain field was ignored when passing a preset to ccmake.
* | | | | | | | Merge topic 'test-RunCMake.ExternalProject'Brad King2024-01-051-0/+3
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 3253bc306e Tests: Optionally run RunCMake.ExternalProject serially Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !9129
| * | | | | | | | Tests: Optionally run RunCMake.ExternalProject seriallyBrad King2024-01-041-0/+3
| | |_|_|_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | On some machines running many tests concurrently, the `INACTIVITY_TIMEOUT` cases do not always complete within their individual timeout. Add an undocumented cache entry to use on those machines to run the test serially.
* | | | | | | | Merge branch 'release-3.28'Brad King2024-01-050-0/+0
|\ \ \ \ \ \ \ \ | | |_|_|_|_|/ / | |/| | | | | |
| * | | | | | | Merge topic 'doc-get-properties-inherited' into release-3.28Brad King2024-01-057-34/+32
| |\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 8086badba4 Help: Fix wrong return values for unset inherited properties 1ca5ec67b0 Help: Add missing DIRECTORY keyword to set_tests_properties() signature 34379f005b Help: Use <variable> consistently in property getter commands 29ccc9a4cd Help: Use <xxx> for non-keyword arguments in property command signatures Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !9131
* | \ \ \ \ \ \ \ Merge topic 'doc-get-properties-inherited'Brad King2024-01-057-34/+32
|\ \ \ \ \ \ \ \ \ | | |/ / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 8086badba4 Help: Fix wrong return values for unset inherited properties 1ca5ec67b0 Help: Add missing DIRECTORY keyword to set_tests_properties() signature 34379f005b Help: Use <variable> consistently in property getter commands 29ccc9a4cd Help: Use <xxx> for non-keyword arguments in property command signatures Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !9131
| * | | | | | | | Help: Fix wrong return values for unset inherited propertiesCraig Scott2024-01-053-22/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since ff6234509e (Help: Clarify behavior of INHERITED properties, 2018-03-21), the docs for some get_..._property() commands incorrectly describe the behavior for inherited properties. When a property is not set, even in a parent scope, the returned result from the get_..._property() command is the same whether the property is inherited or not. The docs incorrectly stated that an empty string would be returned for inherited properties in such cases.
| * | | | | | | | Help: Add missing DIRECTORY keyword to set_tests_properties() signatureCraig Scott2024-01-051-0/+1
| | | | | | | | |
| * | | | | | | | Help: Use <variable> consistently in property getter commandsCraig Scott2024-01-053-14/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Property-related commands used a mix of <VAR>, <var>, or <variable> to specify the variable to store the result in. The <VAR> form is particularly confusing, since being uppercase it looks more like a keyword. Use <variable> consistently across all the commands so that the behavior is clear.
| * | | | | | | | Help: Use <xxx> for non-keyword arguments in property command signaturesCraig Scott2024-01-055-12/+15
| | | | | | | | |
* | | | | | | | | Merge branch 'release-3.28'Brad King2024-01-050-0/+0
|\ \ \ \ \ \ \ \ \ | | |/ / / / / / / | |/| | | | | | |
| * | | | | | | | Merge topic 'check_language_propagate_hip_platform' into release-3.28Brad King2024-01-054-2/+64
| |\ \ \ \ \ \ \ \ | | |/ / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ce9c6d0994 HIP: Propagate CMAKE_HIP_PLATFORM from/to the test project in check_language 9ba3fc91e5 HIP: Really forward CMAKE_HIP_HOST_COMPILER in check_language(HIP) Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !9121
* | | | | | | | | Merge topic 'check_language_propagate_hip_platform'Brad King2024-01-054-2/+64
|\ \ \ \ \ \ \ \ \ | | |/ / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ce9c6d0994 HIP: Propagate CMAKE_HIP_PLATFORM from/to the test project in check_language 9ba3fc91e5 HIP: Really forward CMAKE_HIP_HOST_COMPILER in check_language(HIP) Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !9121
| * | | | | | | | HIP: Propagate CMAKE_HIP_PLATFORM from/to the test project in check_languageGergely Meszaros2024-01-044-1/+63
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes: #25541
| * | | | | | | | HIP: Really forward CMAKE_HIP_HOST_COMPILER in check_language(HIP)Gergely Meszaros2024-01-031-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In commit b3e92775ab (HIP: Add CMAKE_HIP_HOST_COMPILER when compiler is NVCC, 2023-09-25, v3.28.0-rc1~44^2~2) we accidentally left this out.
* | | | | | | | | CMake Nightly Date StampKitware Robot2024-01-051-1/+1
| |_|/ / / / / / |/| | | | | | |