summaryrefslogtreecommitdiffstats
path: root/Source
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-176-5/+59
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-161-0/+26
| | | | | | | | | | | | | | | | | | | | | | | | 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-173-4/+9
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
| * | LCC: Make CMake build without warnings on LCC 1.21makise-homura2024-01-161-1/+6
| | |
| * | 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.
* | | Merge topic 'cxxmodules-obj-lib'Brad King2024-01-171-1/+2
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-141-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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 topic 'vs-scan-for-modules' into release-3.28Brad King2024-01-101-5/+6
| |\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 405dc7d19c Tests/CXXModules/scan_properties: use `ixx` extension 029ddc3410 cmVisualStudio10TargetGenerator: always specify scanning 6c9614cbf4 Tests/CXXModules: add a test case for VS generation without flags 34f4423851 cmVisualStudio10TargetGenerator: fix typo in flag name Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !9104
| * \ \ \ Merge topic 'cxxmodules-no-unity' into release-3.28Brad King2024-01-103-1/+39
| |\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 63bbb3768d cmLocalGenerator: ignore scanned sources for unity builds 76b5383123 cmGlobalGenerator: add unity sources after computing target compile features 7fc2a83fe6 Tests/CXXModules: add a test with unity build support Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: buildbot <buildbot@kitware.com> Merge-request: !9118
* | | | | | CMake Nightly Date StampKitware Robot2024-01-171-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
* | | | | | 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-122-2/+12
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-112-2/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
* | | | | | CMake Nightly Date StampKitware Robot2024-01-121-1/+1
| |/ / / / |/| | | |
* | | | | Merge topic 'file-strings-regex-match'Brad King2024-01-112-1/+41
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | fa00928bcd file: `STRINGS` + `REGEX` store match results ff0085cf74 Help: Drop incorrect versionadded mark from CMAKE_MATCH_<n> Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: buildbot <buildbot@kitware.com> Merge-request: !9124
| * | | | | file: `STRINGS` + `REGEX` store match resultsCristian Le2024-01-092-1/+41
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Cristian Le <cristian.le@mpsd.mpg.de>
* | | | | | CMake Nightly Date StampKitware Robot2024-01-111-1/+1
| | | | | |
* | | | | | Merge topic 'static'Brad King2024-01-101-8/+12
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 9e829779f2 Swift: preserve `-static` for static library swiftmodules Acked-by: Kitware Robot <kwrobot@kitware.com> Tested-by: buildbot <buildbot@kitware.com> Merge-request: !9141
| * | | | | | Swift: preserve `-static` for static library swiftmodulesSaleem Abdulrasool2024-01-091-8/+12
| | |/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The `-static` is important for the emission of the module as it is serialized into the swiftmodule which then is used by the Swift frontend to decide how to link to the symbol (via the GOT or not, or the IAT on Windows). This repairs building static libraries with Swift on Windows.
* | | | | | Merge topic 'orkun_refactor_09_01_2024'Brad King2024-01-101-22/+20
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 63d4e4ec28 Makefile: Reduce string copies 4d928592eb Makefile: Remove redundant push_back 3e533bd64f Makefile: Improve const correctness c2dd5dfbe8 Makefile: Remove no-op call Acked-by: Kitware Robot <kwrobot@kitware.com> Tested-by: buildbot <buildbot@kitware.com> Merge-request: !9142
| * | | | | | Makefile: Reduce string copiesOrkun Tokdemir2024-01-091-3/+3
| | | | | | |
| * | | | | | Makefile: Remove redundant push_backOrkun Tokdemir2024-01-091-2/+1
| | | | | | |
| * | | | | | Makefile: Improve const correctnessOrkun Tokdemir2024-01-091-15/+16
| | | | | | |
| * | | | | | Makefile: Remove no-op callOrkun Tokdemir2024-01-091-2/+0
| | | | | | |
* | | | | | | Merge topic 'vs-scan-for-modules'Brad King2024-01-101-5/+6
|\ \ \ \ \ \ \ | | |_|_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 405dc7d19c Tests/CXXModules/scan_properties: use `ixx` extension 029ddc3410 cmVisualStudio10TargetGenerator: always specify scanning 6c9614cbf4 Tests/CXXModules: add a test case for VS generation without flags 34f4423851 cmVisualStudio10TargetGenerator: fix typo in flag name Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !9104
| * | | | | | cmVisualStudio10TargetGenerator: always specify scanningBen Boeckel2024-01-061-3/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Set that sources should not be scanned on a target-wide basis and then enable on a per-TU basis as needed. Fixes: #25519
| * | | | | | cmVisualStudio10TargetGenerator: fix typo in flag nameBen Boeckel2024-01-061-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It is case insensitive, so this only matches the official flag name and still works.
* | | | | | | Merge topic 'cxxmodules-no-unity'Brad King2024-01-103-1/+39
|\ \ \ \ \ \ \ | | |_|_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 63bbb3768d cmLocalGenerator: ignore scanned sources for unity builds 76b5383123 cmGlobalGenerator: add unity sources after computing target compile features 7fc2a83fe6 Tests/CXXModules: add a test with unity build support Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: buildbot <buildbot@kitware.com> Merge-request: !9118
| * | | | | | cmLocalGenerator: ignore scanned sources for unity buildsBen Boeckel2024-01-051-0/+9
| | | | | | |
| * | | | | | cmGlobalGenerator: add unity sources after computing target compile featuresBen Boeckel2024-01-052-1/+30
| |/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We need to know which sources will be scanned for C++ module dependencies in order to exclude them from unity builds. The addition of unity sources will not change the set of features.
* | | | | | Merge topic 'makefile-depfile'Brad King2024-01-101-4/+3
|\ \ \ \ \ \ | | |_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 7198f0d149 Makefile: Fix double escaping when DEPFILE is used 5162ff64d4 Makefile: Reduce string copies Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: buildbot <buildbot@kitware.com> Merge-request: !9138
| * | | | | Makefile: Fix double escaping when DEPFILE is usedOrkun Tokdemir2024-01-091-3/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In commit cfd8a5ac1f (Makefiles: Add support of DEPFILE for add_custom_command, 2020-12-04, v3.20.0-rc1~237^2~1) we added a `ConvertToOutputPath` call on a path given to the `depends` field of `WriteMakeRule`. The latter already handles escaping for Makefile syntax. Fixes: #25554
| * | | | | Makefile: Reduce string copiesOrkun Tokdemir2024-01-081-1/+1
| | | | | |
* | | | | | CMake Nightly Date StampKitware Robot2024-01-101-1/+1
| |_|_|/ / |/| | | |
* | | | | Merge topic 'ctest-cli-http-headers'Brad King2024-01-093-2/+28
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 20adf8cfce ctest: allow HTTP headers via command line Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: buildbot <buildbot@kitware.com> Merge-request: !9123
| * | | | | ctest: allow HTTP headers via command lineMatthew Woehlke2024-01-083-2/+28
| | | | | | | | | | | | | | | | | | | | | | | | Add ability to specify HTTP headers via `ctest -T Submit`.
* | | | | | Merge topic 'ewilde/swift-ninja-response-files'Brad King2024-01-092-3/+7
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | a6a5c43300 Swift/Ninja: Add support for response files Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: buildbot <buildbot@kitware.com> Merge-request: !9098
| * | | | | | Swift/Ninja: Add support for response filesEvan Wilde2024-01-082-3/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Adding support for `CMAKE_NINJA_FORCE_RESPONSE_FILE` with Swift. Issue: #25490 Fixes: #25563