More debugging

Author: olafurpg

scip-java/src/main/scala/com/sourcegraph/scip_java/buildtools/BazelBuildTool.scala

@@ -32,7 +32,10 @@ class BazelBuildTool(index: IndexCommand) extends BuildTool("Bazel", index) {
         buildCommand += "bazel"
         buildCommand += "build"
         buildCommand += "--noshow_progress"
-        buildCommand += "--sandbox_debug"
+
+        // The local strategy is required for now because we write SemanticDB and SCIP files
+        // to the provided targetroot directory.
+        buildCommand += "--spawn_strategy=local"
 
         val targetSpecs = ListBuffer.empty[String]
         if (index.buildCommand.isEmpty) {

scip-java/src/main/scala/com/sourcegraph/scip_java/buildtools/ScipBuildTool.scala

@@ -424,7 +424,7 @@ class ScipBuildTool(index: IndexCommand) extends BuildTool("SCIP", index) {
       .iterator()
     if (compilers.hasNext) {
       val classpath = deps.classpath ++ scalaLibrary
-      val argsfile = tmp.resolve("javacopts.txt")
+      val argsfile = targetroot.resolve("javacopts.txt")
       Files.createDirectories(argsfile.getParent)
       Files.write(
         argsfile,
@@ -492,7 +492,7 @@ class ScipBuildTool(index: IndexCommand) extends BuildTool("SCIP", index) {
           AbsolutePath.of(Paths.get(path), index.workingDirectory).toString
         )
     actualClasspath ++= deps.classpath.map(_.toString)
-    val argsfile = tmp.resolve("javacopts.txt")
+    val argsfile = targetroot.resolve("javacopts.txt")
     val arguments = ListBuffer.empty[String]
     arguments += "-encoding"
     arguments += "utf8"

semanticdb-javac/src/main/java/com/sourcegraph/semanticdb_javac/SemanticdbTaskListener.java

@@ -7,7 +7,9 @@
 import com.sun.tools.javac.model.JavacTypes;
 
 import javax.tools.JavaFileObject;
+import java.io.ByteArrayOutputStream;
 import java.io.IOException;
+import java.io.PrintWriter;
 import java.net.URI;
 import java.nio.file.Files;
 import java.nio.file.Path;
@@ -71,11 +73,20 @@ public void finished(TaskEvent e) {
             new CompilationUnitException(
                 String.valueOf(e.getSourceFile().toUri().toString()), throwable);
       }
-      reporter.exception(throwable, e.getCompilationUnit(), e.getCompilationUnit());
+      this.reportException(throwable, e);
       throw new RuntimeException(ex.getMessage(), throwable);
     }
   }
 
+  // Uses reporter.error with the full stack trace of the exception instead of reporter.exception
+  // because reporter.exception doesn't seem to print any meaningful information about the
+  // exception, it just prints the location with an empty message.
+  private void reportException(Throwable exception, TaskEvent e) {
+    ByteArrayOutputStream baos = new ByteArrayOutputStream();
+    exception.printStackTrace(new PrintWriter(baos));
+    reporter.error(baos.toString(), e.getCompilationUnit(), e.getCompilationUnit());
+  }
+
   private void onFinishedAnalyze(TaskEvent e) {
     Result<Path, String> path = semanticdbOutputPath(options, e);
     if (path != null) {
@@ -97,7 +108,7 @@ private void writeSemanticdb(TaskEvent event, Path output, Semanticdb.TextDocume
       Files.createDirectories(output.getParent());
       Files.write(output, bytes);
     } catch (IOException e) {
-      reporter.exception(e, event.getCompilationUnit(), event.getCompilationUnit());
+      this.reportException(e, event);
     }
   }