Merge pull request #1513 from tgodzik/fix-print

improvement: Print Java all at once
scalameta · May 29, 2024 · f767021 · f767021
2 parents cda40b5 + 159df6c
commit f767021
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/packages/metals-languageclient/src/getJavaHome.ts b/packages/metals-languageclient/src/getJavaHome.ts
@@ -37,7 +37,6 @@ async function validateJavaVersion(
     });
 
     javaVersionOut.stderr?.on("data", (out: Buffer) => {
-      outputChannel.appendLine(`${javaBin} -version:`);
       const msg = "\t" + out.toString().trim().split("\n").join("\n\t");
       outputChannel.appendLine(msg);
     });

diff --git a/packages/metals-languageclient/src/setupCoursier.ts b/packages/metals-languageclient/src/setupCoursier.ts
@@ -22,8 +22,8 @@ export async function setupCoursier(
   output: OutputChannel
 ): Promise<{ coursier: string; javaHome: string }> {
   const handleOutput = (out: Buffer) => {
-    const msg = out.toString().trim();
-    output.appendLine("Coursier: \n" + msg);
+    const msg = "\t" + out.toString().trim().split("\n").join("\n\t");
+    output.appendLine(msg);
   };
 
   const resolveCoursier = async () => {
@@ -82,7 +82,7 @@ export async function setupCoursier(
 
   if (!javaHome && coursier) {
     output.appendLine(
-      `No installed java with version ${javaVersion} found. Will fetch one using coursier.`
+      `No installed java with version ${javaVersion} found. Will fetch one using coursier:`
     );
     javaHome = await resolveJavaHomeWithCoursier(coursier);
   }