Update message to include note on GOBIN

microsoft · Jan 28, 2020 · 1f5fa4f · 1f5fa4f
1 parent b52692e
commit 1f5fa4f
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/src/goInstallTools.ts b/src/goInstallTools.ts
@@ -144,13 +144,13 @@ export function installTools(missing: Tool[], goVersion: GoVersion): Promise<voi
  return;
  }
 
- let binpath = toolsGopath + path.sep + 'bin';
- if (envForTools['GOBIN'] != undefined && envForTools['GOBIN'] != '') {
- binpath = 'GOBIN path ' + envForTools['GOBIN'];
+ let installingMsg = `Installing ${missing.length} ${missing.length > 1 ? 'tools' : 'tool'} at `;
+ if (envForTools['GOBIN']) {
+ installingMsg += `the configured GOBIN: ${envForTools['GOBIN']}`;
+ } else {
+ installingMsg += toolsGopath + path.sep + 'bin';
  }
 
- let installingMsg = `Installing ${missing.length} ${missing.length > 1 ? 'tools' : 'tool'} at ${binpath}`;
-
  // If the user is on Go >= 1.11, tools should be installed with modules enabled.
  // This ensures that users get the latest tagged version, rather than master,
  // which may be unstable.