summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | | Merge topic 'ctest-windows-std-handles' into release-3.30Brad King2024-06-143-34/+0
| |\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | e4d0169107 ctest: Fix spurious build failures with CTEST_USE_LAUNCHERS on Windows Acked-by: Kitware Robot <kwrobot@kitware.com> Tested-by: buildbot <buildbot@kitware.com> Merge-request: !9598
* | \ \ \ \ Merge topic 'ctest-windows-std-handles'Brad King2024-06-143-34/+0
|\ \ \ \ \ \ | | |/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | e4d0169107 ctest: Fix spurious build failures with CTEST_USE_LAUNCHERS on Windows Acked-by: Kitware Robot <kwrobot@kitware.com> Tested-by: buildbot <buildbot@kitware.com> Merge-request: !9598
| * | | | | ctest: Fix spurious build failures with CTEST_USE_LAUNCHERS on WindowsBrad King2024-06-143-34/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Remove the stdio handle inheritance suppression originally added by commit f262298bb0 (... do not inherit pipes in child procs for ctest so it can kill them, 2007-09-11, v2.6.0~1136). It's not clear what problem it was trying to solve, was only done in `ctest` and not `cmake`, and since commit 9c3ffe2474 (BUG: fix problem with stdout and stderr not showing up in ms dos shells, 2007-09-25, v2.6.0~1066) has not been done in `ctest` launched under interactive consoles. Furthermore, the code has been spuriously breaking stdio when `ctest` is started with both stdout and stderr connected to the same pipe, such as when `ctest --launch` is used under `ninja`. This is because it used `DuplicateHandle` with `DUPLICATE_CLOSE_SOURCE` on the stdout handle and then the stderr handle. If the handles are the same, then the stderr handle becomes invalid in between these operations, leading to likely-undefined behavior. Since commit 96b3dd329e (cmCTestLaunchReporter: Replace cmsysProcess with cmUVProcessChain, 2023-07-26, v3.28.0-rc1~138^2~2) this became more noticeable because `uv_spawn` performs additional verification on stdio handles. This could be fixed by instead suppressing inheritance via SetHandleInformation(h, HANDLE_FLAG_INHERIT, 0); However, the functionality no longer seems necessary, so remove it.
* | | | | | Merge branch 'release-3.30'Brad King2024-06-140-0/+0
|\ \ \ \ \ \ | | |/ / / / | |/| | | |
| * | | | | Merge branch 'release-3.29' into release-3.30Brad King2024-06-140-0/+0
| |\ \ \ \ \ | | | |_|/ / | | |/| | |
* | | | | | Merge branch 'release-3.29'Brad King2024-06-140-0/+0
|\ \ \ \ \ \ | | |/ / / / | |/| | | |
| * | | | | Merge topic 'vs-quote-semicolons' into release-3.29Brad King2024-06-141-6/+4
| |\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 9b672e8075 VS: Quote custom command arguments containing semicolons Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !9597
* | \ \ \ \ \ Merge branch 'release-3.30'Brad King2024-06-140-0/+0
|\ \ \ \ \ \ \ | | |_|/ / / / | |/| | | | |
| * | | | | | Merge topic 'vs-quote-semicolons' into release-3.30Brad King2024-06-141-6/+4
| |\ \ \ \ \ \ | | | |/ / / / | | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 9b672e8075 VS: Quote custom command arguments containing semicolons Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !9597
* | | | | | | Merge topic 'vs-quote-semicolons'Brad King2024-06-141-6/+4
|\ \ \ \ \ \ \ | | |/ / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 9b672e8075 VS: Quote custom command arguments containing semicolons Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !9597
| * | | | | | VS: Quote custom command arguments containing semicolonsBrad King2024-06-141-6/+4
| | |/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The `GoogleTest` module needs this to pass the `TEST_EXECUTOR` definition to its `GoogleTestAddTests.cmake` helper script in the `POST_BUILD` command since commit f875c479f5 (GoogleTest: Honor TEST_LAUNCHER in gtest_discover_tests, 2024-01-17, v3.29.0-rc1~34^2). Previously it worked only if other characters, such as spaces, caused the argument to be quoted. This was exposed by running the `RunCMake.GoogleTest` test in a path without spaces. Reported-by: Garrett Campbell <gcampbell@microsoft.com>
* | | | | | Merge branch 'release-3.30'Brad King2024-06-140-0/+0
|\ \ \ \ \ \ | | |/ / / / | |/| | | |
| * | | | | Merge topic 'xcode-16' into release-3.30Brad King2024-06-142-2/+3
| |\ \ \ \ \ | | |_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | b243c19986 Tests: Update expected results for Xcode 16 Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !9596
* | | | | | Merge topic 'xcode-16'Brad King2024-06-142-2/+3
|\ \ \ \ \ \ | | |/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | b243c19986 Tests: Update expected results for Xcode 16 Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !9596
| * | | | | Tests: Update expected results for Xcode 16Brad King2024-06-132-2/+3
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In `Tests/RunCMake/LinkerSelection`, Xcode 16 warns when building the AppleClassic case: ld: warning: -ld_classic is deprecated and will be removed in a future release Tolerate all build warnings.
* | | | | CMake Nightly Date StampKitware Robot2024-06-141-1/+1
| | | | |
* | | | | Merge topic 'sunpro'Brad King2024-06-132-0/+8
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 8a33068afe visibility: add support for Oracle Developer Studio 12.6 Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !9594
| * | | | | visibility: add support for Oracle Developer Studio 12.6Carlo Marcelo Arenas Belón2024-06-122-0/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | While an `-fvisibility` flag was added in the previous version, it throws warnings indicating it would be ignored unless given to the linker and fails to work properly. Tested on Solaris 11.3 SPARC and Solaris 11.4 x86/SPARC.
* | | | | | CMake Nightly Date StampKitware Robot2024-06-131-1/+1
| | | | | |
* | | | | | Merge branch 'release-3.30'Brad King2024-06-120-0/+0
|\ \ \ \ \ \ | |/ / / / / |/| / / / / | |/ / / /
| * | | | CMake 3.30.0-rc2v3.30.0-rc2Brad King2024-06-121-1/+1
| | | | |
* | | | | CMake Nightly Date StampKitware Robot2024-06-121-1/+1
| |_|_|/ |/| | |
* | | | Merge branch 'release-3.30'Brad King2024-06-110-0/+0
|\ \ \ \ | |/ / /
| * | | Merge topic 'rel-win-wix-4' into release-3.30Brad King2024-06-1116-107/+108
| |\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | f6c2f64d04 Utilities/Release: Update MSI generation to use WIX 4 a8cecc3b7f ci: Update CMake version to 3.30.0-rc1 Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !9591
* | \ \ \ Merge topic 'rel-win-wix-4'Brad King2024-06-1116-107/+108
|\ \ \ \ \ | | |/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | f6c2f64d04 Utilities/Release: Update MSI generation to use WIX 4 a8cecc3b7f ci: Update CMake version to 3.30.0-rc1 Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !9591
| * | | | Utilities/Release: Update MSI generation to use WIX 4Brad King2024-06-1014-100/+101
| | | | |
| * | | | ci: Update CMake version to 3.30.0-rc1Brad King2024-06-062-7/+7
| | | | |
* | | | | Merge branch 'release-3.30'Brad King2024-06-110-0/+0
|\ \ \ \ \ | | |/ / / | |/| | |
| * | | | Merge branch 'release-3.29' into release-3.30Brad King2024-06-110-0/+0
| |\ \ \ \ | | | |/ / | | |/| |
| | * | | Merge branch 'release-3.28' into release-3.29Brad King2024-06-110-0/+0
| | |\ \ \ | | | | |/ | | | |/|
| | | * | Merge topic 'clang-ansi-color' into release-3.28Brad King2024-06-111-1/+1
| | | |\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 1004cc2b52 Clang: Drop non-existent -fno-ansi-escape-codes flag Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !9592
| | * | \ \ Merge topic 'clang-ansi-color' into release-3.29Brad King2024-06-111-1/+1
| | |\ \ \ \ | | | | |/ / | | | |/| | | | | | | | | | | | | | | | | | | | | | | | | | 1004cc2b52 Clang: Drop non-existent -fno-ansi-escape-codes flag Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !9592
* | | | | | Merge branch 'release-3.30'Brad King2024-06-110-0/+0
|\ \ \ \ \ \ | |/ / / / /
| * | | | | Merge topic 'clang-ansi-color' into release-3.30Brad King2024-06-111-1/+1
| |\ \ \ \ \ | | | |/ / / | | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | 1004cc2b52 Clang: Drop non-existent -fno-ansi-escape-codes flag Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !9592
* | | | | | Merge topic 'clang-ansi-color'Brad King2024-06-111-1/+1
|\ \ \ \ \ \ | | |/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | 1004cc2b52 Clang: Drop non-existent -fno-ansi-escape-codes flag Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !9592
| * | | | | Clang: Drop non-existent -fno-ansi-escape-codes flagBrad King2024-06-101-1/+1
| | |_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In commit 74b5fae52d (Clang: Use -fno-ansi-escape-codes for color diagnostics on Windows, 2023-10-19, v3.28.0-rc3~3^2) we added the positive option `-fansi-escape-codes` and the negative form of the option, `-fno-ansi-escape-codes`. According to the LLVM/Clang documentation: * https://releases.llvm.org/18.1.4/tools/clang/docs/UsersManual.html#cmdoption-fansi-escape-codes only the positive form of the option exists. Fixes: #26036
* | | | | CMake Nightly Date StampKitware Robot2024-06-111-1/+1
| | | | |
* | | | | Merge branch 'release-3.30'Brad King2024-06-100-0/+0
|\ \ \ \ \ | | |/ / / | |/| | |
| * | | | Merge topic 'bash-completion-ctest-cpack' into release-3.30Brad King2024-06-102-2/+6
| |\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 3889640e8b Auxiliary: Modernize bash completion for ctest and cpack Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !9590
* | \ \ \ \ Merge topic 'bash-completion-ctest-cpack'Brad King2024-06-102-2/+6
|\ \ \ \ \ \ | | |/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | 3889640e8b Auxiliary: Modernize bash completion for ctest and cpack Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !9590
| * | | | | Auxiliary: Modernize bash completion for ctest and cpackCraig Scott2024-06-072-2/+6
| | |_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | This applies a similar modernization as was done in 1ff41ba26e (Auxiliary: bash-completion: use _comp_initialize, 2024-06-02) for the cmake executable. The _init_completion function was deprecated upstream in bash-completion 2.12.
* | | | | Merge branch 'release-3.30'Brad King2024-06-100-0/+0
|\ \ \ \ \ | | |/ / / | |/| | |
| * | | | Merge topic 'fc-cmp0168-fresh' into release-3.30Brad King2024-06-107-23/+52
| |\ \ \ \ | | |/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 9299cbbdb4 FetchContent: Force cmake --fresh to re-execute direct population steps e82e2c38c1 Tests: RunCMake.FetchContent should not always force _deps removal f97b25ec4b Tests: Fix -direct variants of FetchContent tests using wrong files 11b684c449 FetchContent: Fix typos in stamp/step file names a02eec4a9f FetchContent,ExternalProject: Fix extra semicolons in step commands Acked-by: Kitware Robot <kwrobot@kitware.com> Tested-by: buildbot <buildbot@kitware.com> Merge-request: !9589
* | | | | Merge topic 'fc-cmp0168-fresh'Brad King2024-06-107-23/+52
|\ \ \ \ \ | | |/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 9299cbbdb4 FetchContent: Force cmake --fresh to re-execute direct population steps e82e2c38c1 Tests: RunCMake.FetchContent should not always force _deps removal f97b25ec4b Tests: Fix -direct variants of FetchContent tests using wrong files 11b684c449 FetchContent: Fix typos in stamp/step file names a02eec4a9f FetchContent,ExternalProject: Fix extra semicolons in step commands Acked-by: Kitware Robot <kwrobot@kitware.com> Tested-by: buildbot <buildbot@kitware.com> Merge-request: !9589
| * | | | FetchContent: Force cmake --fresh to re-execute direct population stepsCraig Scott2024-06-075-9/+43
| | | | | | | | | | | | | | | | | | | | Issue: #25468
| * | | | Tests: RunCMake.FetchContent should not always force _deps removalCraig Scott2024-06-071-3/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | To properly test some functionality, tests may rely on not clearing things like time stamps between cmake invocations. The RunCMake infrastructure clears the build directory by default anyway, and tests may individually ask for that to be disabled where needed. The line being removed here was originally added to assist with manually re-running individual tests locally outside the control of RunCMake. That is no longer appropriate.
| * | | | Tests: Fix -direct variants of FetchContent tests using wrong filesCraig Scott2024-06-072-8/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The -direct variants of the RunCMake.FetchContent tests were meant to be using the same result, stdout and stderr files as the non-direct tests. The -direct tests were specified in the wrong way for that and ended up using no files at all, so they weren't testing the full set of expected conditions. Use the test variant feature provided by the RunCMake infrastructure instead, which is the proper way to handle this sort of scenario.
| * | | | FetchContent: Fix typos in stamp/step file namesCraig Scott2024-06-071-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | The exact file names don't technically matter, but the wrong names are confusing and misleading.
| * | | | FetchContent,ExternalProject: Fix extra semicolons in step commandsCraig Scott2024-06-071-1/+1
| |/ / / | | | | | | | | | | | | | | | | The extra semicolons weren't ultimately causing any errors when running the step commands, but they shouldn't be there.
* | | | CMake Nightly Date StampKitware Robot2024-06-101-1/+1
| | | |