From d0f0c4b75b47891b1185f0476a699cdbe5cac8ac Mon Sep 17 00:00:00 2001
From: "Cagri (Charlie) Aslan" <caslan@microsoft.com>
Date: Fri, 18 Mar 2016 11:17:22 -0700
Subject: [PATCH] Improve debugger install error messages

---
 package.json         |  3 ++-
 src/coreclr-debug.ts | 25 ++++++++++++++++++-------
 2 files changed, 20 insertions(+), 8 deletions(-)

diff --git a/package.json b/package.json
index 9c8163844..b3b949df4 100644
--- a/package.json
+++ b/package.json
@@ -25,7 +25,8 @@
     "run-in-terminal": "*",
     "semver": "*",
     "vscode-extension-telemetry": "0.0.4",
-    "tmp": "0.0.28"
+    "tmp": "0.0.28",
+    "open": "*"
   },
   "devDependencies": {
     "gulp": "^3.9.1",
diff --git a/src/coreclr-debug.ts b/src/coreclr-debug.ts
index df58013ef..da1fbd4ca 100644
--- a/src/coreclr-debug.ts
+++ b/src/coreclr-debug.ts
@@ -27,8 +27,15 @@ export function installCoreClrDebug(context: vscode.ExtensionContext) {
     }
     
     if (!isOnPath('dotnet')) {
-        // TODO: In a future release, this should be an error. For this release, we will let it go
-        console.log("The .NET CLI tools are not installed. .NET Core debugging will not be enabled.");
+        const getDotNetMessage = "Get .NET CLI tools"; 
+        vscode.window.showErrorMessage("The .NET CLI tools cannot be located. .NET Core debugging will not be enabled. Make sure .NET CLI tools are installed and are on the path.",
+            getDotNetMessage).then(function (value) {
+                if (value === getDotNetMessage) {
+                    var open = require('open');
+                    open("http://dotnet.github.io/getting-started/");
+                }
+            });
+            
         return;
     }
     
@@ -45,9 +52,8 @@ export function installCoreClrDebug(context: vscode.ExtensionContext) {
         };
     })();
 
-    _channel.appendLine("Downloading and configuring the .NET Core Debugger...");
-    _channel.show(vscode.ViewColumn.Three);
-    
+    vscode.window.setStatusBarMessage("Downloading and configuring the .NET Core Debugger...");
+       
     let installStage = 'dotnet restore';
     let installError = '';
     
@@ -71,10 +77,15 @@ export function installCoreClrDebug(context: vscode.ExtensionContext) {
         return writeCompletionFile();
     }).then(function() {
         installStage = "completeSuccess";
-        _channel.appendLine('Succesfully installed .NET Core Debugger.');
+        vscode.window.setStatusBarMessage('Successfully installed .NET Core Debugger.');
     })
     .catch(function(error) {
-        _channel.appendLine('Error while installing .NET Core Debugger.');
+        const viewLogMessage = "View Log";
+        vscode.window.showErrorMessage('Error while installing .NET Core Debugger.', viewLogMessage).then(function (value) {
+            if (value === viewLogMessage) {
+                _channel.show(vscode.ViewColumn.Three);
+            }
+        })
         
         installError = error.toString();
         console.log(error);