Merge pull request #94 from lucidsoftware/jjudd-write-text-files

Write text analysis store files to help debug determinism issues

Author: jjudd

rules/private/phases/phase_zinc_compile.bzl

@@ -18,6 +18,7 @@ load(
 def phase_zinc_compile(ctx, g):
     toolchain = ctx.toolchains["//rules/scala:toolchain_type"]
     analysis_store = ctx.actions.declare_file("{}/analysis_store.gz".format(ctx.label.name))
+    analysis_store_text = ctx.actions.declare_file("{}/analysis_store.text.gz".format(ctx.label.name))
     mains_file = ctx.actions.declare_file("{}.jar.mains.txt".format(ctx.label.name))
     used = ctx.actions.declare_file("{}/deps_used.txt".format(ctx.label.name))
     tmp = ctx.actions.declare_directory("{}/tmp".format(ctx.label.name))
@@ -76,6 +77,7 @@ def phase_zinc_compile(ctx, g):
         g.classpaths.jar,
         mains_file,
         analysis_store,
+        analysis_store_text,
         used,
         tmp,
     ] + g.semanticdb.outputs

src/main/scala/higherkindness/rules_scala/workers/zinc/compile/ZincRunner.scala

@@ -335,17 +335,13 @@ object ZincRunner extends WorkerMain[ZincRunnerWorkerConfig] {
       )
     }
 
-    // This will be true if the `--worker_verbose` Bazel flag is set
-    if (task.verbosity >= 10) {
-      val analysisStoreText = AnalysisUtil.getAnalysisStore(
-        new File(pathString.substring(0, pathString.length() - 3) + ".text.gz"),
-        true,
-        readWriteMappers,
-      )
-
-      analysisStoreText.set(AnalysisContents.create(resultAnalysis, compileResult.setup))
-    }
+    val analysisStoreText = AnalysisUtil.getAnalysisStore(
+      new File(pathString.substring(0, pathString.length() - 3) + ".text.gz"),
+      true,
+      readWriteMappers,
+    )
 
+    analysisStoreText.set(AnalysisContents.create(resultAnalysis, compileResult.setup))
     analysisStore.set(AnalysisContents.create(resultAnalysis, compileResult.setup))
 
     // create used deps