summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Source: Avoid out-of-range inputs to std::isspace()Brad King2024-01-178-23/+18
| | | | | | | | | | | | | `isspace` takes `int` but documents that the value must be representable by `unsigned char`, or be EOF. Use a wrapper to cast to `unsigned char` to avoid sign extension while converting to `int`. This generalizes the fix from commit 5e8c176e2a (cmExecuteProcessCommand: Cast c to unsigned char before cast to int, 2024-01-05) to other `isspace` call sites. This was detected by assertions in the MSVC standard library while processing UTF-8 text. Issue: #25561
* cmXMLParser: Remove unused IsSpace methodBrad King2024-01-172-10/+0
|
* Merge topic 'cmake-language-exit-code'Brad King2024-01-1734-5/+126
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | 1bb1769235 cmake_language: Add EXIT subcommand 4f160f7906 cmakemain: Return the SCRIPT_MODE exit code if it was set b62dcbf5d2 cmMakefile: check cmake script mode exit code after command 3d9d504646 cmMakefile: Store the exit code from cmExecutionStatus to cmake instance 9f6c937408 Source/cmake.h: Add ScriptModeExitCode for proper storing exit code 1082b9cb9a cmExecutionStatus: Add ability to set optional custom exit code Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: buildbot <buildbot@kitware.com> Acked-by: Leonid Pospelov <pospelovlm@yandex.ru> Acked-by: NoMaY (a user of Renesas Rulz Japanese Forum) <nomay-jp@outlook.com> Merge-request: !8228
| * cmake_language: Add EXIT subcommandleha-bot2024-01-1629-0/+93
| | | | | | | | | | | | | | | | | | | | | | | | Add tests to cover these cases: * run as regular CMake module, in NORMAL_MODE (expected to fail); * run as CMake script in SCRIPT_MODE (expected to exit with given code); * run as CMake script that `include()`-s another script with EXIT subcommand; * run as CMake script which EVAL-uates EXIT subcommand via `cmake_language(EVAL CODE "<cmake code>")`. Fixes: #23162
| * cmakemain: Return the SCRIPT_MODE exit code if it was setleha-bot2024-01-162-5/+8
| | | | | | | | | | | | | | We determine it via checking cmake.Run() return code and ScriptModeExitCode. If ScriptModeExitCode is zero, then we would return 1 if retcode was non-zero. Otherwise it will be returned if retcode was also non-zero.
| * cmMakefile: check cmake script mode exit code after commandleha-bot2024-01-161-0/+6
| | | | | | | | | | Which potentially may set it, and forward it to the cmExecutionStatus for proper handling in caller.
| * cmMakefile: Store the exit code from cmExecutionStatus to cmake instanceleha-bot2024-01-161-0/+5
| |
| * Source/cmake.h: Add ScriptModeExitCode for proper storing exit codeleha-bot2024-01-161-0/+6
| | | | | | | | From executed CMake script in SCRIPT_MODE.
| * cmExecutionStatus: Add ability to set optional custom exit codeleha-bot2024-01-161-0/+8
| |
* | Merge topic 'lcc-updates-2024-01'Brad King2024-01-1711-17/+33
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 202f4b3161 Tests: Exclude some tests on broken libc on Elbrus 9dd0ab9c88 LCC: Make CMake build without warnings on LCC 1.21 1dbb31cea2 libarchive: avoid lchmod not implemented warning on old LCC 83af26d9ad LCC: Don't enable debugger on LCC that don't have <future> fa764ce311 liblzma: Make cmliblzma buildable on LCC 1.21 77e046b47c jsoncpp: fix missing template deletion support on LCC < 1.23 67de0c197b cmcurl: fix X509_STORE_up_ref issue not just on LCC 1.23, but on LCC <= 1.23 9bc2aba3b4 LCC: get rid of ambiguous assignments of {} for LCC ... Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: buildbot <buildbot@kitware.com> Merge-request: !9157
| * | Tests: Exclude some tests on broken libc on Elbrusmakise-homura2024-01-161-5/+9
| | | | | | | | | | | | | | | These tests found to be occasionally failing, so just in case CMake is used in such environment, don't test this at all.
| * | LCC: Make CMake build without warnings on LCC 1.21makise-homura2024-01-161-1/+6
| | |
| * | libarchive: avoid lchmod not implemented warning on old LCCmakise-homura2024-01-161-1/+3
| | |
| * | LCC: Don't enable debugger on LCC that don't have <future>makise-homura2024-01-161-0/+1
| | |
| * | liblzma: Make cmliblzma buildable on LCC 1.21makise-homura2024-01-161-2/+2
| | |
| * | jsoncpp: fix missing template deletion support on LCC < 1.23makise-homura2024-01-161-0/+4
| | |
| * | cmcurl: fix X509_STORE_up_ref issue not just on LCC 1.23, but on LCC <= 1.23makise-homura2024-01-161-3/+3
| | |
| * | LCC: get rid of ambiguous assignments of {} for LCCmakise-homura2024-01-162-3/+3
| | | | | | | | | | | | | | | | | | LCC 1.21 can't determine the exact type of right side of assignment and so produces an error. Here it is resolved by an intermediate variable.
| * | cppdap: Use std::is_same<>::value instead of std::is_same<>() for old LCCmakise-homura2024-01-161-1/+1
| | | | | | | | | | | | | | | | | | std::is_same<>() is not const instead of std::is_same<>::value at least on LCC 1.21, so this produces an error. Still ()-notation seems to be fairly equivalent functionally to ::value-notation.
| * | LCC: Correct C compiler default C standardsmakise-homura2024-01-161-1/+1
| | |
* | | Merge topic 'FindX11-Xdbe'Brad King2024-01-171-0/+12
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | b118132c5f FindX11: Add awareness of Xdbe Acked-by: Kitware Robot <kwrobot@kitware.com> Reviewed-by: Ben Boeckel <ben.boeckel@kitware.com> Acked-by: Alex <leha-bot@yandex.ru> Merge-request: !9153
| * | | FindX11: Add awareness of XdbeKristian Spangsege2024-01-161-0/+12
| | | | | | | | | | | | | | | | Fixes: #25591
* | | | Merge branch 'release-3.28'Brad King2024-01-170-0/+0
|\ \ \ \
| * \ \ \ Merge topic 'cxxmodules-obj-lib' into release-3.28Brad King2024-01-175-1/+22
| |\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | c09f8d27cd cxxmodules: compute link information for C++ module-consuming targets Acked-by: Kitware Robot <kwrobot@kitware.com> Tested-by: buildbot <buildbot@kitware.com> Reviewed-by: Ben Boeckel <ben.boeckel@kitware.com> Merge-request: !9154
* | \ \ \ \ Merge topic 'cxxmodules-obj-lib'Brad King2024-01-175-1/+22
|\ \ \ \ \ \ | | |/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | c09f8d27cd cxxmodules: compute link information for C++ module-consuming targets Acked-by: Kitware Robot <kwrobot@kitware.com> Tested-by: buildbot <buildbot@kitware.com> Reviewed-by: Ben Boeckel <ben.boeckel@kitware.com> Merge-request: !9154
| * | | | | cxxmodules: compute link information for C++ module-consuming targetsMatheus Izvekov2024-01-145-1/+22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Compute link information for all C++ targets which support modules instead of just those which may provide modules, as they may import modules as well. This captures `OBJECT` libraries using modules which otherwise do not have link steps. Fixes: #25592
* | | | | | Merge branch 'release-3.28'Brad King2024-01-170-0/+0
|\ \ \ \ \ \ | | |/ / / / | |/| | | |
| * | | | | Merge topic 'matlab-win' into release-3.28Brad King2024-01-171-61/+53
| |\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 42fbe01eba FindMatlab: Accept long version in matlab_get_release_name_from_version c608adc236 FindMatlab: Fix major.minor version lookups in Windows Registry Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !9148
* | \ \ \ \ \ Merge topic 'matlab-win'Brad King2024-01-171-61/+53
|\ \ \ \ \ \ \ | | |/ / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 42fbe01eba FindMatlab: Accept long version in matlab_get_release_name_from_version c608adc236 FindMatlab: Fix major.minor version lookups in Windows Registry Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !9148
| * | | | | | FindMatlab: Accept long version in matlab_get_release_name_from_versionscivision2024-01-161-17/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Make this work: matlab_get_release_name_from_version(${Matlab_VERSION}) which the user would expect. While at it, simplify this function code.
| * | | | | | FindMatlab: Fix major.minor version lookups in Windows Registryscivision2024-01-161-45/+47
| |/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The registry key names use only the first two version components. Previously we were using the full versions. Fixes: #25582
* | | | | | CMake Nightly Date StampKitware Robot2024-01-171-1/+1
| | | | | |
* | | | | | Merge topic 'doc-CMP0156'Brad King2024-01-161-1/+1
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 64b682d3ba Help: Fix typo in CMP0156 documentation Acked-by: Kitware Robot <kwrobot@kitware.com> Reviewed-by: Raul Tambre <raul@tambre.ee> Merge-request: !9159
| * | | | | | Help: Fix typo in CMP0156 documentationBrad King2024-01-161-1/+1
|/ / / / / /
* | | | | | Merge topic 'ewilde/swift-escapism'Brad King2024-01-161-2/+5
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 9950a69f26 Swift/Ninja: Fix compile commands output file path Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: buildbot <buildbot@kitware.com> Merge-request: !9150
| * | | | | | Swift/Ninja: Fix compile commands output file pathEvan Wilde2024-01-121-2/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The exported compile commands did not use the proper file path separator for Windows in the output field for Swift files. This patch adds the appropriate filepath fix-ups to the output-field. Fixes: #25580
* | | | | | | Merge branch 'release-3.28'Brad King2024-01-160-0/+0
|\ \ \ \ \ \ \ | | |/ / / / / | |/| | | | |
| * | | | | | Merge topic 'FindMatlab-fix-version-without-versioninfoxml' into release-3.28Brad King2024-01-161-12/+13
| |\ \ \ \ \ \ | | |_|/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 694727bb45 FindMatlab: Restore support for versions without VersionInfo.xml Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !9151
* | | | | | | Merge topic 'FindMatlab-fix-version-without-versioninfoxml'Brad King2024-01-161-12/+13
|\ \ \ \ \ \ \ | | |/ / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 694727bb45 FindMatlab: Restore support for versions without VersionInfo.xml Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !9151
| * | | | | | FindMatlab: Restore support for versions without VersionInfo.xmlHermann von Kleist2024-01-121-12/+13
| |/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Report "unknown" in `_Matlab_VersionInfoXML` if file is missing. Fix omitted MATLAB root with unknown version from XML. Fixes: #25585 Fixes: #25586
* | | | | | Merge topic 'doc-policies'Brad King2024-01-16152-557/+510
|\ \ \ \ \ \ | |_|_|_|_|/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 83b9029208 Help: Clarify standard policy advice paragraph wording on how policies are set faa2bf22df Help: Drop current CMake version from standard policy advice paragraph ad3c2c0b03 Help: Normalize and consolidate standard policy advice paragraph Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: Mi-La <milan.kriz@eccam.com> Acked-by: Robert Maynard <robertjmaynard@gmail.com> Merge-request: !9149
| * | | | | Help: Clarify standard policy advice paragraph wording on how policies are setBrad King2024-01-111-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | Avoid implying that a policy must always be set explicitly.
| * | | | | Help: Drop current CMake version from standard policy advice paragraphBrad King2024-01-111-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Every policy documents whether CMake warns when it is not set. This behavior does not change with the CMake version, so avoid possibly confusing mention of the current CMake version.
| * | | | | Help: Normalize and consolidate standard policy advice paragraphBrad King2024-01-11152-557/+510
| | |/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Every policy's documentation has a paragraph on what version of CMake introduced it, how to set the policy, and whether CMake warns if the policy is not set. The wording of this paragraph has diverged across policies over time. Factor the paragraph out into a standard advice document included by every policy.
* | | | | CMake Nightly Date StampKitware Robot2024-01-161-1/+1
| |_|_|/ |/| | |
* | | | CMake Nightly Date StampKitware Robot2024-01-151-1/+1
| | | |
* | | | CMake Nightly Date StampKitware Robot2024-01-141-1/+1
| | | |
* | | | CMake Nightly Date StampKitware Robot2024-01-131-1/+1
| |_|/ |/| |
* | | Merge topic 'swift-sources-in-resp-file'Brad King2024-01-123-6/+14
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | b767917ebb Swift/Ninja: Include sources in response file Acked-by: Kitware Robot <kwrobot@kitware.com> Tested-by: buildbot <buildbot@kitware.com> Merge-request: !9143
| * | | Swift/Ninja: Include sources in response fileEvan Wilde2024-01-113-6/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Unlike C/C++, Swift compiles all sources in a module at once. This can quickly overwhelm the commandline length limit on Windows, so it is useful to place the source files in the response file. Issue: #25572