Merge pull request #1760 from ahoppen/log-to-stdout

Make `logHandler` print to stdout instead of stderr

Author: ahoppen

Sources/SKLogging/NonDarwinLogging.swift

@@ -281,7 +281,12 @@ 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 = { fputs($0 + "\n", stderr) }
+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")
+}
 
 /// The queue on which we log messages.
 ///