add toolchain interface to BuildServer

Author: Richard Howell

Sources/SKCore/BuildSystem.swift

@@ -32,5 +32,8 @@ public protocol BuildSystem {
   /// Returns the settings for the given url and language mode, if known.
   func settings(for: URL, _ language: Language) -> FileBuildSettings?
 
+  /// Returns the toolchain to use to compile this file
+  func toolchain(for: URL, _ language: Language) -> Toolchain?
+
   // TODO: notifications when settings change.
 }

Sources/SKCore/BuildSystemList.swift

@@ -38,4 +38,8 @@ extension BuildSystemList: BuildSystem {
     }
     return nil
   }
+
+  public func toolchain(for url: URL, _ language: Language) -> Toolchain? {
+    return providers.first?.toolchain(for: url, language)
+  }
 }

Sources/SKCore/CompilationDatabaseBuildSystem.swift

@@ -48,6 +48,8 @@ extension CompilationDatabaseBuildSystem: BuildSystem {
     )
   }
 
+  public func toolchain(for: URL, _ language: Language) -> Toolchain? { return nil }
+
   func database(for url: URL) -> CompilationDatabase? {
     if let path = try? AbsolutePath(validating: url.path) {
       return database(for: path)

Sources/SKCore/FallbackBuildSystem.swift

@@ -50,6 +50,8 @@ public final class FallbackBuildSystem: BuildSystem {
     }
   }
 
+  public func toolchain(for: URL, _ language: Language) -> Toolchain? { return nil }
+
   func settingsSwift(_ path: AbsolutePath) -> FileBuildSettings {
     var args: [String] = []
     if let sdkpath = sdkpath {

Sources/SKSupport/Logging.swift

@@ -107,7 +107,7 @@ public final class Logger {
   }
 
   public func setLogLevel(environmentVariable: String) {
-    if let string = Process.env[environmentVariable],
+    if let string = ProcessEnv.vars[environmentVariable],
        let level = try? LogLevel(argument: string)
     {
       currentLevel = level

Sources/SKSwiftPMWorkspace/SwiftPMWorkspace.swift

@@ -214,6 +214,11 @@ extension SwiftPMWorkspace: BuildSystem {
     return nil
   }
 
+  public func toolchain(for: LanguageServerProtocol.URL, _ language: Language) -> SKCore.Toolchain? {
+    return nil
+    
+  }
+
   /// Returns the resolved target description for the given file, if one is known.
   func targetDescription(for file: AbsolutePath) -> TargetBuildDescription? {
     if let td = fileToTarget[file] {

Sources/SourceKit/SourceKitServer.swift

@@ -134,7 +134,7 @@ public final class SourceKitServer: LanguageServer {
   }
 
   func toolchain(for url: URL, _ language: Language) -> Toolchain? {
-    if let toolchain = workspace?.buildSettings.settings(for: url, language)?.preferredToolchain {
+    if let toolchain = workspace?.buildSettings.toolchain(for: url, language) {
       return toolchain
     }
 

Tests/SKCoreTests/FallbackBuildSystemTests.swift

@@ -28,7 +28,6 @@ final class FallbackBuildSystemTests: XCTestCase {
     XCTAssertNil(bs.indexDatabasePath)
 
     let settings = bs.settings(for: source.asURL, .swift)!
-    XCTAssertNil(settings.preferredToolchain)
     XCTAssertNil(settings.workingDirectory)
 
     let args = settings.compilerArguments
@@ -53,7 +52,6 @@ final class FallbackBuildSystemTests: XCTestCase {
     bs.sdkpath = sdk
 
     let settings = bs.settings(for: source.asURL, .cpp)!
-    XCTAssertNil(settings.preferredToolchain)
     XCTAssertNil(settings.workingDirectory)
 
     let args = settings.compilerArguments