commit d8ffad56129e2c358c5b208edb62918c8ed1220c
author Benjamin Peterson <bp@benjamin.pe> Mon Mar 11 12:58:09 2019 -0700
committer Copybara-Service <copybara-worker@google.com> Mon Mar 11 12:59:40 2019 -0700
tree 18427c9583405348732134dff4136352bde4be3c
parent 05a1927cd1579cc630c26401f316dab06d866ad0

Make jvm_import_external.licenses optional.

I don't see why this attribute is required. Particularly since callers can simply pass licenses=[].

Closes #7622.

PiperOrigin-RevId: 237863515

tools/build_defs/repo/jvm.bzl

diff --git a/tools/build_defs/repo/jvm.bzl b/tools/build_defs/repo/jvm.bzl
index c3479d2..893921b 100644
--- a/tools/build_defs/repo/jvm.bzl
+++ b/tools/build_defs/repo/jvm.bzl
@@ -203,10 +203,7 @@
 jvm_import_external = repository_rule(
     attrs = {
         "rule_name": attr.string(mandatory = True),
-        "licenses": attr.string_list(
-            mandatory = True,
-            allow_empty = False,
-        ),
+        "licenses": attr.string_list(default = ["none"]),
         "artifact_urls": attr.string_list(
             mandatory = True,
             allow_empty = False,