summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorBrad King <brad.king@kitware.com>2024-06-24 13:36:19 (GMT)
committerBrad King <brad.king@kitware.com>2024-06-24 13:36:19 (GMT)
commit1c4c5ad545ca66af1b7e2c8503bcb298ca326ffe (patch)
tree6473df0f62e6ddba11cdced0fa8318b34d4f26cd
parentcdc901797ac4ce0d1feeec454ecdd29e8ef5d4ff (diff)
parente4d0169107b6359946997b06bbc2cdd4f119bf16 (diff)
downloadCMake-1c4c5ad545ca66af1b7e2c8503bcb298ca326ffe.zip
CMake-1c4c5ad545ca66af1b7e2c8503bcb298ca326ffe.tar.gz
CMake-1c4c5ad545ca66af1b7e2c8503bcb298ca326ffe.tar.bz2
Merge topic 'ctest-windows-std-handles' into release-3.29
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.cxx27
-rw-r--r--Source/cmSystemTools.h6
-rw-r--r--Source/ctest.cxx1
3 files changed, 0 insertions, 34 deletions
diff --git a/Source/cmSystemTools.cxx b/Source/cmSystemTools.cxx
index 964bac1..093a18b 100644
--- a/Source/cmSystemTools.cxx
+++ b/Source/cmSystemTools.cxx
@@ -2429,33 +2429,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 7e33e58..d12ab07 100644
--- a/Source/cmSystemTools.h
+++ b/Source/cmSystemTools.h
@@ -505,12 +505,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 f6a11b4..68c36df 100644
--- a/Source/ctest.cxx
+++ b/Source/ctest.cxx
@@ -177,7 +177,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]);