Add useX86Host setting for language service

This change adds a new 'useX86Host' setting for the language service which
causes the 32-bit host to be launched in 64-bit Windows.

Part of the solution for #55.

Author: daviwil

package.json

@@ -221,15 +221,20 @@
             "type": "object",
             "title": "PowerShell Configuration",
             "properties": {
+                "powershell.useX86Host": {
+                    "type": "boolean",
+                    "default": false,
+                    "description": "If true, causes the 32-bit language service to be used on 64-bit Windows.  On 32-bit Windows this setting has no effect.  This setting does not affect the debugger which has its own architecture configuration."
+                },
                 "powershell.scriptAnalysis.enable": {
                     "type": "boolean",
                     "default": true,
                     "description": "Enables real-time script analysis using PowerShell Script Analyzer."
                 },
                 "powershell.developer.editorServicesHostPath": {
                     "type": "string",
-                    "default": "../bin/Microsoft.PowerShell.EditorServices.Host.exe",
-                    "description": "Specifies the path to the PowerShell Editor Services host executable."
+                    "default": "../bin/",
+                    "description": "Specifies the path to the folder containing the PowerShell Editor Services host executables."
                 },
                 "powershell.developer.editorServicesLogLevel": {
                     "type": "string",

src/main.ts

@@ -76,37 +76,45 @@ export function activate(context: vscode.ExtensionContext): void {
             args.push('/logLevel:' + settings.developer.editorServicesLogLevel)
         }
 
-        let serverPath = resolveLanguageServerPath(settings);
-        let serverOptions = {
-            run: {
-                command: serverPath,
-                args: args
-            },
-            debug: {
-                command: serverPath,
-                args: ['/waitForDebugger']
-            }
-        };
-
-        let clientOptions: LanguageClientOptions = {
-            documentSelector: [PowerShellLanguageId],
-            synchronize: {
-                configurationSection: PowerShellLanguageId,
-                //fileEvents: vscode.workspace.createFileSystemWatcher('**/.eslintrc')
+        try
+        {
+            let serverPath = resolveLanguageServerPath(settings);
+            let serverOptions = {
+                run: {
+                    command: serverPath,
+                    args: args
+                },
+                debug: {
+                    command: serverPath,
+                    args: ['/waitForDebugger']
+                }
+            };
+
+            let clientOptions: LanguageClientOptions = {
+                documentSelector: [PowerShellLanguageId],
+                synchronize: {
+                    configurationSection: PowerShellLanguageId,
+                    //fileEvents: vscode.workspace.createFileSystemWatcher('**/.eslintrc')
+                }
             }
-        }
 
-        languageServerClient =
-            new LanguageClient(
-                'PowerShell Editor Services',
-                serverOptions,
-                clientOptions);
+            languageServerClient =
+                new LanguageClient(
+                    'PowerShell Editor Services',
+                    serverOptions,
+                    clientOptions);
 
-        languageServerClient.onReady().then(
-            () => registerFeatures(),
-            (reason) => vscode.window.showErrorMessage("Could not start language service: " + reason));
+            languageServerClient.onReady().then(
+                () => registerFeatures(),
+                (reason) => vscode.window.showErrorMessage("Could not start language service: " + reason));
 
-        languageServerClient.start();
+            languageServerClient.start();
+        }
+        catch (e)
+        {
+            vscode.window.showErrorMessage(
+                "The language service could not be started: " + e);
+        }
     }
 }
 
@@ -133,11 +141,17 @@ function resolveLanguageServerPath(settings: settingsManager.ISettings): string
     if (editorServicesHostPath) {
         console.log("Found Editor Services path from config: " + editorServicesHostPath);
 
+        // Does the path end in a .exe?  Alert the user if so.
+        if (path.extname(editorServicesHostPath) != '') {
+            throw "The editorServicesHostPath setting must point to a directory, not a file.";
+        }
+
         // Make the path absolute if it's not
         editorServicesHostPath =
             path.resolve(
                 __dirname,
-                editorServicesHostPath);
+                editorServicesHostPath,
+                getHostExeName(settings.useX86Host));
 
         console.log("    Resolved path to: " + editorServicesHostPath);
     }
@@ -148,10 +162,17 @@ function resolveLanguageServerPath(settings: settingsManager.ISettings): string
                 __dirname,
                 '..',
                 'bin',
-                'Microsoft.PowerShell.EditorServices.Host.exe');
+                getHostExeName(settings.useX86Host));
 
         console.log("Using default Editor Services path: " + editorServicesHostPath);
     }
 
     return editorServicesHostPath;
 }
+
+function getHostExeName(useX86Host: boolean): string {
+    // The useX86Host setting is only relevant on 64-bit OS
+    var is64BitOS = process.env.hasOwnProperty('PROCESSOR_ARCHITEW6432');
+    var archText = useX86Host && is64BitOS ? ".x86" : "";
+    return "Microsoft.PowerShell.EditorServices.Host" + archText + ".exe";
+}

src/settings.ts

@@ -17,6 +17,7 @@ export interface IDeveloperSettings {
 }
 
 export interface ISettings {
+    useX86Host?: boolean,
     scriptAnalysis?: IScriptAnalysisSettings,
     developer?: IDeveloperSettings,
 }
@@ -29,12 +30,13 @@ export function load(myPluginId: string): ISettings {
     };
 
     let defaultDeveloperSettings = {
-        editorServicesHostPath: "../bin/Microsoft.PowerShell.EditorServices.Host.exe",
+        editorServicesHostPath: "../bin/",
         editorServicesLogLevel: "Normal",
         editorServicesWaitForDebugger: false
     }
 
     return {
+        useX86Host: configuration.get<boolean>("useX86Host", false),
         scriptAnalysis: configuration.get<IScriptAnalysisSettings>("scriptAnalysis", defaultScriptAnalysisSettings),
         developer: configuration.get<IDeveloperSettings>("developer", defaultDeveloperSettings)
     }