fixed bug where bazel runs with curses when using the old sync view (b…

…azelbuild#7139)

The old sync view does not use a pty based terminal and cannot handle bazel output correctly when it runs with curses.

(cherry picked from commit 0fb3f81)

base/src/com/google/idea/blaze/base/buildview/BazelExecService.kt

@@ -66,7 +66,12 @@ class BazelExecService(private val project: Project) : Disposable {
   }
 
   private suspend fun execute(ctx: BlazeContext, cmdBuilder: BlazeCommand.Builder): Int {
-    val cmd = cmdBuilder.apply { addBlazeFlags("--curses=yes") }.build()
+    // the old sync view does not use a PTY based terminal
+    if (BuildViewMigration.present(ctx)) {
+      cmdBuilder.addBlazeFlags("--curses=yes")
+    }
+
+    val cmd = cmdBuilder.build()
     val root = cmd.effectiveWorkspaceRoot.orElseGet { WorkspaceRoot.fromProject(project).path() }
     val size = BuildViewScope.of(ctx)?.consoleSize ?: PtyConsoleView.DEFAULT_SIZE
 
@@ -182,5 +187,4 @@ class BazelExecService(private val project: Project) : Disposable {
       }
     }
   }
-}
-
+}

base/src/com/google/idea/blaze/base/toolwindow/ConsoleView.java

@@ -237,6 +237,10 @@ void println(PrintOutput output) {
   }
 
   private void println(String text, OutputType outputType) {
+    if (outputType == OutputType.PROCESS) {
+      text = text.stripTrailing();
+    }
+
     ansiEscapeDecoder.escapeText(
         text,
         outputType == OutputType.ERROR ? ProcessOutputTypes.STDERR : ProcessOutputTypes.STDOUT,