summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
| * | KWSys: ConsoleBuf: Fix test case when running under Windows TerminalBrad King2024-05-281-1/+25
| |/ | | | | | | | | | | | | Our test case writes a NUL byte to the console to test its behavior. The behavior of Windows Terminal differs from Windows Console Host (conhost.exe). Detect which of these is in use at runtime and adjust our expected result accordingly.
* | Merge branch 'release-3.28' into release-3.29Brad King2024-05-280-0/+0
|\ \ | |/
| * Merge topic 'libuv-win-process-no-extra-stdio' into release-3.28Brad King2024-05-281-0/+7
| |\ | | | | | | | | | | | | | | | | | | a590382850 libuv: win/spawn: disable extra-file-descriptor support not needed by CMake Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !9541
* | \ Merge topic 'execute_process-child-startup-info' into release-3.29Brad King2024-05-285-0/+41
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | fa8c04b421 Tests/RunCMake/execute_process: Check STARTUPINFOW reserved members d98df689ab Merge branch 'libuv-win-process-no-extra-stdio' a590382850 libuv: win/spawn: disable extra-file-descriptor support not needed by CMake Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !9541
| * | | Tests/RunCMake/execute_process: Check STARTUPINFOW reserved membersBrad King2024-05-244-0/+34
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Verify that `execute_process` launches processes on Windows such that `GetStartupInfoW` in the child does not populate `STARTUPINFOW` members reserved for the MSVC C run-time. Issue: #25996
| * | | Merge branch 'libuv-win-process-no-extra-stdio'Brad King2024-05-221-0/+7
| |\ \ \ | | | |/ | | |/|
| | * | libuv: win/spawn: disable extra-file-descriptor support not needed by CMakeBrad King2024-05-221-0/+7
| | |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Upstream libuv supports passing file descriptors >= 3 to child processes via `STARTUPINFOW` members reserved by the MSVC C run-time. However, some programs use `GetStartupInfoW` to initialize a `STARTUPINFOW` structure to pass to `CreateProcessW` without clearing the reserved members. If we launch such programs with non-zero values in the reserved members, the MSVC C run-time in *their* children may not correctly associate the stdin/stdout/stderr streams' file descriptors with the corresponding `HANDLE`s. Patch our copy of libuv to avoid using the reserved members. This restores `execute_process` support for the above-described programs as we had prior to commit 5420639a8d (cmExecuteProcessCommand: Replace cmsysProcess with cmUVProcessChain, 2023-06-01, v3.28.0-rc1~138^2~8). It also enables support for such programs when launched by `ctest`. Fixes: #25996 Fixes: #25889
* | | Merge topic 'cuda-vs-17.10' into release-3.29Brad King2024-05-241-1/+1
|\ \ \ | |/ / |/| | | | | | | | | | | | | | 69f26d8156 VS: Fix compiler identification of nvcc with unsupported host compiler Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !9546
| * | VS: Fix compiler identification of nvcc with unsupported host compilerBrad King2024-05-231-1/+1
|/ / | | | | | | | | | | | | | | | | | | | | | | | | `nvcc` from CUDA < 12.4 does support the MSVC 14.40.17.10 toolset. Users may specify `CUDAFLAGS=-allow-unsupported-compiler` to bypass the check. However, we do not use arbitrary user-specified flags during compiler identification in the VS generator because escaping them for the `AdditionalOptions` of the `.vcxproj` file requires non-trivial logic that we currently only implement in the C++ generator code. Instead, just always pass `-allow-unsupported-compiler` during CUDA compiler identification in the VS generator. Fixes: #26003
* | Merge branch 'release-3.28' into release-3.29Brad King2024-05-170-0/+0
|\ \ | |/
| * Merge topic 'cxxmodules-gcc-binary-mode' into release-3.28Brad King2024-05-173-1/+29
| |\ | | | | | | | | | | | | | | | | | | 9e2f31ec23 cmCxxModuleMapper: add a query for the open mode for the modmap Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !9525
* | \ Merge topic 'cxxmodules-gcc-binary-mode' into release-3.29Brad King2024-05-173-1/+29
|\ \ \ | | |/ | |/| | | | | | | | | | | | | 9e2f31ec23 cmCxxModuleMapper: add a query for the open mode for the modmap Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !9525
| * | cmCxxModuleMapper: add a query for the open mode for the modmapBen Boeckel2024-05-163-1/+29
| | | | | | | | | | | | | | | | | | GCC (MinGW) wants to use `\n` on Windows too. Fixes: #25974
* | | Merge branch 'release-3.28' into release-3.29Brad King2024-05-150-0/+0
|\ \ \ | | |/ | |/|
| * | Merge topic 'cmp0037-message' into release-3.28Brad King2024-05-1510-3/+45
| |\ \ | | | | | | | | | | | | | | | | | | | | | | | | c773d5b436 CMP0037: Restore diagnostic message for invalid ALIAS target names Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !9519
* | \ \ Merge topic 'cmp0037-message' into release-3.29Brad King2024-05-1510-3/+45
|\ \ \ \ | | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | c773d5b436 CMP0037: Restore diagnostic message for invalid ALIAS target names Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: buildbot <buildbot@kitware.com> Reviewed-by: Ben Boeckel <ben.boeckel@kitware.com> Merge-request: !9519
| * | | CMP0037: Restore diagnostic message for invalid ALIAS target namesBrad King2024-05-1410-3/+45
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | Refactoring in commit 7a4c02cb38 (cmGlobalGenerator: factor out messaging for CMP0037, 2023-09-24, v3.28.0-rc1~39^2~7) incorrectly switched to reporting the aliased target name instead of the invalid name of the alias itself. Fixes: #25979
* | | Merge branch 'release-3.28' into release-3.29Brad King2024-05-140-0/+0
|\ \ \ | |/ /
| * | Merge topic 'ExternalProject-xcode-effective-platform' into release-3.28Brad King2024-05-141-1/+10
| |\ \ | | |/ | |/| | | | | | | | | | | | | 0a3caf0878 ExternalProject: Restore support for Xcode with an effective platform Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !9517
* | | Merge topic 'ExternalProject-xcode-effective-platform' into release-3.29Brad King2024-05-141-1/+10
|\ \ \ | | |/ | |/| | | | | | | | | | | | | 0a3caf0878 ExternalProject: Restore support for Xcode with an effective platform Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !9517
| * | ExternalProject: Restore support for Xcode with an effective platformBrad King2024-05-131-1/+10
| |/ | | | | | | | | | | | | | | | | | | | | | | Revert commit cabad8a37f (ExternalProject: Always use $<CONFIG> for source files, 2023-02-02, v3.27.0-rc1~550^2~3) and restore Xcode-specific behavior intentionally preserved by commit c111d440ce (ExternalProject: Express per-config step stamp file paths using CONFIG genex, 2022-06-08, v3.24.0-rc1~15^2). Unfortunately we still do not have a test case, so leave a comment to avoid reverting this. Issue: #23645 Issue: #23652
* | CMake 3.29.3v3.29.3Brad King2024-05-072-4/+4
| |
* | Merge branch 'release-3.28' into release-3.29Brad King2024-05-071-0/+6
|\ \ | |/
| * CMake 3.28.5v3.28.5Brad King2024-05-072-1/+7
| |
* | Merge branch 'release-3.28' into release-3.29Brad King2024-05-070-0/+0
|\ \ | |/
| * Merge topic 'cxx-inspect-without-scanning' into release-3.28Brad King2024-05-072-0/+16
| |\ | | | | | | | | | | | | | | | | | | 6df925b595 cxxmodules: Do not scan sources while testing the compiler Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !9494
* | \ Merge topic 'cxx-inspect-without-scanning' into release-3.29Brad King2024-05-072-0/+16
|\ \ \ | | |/ | |/| | | | | | | | | | | | | | | | 6df925b595 cxxmodules: Do not scan sources while testing the compiler Acked-by: Kitware Robot <kwrobot@kitware.com> Reviewed-by: Ben Boeckel <ben.boeckel@kitware.com> Merge-request: !9494
| * | cxxmodules: Do not scan sources while testing the compilerBrad King2024-05-062-0/+16
| |/ | | | | | | | | | | | | | | | | The C++ sources we use to test the compiler do not use modules. Avoid requiring a compiler that can scan just to enable the language, even when CMP0155 is NEW. The project may explicitly turn off `CMAKE_CXX_SCAN_FOR_MODULES` before adding any targets. Fixes: #25956
* | Merge branch 'release-3.28' into release-3.29Brad King2024-05-030-0/+0
|\ \ | |/
| * Merge topic 'cxxmodules-build-export-different-directory' into release-3.28Brad King2024-05-0311-2/+153
| |\ | | | | | | | | | | | | | | | | | | ec35251510 cmDyndepCollation: look at all `export(TARGETS)` calls Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !9485
* | \ Merge topic 'cxxmodules-build-export-different-directory' into release-3.29Brad King2024-05-0311-2/+153
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 1fba6d3233 Merge branch 'cxxmodules-build-export-different-directory-3.28' into cxxmodules-build-export-different-directory ec35251510 cmDyndepCollation: look at all `export(TARGETS)` calls Acked-by: Kitware Robot <kwrobot@kitware.com> Tested-by: buildbot <buildbot@kitware.com> Merge-request: !9485
| * \ \ Merge branch 'cxxmodules-build-export-different-directory-3.28' into ↵Ben Boeckel2024-05-0211-2/+153
| |\ \ \ | | | |/ | | |/| | | | | | | | | | | | | | | | | cxxmodules-build-export-different-directory * cxxmodules-build-export-different-directory-3.28: cmDyndepCollation: look at all `export(TARGETS)` calls
| | * | cmDyndepCollation: look at all `export(TARGETS)` callsBen Boeckel2024-05-0211-2/+153
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously, only `export()` calls in the same directory were noticed. Also add a test that exports in a different directory than the target itself resides in. Fixes: #25813
* | | | Merge branch 'release-3.28' into release-3.29Brad King2024-05-020-0/+0
|\ \ \ \ | | |_|/ | |/| |
| * | | Merge topic 'vs-scan-module-deps-settings' into release-3.28Brad King2024-05-024-14/+52
| |\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 3022f0363f VS: set ScanSourceForModuleDependencies at vcxproj level dff511ad28 cmGeneratorTarget: add a target-level query for "needs dyndep" Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !9471
* | \ \ \ Merge topic 'vs-scan-module-deps-settings' into release-3.29Brad King2024-05-024-14/+52
|\ \ \ \ \ | | |/ / / | |/| / / | |_|/ / |/| | | | | | | | | | | | | | | | | | | 3022f0363f VS: set ScanSourceForModuleDependencies at vcxproj level dff511ad28 cmGeneratorTarget: add a target-level query for "needs dyndep" Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: AaronRuizMoraUK <aaronruizmora@gmail.com> Merge-request: !9471
| * | | VS: set ScanSourceForModuleDependencies at vcxproj levelBen Boeckel2024-05-012-5/+21
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Implement the target-wide `CXX_SCAN_FOR_MODULES`/`CMP0155` selection with the `.vcxproj`-wide `ScanSourceForModuleDependencies` setting. Set the per-source equivalent only when needed for a per-source `CXX_SCAN_FOR_MODULES` property. This approach enables Intellisense for interfaces imported from modules. It is also more consistent with what a user might expect when investigating the state of module scanning from the VS property panels. Fixes: #25806 Fixes: #25947
| * | | cmGeneratorTarget: add a target-level query for "needs dyndep"Ben Boeckel2024-05-012-9/+31
| |/ / | | | | | | | | | | | | This can still be overridden per-source, but it indicates the state of the target's default behavior.
* | | Merge branch 'release-3.28' into release-3.29Brad King2024-05-010-0/+0
|\ \ \ | |/ /
| * | Merge topic 'export-name-safe-name' into release-3.28Brad King2024-05-0112-6/+321
| |\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | 142a85f9c1 cxxmodules: use filesystem-safe export names in filenames 4452d41488 cmGeneratorTarget: add method to get a filesystem-safe export name Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !9474
* | \ \ Merge topic 'export-name-safe-name' into release-3.29Brad King2024-05-0112-6/+321
|\ \ \ \ | | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | 142a85f9c1 cxxmodules: use filesystem-safe export names in filenames 4452d41488 cmGeneratorTarget: add method to get a filesystem-safe export name Acked-by: Kitware Robot <kwrobot@kitware.com> Tested-by: buildbot <buildbot@kitware.com> Merge-request: !9474
| * | | cxxmodules: use filesystem-safe export names in filenamesBen Boeckel2024-04-3010-6/+310
| | | | | | | | | | | | | | | | | | | | | | | | Also add tests. Fixes: #25828
| * | | cmGeneratorTarget: add method to get a filesystem-safe export nameBen Boeckel2024-04-292-0/+11
| | |/ | |/| | | | | | | | | | | | | | | | | | | C++ module support puts the export name in a filename. Export names with `:` in them are not valid filenames on Windows. Add a method to escape names as necessary. See: #25828
* | | Merge topic 'FindHDF5-version-4-components' into release-3.29Brad King2024-05-011-1/+1
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | 928943f91b FindHDF5: Fix detection of versions with more than three components Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !9476
| * | | FindHDF5: Fix detection of versions with more than three componentsمهدي شينون (Mehdi Chinoune)2024-04-301-1/+1
|/ / / | | | | | | | | | | | | | | | Fix detecting the version of HDF5 1.14.4.2. Fixes: #25945
* | | Merge branch 'release-3.28' into release-3.29Brad King2024-04-300-0/+0
|\ \ \ | | |/ | |/|
| * | Merge topic 'exported-modules-with-headers' into release-3.28Brad King2024-04-3011-0/+188
| |\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | 2041f7c9bf cmGeneratorTarget: add the original target as a COMPILE_ONLY link 051c2110c8 Tests/CXXModules: test exporting modules which include headers Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !9469
* | \ \ Merge topic 'exported-modules-with-headers' into release-3.29Brad King2024-04-3011-0/+188
|\ \ \ \ | | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | 2041f7c9bf cmGeneratorTarget: add the original target as a COMPILE_ONLY link 051c2110c8 Tests/CXXModules: test exporting modules which include headers Acked-by: Kitware Robot <kwrobot@kitware.com> Tested-by: buildbot <buildbot@kitware.com> Merge-request: !9469
| * | | cmGeneratorTarget: add the original target as a COMPILE_ONLY linkBen Boeckel2024-04-271-0/+4
| | | | | | | | | | | | | | | | Fixes: #25909
| * | | Tests/CXXModules: test exporting modules which include headersBen Boeckel2024-04-2710-0/+184
| | |/ | |/| | | | | | | | | | Test that headers that are part of the same target are available to modules in the target itself.