diff options
author | Brad King <brad.king@kitware.com> | 2024-07-02 12:07:09 (GMT) |
---|---|---|
committer | Brad King <brad.king@kitware.com> | 2024-07-02 12:07:09 (GMT) |
commit | 012bca3ccb1146db8b700eebc70b6a64911ab11f (patch) | |
tree | b731da050b49d5b732e5ed9a8b4e939333ebd073 | |
parent | 63b12b7702043090e6af733e0f2ebc4af6a81bf9 (diff) | |
parent | e4d0169107b6359946997b06bbc2cdd4f119bf16 (diff) | |
download | CMake-012bca3ccb1146db8b700eebc70b6a64911ab11f.zip CMake-012bca3ccb1146db8b700eebc70b6a64911ab11f.tar.gz CMake-012bca3ccb1146db8b700eebc70b6a64911ab11f.tar.bz2 |
Merge topic 'ctest-windows-std-handles' into release-3.28
e4d0169107 ctest: Fix spurious build failures with CTEST_USE_LAUNCHERS on Windows
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !9598
-rw-r--r-- | Source/cmSystemTools.cxx | 27 | ||||
-rw-r--r-- | Source/cmSystemTools.h | 6 | ||||
-rw-r--r-- | Source/ctest.cxx | 1 |
3 files changed, 0 insertions, 34 deletions
diff --git a/Source/cmSystemTools.cxx b/Source/cmSystemTools.cxx index 2bdc928..9945305 100644 --- a/Source/cmSystemTools.cxx +++ b/Source/cmSystemTools.cxx @@ -2375,33 +2375,6 @@ void cmSystemTools::EnsureStdPipes() } #endif -void cmSystemTools::DoNotInheritStdPipes() -{ -#ifdef _WIN32 - // Check to see if we are attached to a console - // if so, then do not stop the inherited pipes - // or stdout and stderr will not show up in dos - // shell windows - CONSOLE_SCREEN_BUFFER_INFO hOutInfo; - HANDLE hOut = GetStdHandle(STD_OUTPUT_HANDLE); - if (GetConsoleScreenBufferInfo(hOut, &hOutInfo)) { - return; - } - { - HANDLE out = GetStdHandle(STD_OUTPUT_HANDLE); - DuplicateHandle(GetCurrentProcess(), out, GetCurrentProcess(), &out, 0, - FALSE, DUPLICATE_SAME_ACCESS | DUPLICATE_CLOSE_SOURCE); - SetStdHandle(STD_OUTPUT_HANDLE, out); - } - { - HANDLE out = GetStdHandle(STD_ERROR_HANDLE); - DuplicateHandle(GetCurrentProcess(), out, GetCurrentProcess(), &out, 0, - FALSE, DUPLICATE_SAME_ACCESS | DUPLICATE_CLOSE_SOURCE); - SetStdHandle(STD_ERROR_HANDLE, out); - } -#endif -} - #ifdef _WIN32 # ifndef CRYPT_SILENT # define CRYPT_SILENT 0x40 /* Not defined by VS 6 version of header. */ diff --git a/Source/cmSystemTools.h b/Source/cmSystemTools.h index 9563fd6..0f96500 100644 --- a/Source/cmSystemTools.h +++ b/Source/cmSystemTools.h @@ -494,12 +494,6 @@ public: const std::vector<std::string>& files, cmTarExtractTimestamps extractTimestamps, bool verbose); - // This should be called first thing in main - // it will keep child processes from inheriting the - // stdin and stdout of this process. This is important - // if you want to be able to kill child processes and - // not get stuck waiting for all the output on the pipes. - static void DoNotInheritStdPipes(); static void EnsureStdPipes(); diff --git a/Source/ctest.cxx b/Source/ctest.cxx index fa38a65..55defe7 100644 --- a/Source/ctest.cxx +++ b/Source/ctest.cxx @@ -173,7 +173,6 @@ int main(int argc, char const* const* argv) argc = encoding_args.argc(); argv = encoding_args.argv(); - cmSystemTools::DoNotInheritStdPipes(); cmSystemTools::InitializeLibUV(); cmSystemTools::FindCMakeResources(argv[0]); |