From 993b90741b4e9d8ab000cfb3ec01f4c0b59325c5 Mon Sep 17 00:00:00 2001 From: Alex Hoppen Date: Wed, 16 Oct 2024 11:15:51 -0700 Subject: [PATCH] Revert "Make `logHandler` print to stdout instead of stderr" This reverts commit 6fbddd486707f683735e24d92183624b1120aa1e. --- Sources/SKLogging/NonDarwinLogging.swift | 7 +------ 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/Sources/SKLogging/NonDarwinLogging.swift b/Sources/SKLogging/NonDarwinLogging.swift index 6ddadb56c..fd9c5940a 100644 --- a/Sources/SKLogging/NonDarwinLogging.swift +++ b/Sources/SKLogging/NonDarwinLogging.swift @@ -281,12 +281,7 @@ actor LogHandlerActor { /// The handler that is called to log a message from `NonDarwinLogger` unless `overrideLogHandler` is set on the logger. @LogHandlerActor -var logHandler: @Sendable (String) async -> Void = { message in - // Print to stdout. When using the sourcekit-lsp binary, we will have stdout redirected to stderr, so it ends up - // logging to stderr. During test execution, we log to stdout, which is generally better handled than logging to - // stderr by XCTest (for some reason logging to stderr will hang test execution when running tests in parallel). - print(message + "\n") -} +var logHandler: @Sendable (String) async -> Void = { fputs($0 + "\n", stderr) } /// The queue on which we log messages. ///