Allow users to configure log level from compiler config (#254)

Author: stephenjudkins

docs/stardoc/providers.md

@@ -439,7 +439,7 @@ A Scala compiler plugin
 ## ZincConfiguration
 
 <pre>
-ZincConfiguration(<a href="#ZincConfiguration-compiler_bridge">compiler_bridge</a>, <a href="#ZincConfiguration-compile_worker">compile_worker</a>)
+ZincConfiguration(<a href="#ZincConfiguration-compiler_bridge">compiler_bridge</a>, <a href="#ZincConfiguration-compile_worker">compile_worker</a>, <a href="#ZincConfiguration-log_level">log_level</a>)
 </pre>
 
 Zinc configuration.
@@ -464,6 +464,12 @@ Zinc configuration.
         <p>the worker label for compilation with Zinc</p>
       </td>
     </tr>
+    <tr id="ZincConfiguration-log_level">
+      <td><code>log_level</code></td>
+      <td>
+        <p>log level for the Zinc compiler</p>
+      </td>
+    </tr>
   </tbody>
 </table>
 

rules/private/phases/phase_zinc_compile.bzl

@@ -62,6 +62,7 @@ def phase_zinc_compile(ctx, g):
     args.add_all("--plugins", g.classpaths.plugin)
     args.add_all("--source_jars", g.classpaths.src_jars)
     args.add("--tmp", tmp.path)
+    args.add("--log_level", zinc_configuration.log_level)
     args.add_all("--", g.classpaths.srcs)
     args.set_param_file_format("multiline")
     args.use_param_file("@%s", use_always = True)

rules/providers.bzl

@@ -63,6 +63,7 @@ ZincConfiguration = provider(
     fields = {
         "compiler_bridge": "compiled Zinc compiler bridge",
         "compile_worker": "the worker label for compilation with Zinc",
+        "log_level": "log level for the Zinc compiler",
     },
 )
 

rules/scala.bzl

@@ -512,6 +512,10 @@ _configure_zinc_scala = rule(
         "global_scalacopts": attr.string_list(
             doc = "Scalac options that will always be enabled.",
         ),
+        "log_level": attr.string(
+            doc = "Compiler log level",
+            default = "warn",
+        ),
         "deps_direct": attr.string(default = "error"),
         "deps_used": attr.string(default = "error"),
         "_compile_worker": attr.label(

rules/scala/private/provider.bzl

@@ -44,6 +44,7 @@ def configure_zinc_scala_implementation(ctx):
         _ZincConfiguration(
             compile_worker = ctx.attr._compile_worker,
             compiler_bridge = ctx.file.compiler_bridge,
+            log_level = ctx.attr.log_level,
         ),
         _DepsConfiguration(
             direct = ctx.attr.deps_direct,

tests/compile/log_level/BUILD

@@ -0,0 +1,8 @@
+load("@rules_scala_annex//rules:scala.bzl", "scala_binary")
+
+scala_binary(
+    name = "lib",
+    srcs = ["Example.scala"],
+    scala = "//scala:zinc_2_12_6_log_level_debug",
+    deps = [],
+)

tests/compile/log_level/Example.scala

@@ -0,0 +1,3 @@
+object Example {
+  def foo = 42
+}

tests/compile/log_level/test

@@ -0,0 +1,7 @@
+#!/bin/bash -e
+. "$(dirname "$0")"/../../common.sh
+
+bazel build :lib > output 2>&1
+
+cat output | grep "Compiling 1 Scala source"
+rm output

tests/scala/BUILD

@@ -156,3 +156,14 @@ configure_zinc_scala(
     version = "2.12.6",
     visibility = ["//scalacopts/rule:__subpackages__"],
 )
+
+# for log_level test
+configure_zinc_scala(
+    name = "zinc_2_12_6_log_level_debug",
+    compiler_bridge = ":compiler_bridge_2_12_6",
+    compiler_classpath = compiler_classpath_2_12_6,
+    log_level = "debug",
+    runtime_classpath = runtime_classpath_2_12_6,
+    version = "2.12.6",
+    visibility = ["//visibility:public"],
+)