fix: Conditional usage of os.log for Linux support

surpher · Jul 29, 2021 · bc1c129 · bc1c129
1 parent 703bb15
commit bc1c129
Show file tree

Hide file tree

Showing 2 changed files with 21 additions and 3 deletions.
diff --git a/Sources/PactSwiftToolbox/Logger.swift b/Sources/PactSwiftToolbox/Logger.swift
@@ -16,13 +16,18 @@
 //
 
 import Foundation
+
+#if !os(Linux)
 import os.log
+#endif
+
+// TODO: - Use something like apple/swift-log instead of os.log
 
 public enum Logger {
 
-	/// Uses `os_log` to log Pact related messages.
+	/// Logs Pact related messages.
 	///
-	/// Looks for environment variable `PACT_ENABLE_LOGGING = "all"`. Can be set in project's scheme.
+	/// Looks for environment variable `PACT_ENABLE_LOGGING = "all"`. Can be set in project's scheme. Uses `os_log` on Apple platforms.
 	///
 	/// - Parameters:
 	///    - message: The message to log
@@ -36,21 +41,30 @@ public enum Logger {
 		let stringData = data.flatMap { String(data: $0, encoding: .utf8) } ?? ""
 
 		if #available(iOS 10, OSX 10.14, *) {
+			#if !os(Linux)
 			os_log(
 				"PactSwift: %{private}s",
 				log: .default,
 				type: .default,
 				"\(message): \(stringData)"
 			)
+			#else
+			print(message: "PactSwift: \(message)\n\(stringData)")
+			#endif
+
 		} else {
-			debugPrint("PactSwift: \(message)\n\(stringData)")
+			print(message: "PactSwift: \(message)\n\(stringData)")
 		}
 	}
 
 }
 
 private extension Logger {
 
+	static func print(message: String) {
+		debugPrint(message)
+	}
+
 	enum PactLoggingLevel: String {
 		case all
 		case disabled

diff --git a/Sources/PactSwiftToolbox/SocketBinder.swift b/Sources/PactSwiftToolbox/SocketBinder.swift
@@ -15,6 +15,8 @@
 //  IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
 //
 
+#if !os(Linux)
+
 import Foundation
 
 @frozen public enum SocketBinder {
@@ -80,3 +82,5 @@ private extension SocketBinder {
 	}
 
 }
+
+#endif