From 82a45a124bd312818cd7d5c782671e7463c08964 Mon Sep 17 00:00:00 2001
From: Tony Aiuto <aiuto@google.com>
Date: Mon, 24 Apr 2023 21:45:15 -0400
Subject: [PATCH] Move the manifest() rule to the examples tree.

- It does nothing that useful, and does not actually work yet.
- It relies on the older, get_licenses_info, API.

Part of #85
---
 examples/manifest/android_mock.bzl |  2 +-
 examples/manifest/manifest.bzl     | 89 ++++++++++++++++++++++++++++++
 rules/compliance.bzl               | 34 ------------
 3 files changed, 90 insertions(+), 35 deletions(-)
 create mode 100644 examples/manifest/manifest.bzl

diff --git a/examples/manifest/android_mock.bzl b/examples/manifest/android_mock.bzl
index 0dee3c9..81b7417 100644
--- a/examples/manifest/android_mock.bzl
+++ b/examples/manifest/android_mock.bzl
@@ -1,4 +1,4 @@
-load("@rules_license//rules:compliance.bzl", "manifest")
+load("manifest.bzl", "manifest")
 
 """This is a proof of concept to show how to modify a macro definition to
 create a sub-graph allowing for build time injection of license information. We
diff --git a/examples/manifest/manifest.bzl b/examples/manifest/manifest.bzl
new file mode 100644
index 0000000..7e4adb4
--- /dev/null
+++ b/examples/manifest/manifest.bzl
@@ -0,0 +1,89 @@
+# Copyright 2023 Google LLC
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+# https://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+"""An example using gather_licenses_info as input to another action."""
+
+load(
+    "@rules_license//rules:gather_licenses_info.bzl",
+    "gather_licenses_info",
+)
+load(
+    "@rules_license//rules/private:gathering_providers.bzl",
+    "TransitiveLicensesInfo",
+)
+
+def get_licenses_mapping(deps, warn = False):
+    """Creates list of entries representing all licenses for the deps.
+
+    Args:
+
+      deps: a list of deps which should have TransitiveLicensesInfo providers.
+            This requires that you have run the gather_licenses_info
+            aspect over them
+
+      warn: boolean, if true, display output about legacy targets that need
+            update
+
+    Returns:
+      {File:package_name}
+    """
+    tls = []
+    for dep in deps:
+        lds = dep[TransitiveLicensesInfo].licenses
+        tls.append(lds)
+
+    ds = depset(transitive = tls)
+
+    # Ignore any legacy licenses that may be in the report
+    mappings = {}
+    for lic in ds.to_list():
+        if type(lic.license_text) == "File":
+            mappings[lic.license_text] = lic.package_name
+        elif warn:
+            print("Legacy license %s not included, rule needs updating" % lic.license_text)
+    return mappings
+
+
+def _manifest_impl(ctx):
+    # Gather all licenses and make it available as deps for downstream rules
+    # Additionally write the list of license filenames to a file that can
+    # also be used as an input to downstream rules.
+    licenses_file = ctx.actions.declare_file(ctx.attr.out.name)
+    mappings = get_licenses_mapping(ctx.attr.deps, ctx.attr.warn_on_legacy_licenses)
+    ctx.actions.write(
+        output = licenses_file,
+        content = "\n".join([",".join([f.path, p]) for (f, p) in mappings.items()]),
+    )
+    return [DefaultInfo(files = depset(mappings.keys()))]
+
+_manifest = rule(
+    implementation = _manifest_impl,
+    doc = """Internal tmplementation method for manifest().""",
+    attrs = {
+        "deps": attr.label_list(
+            doc = """List of targets to collect license files for.""",
+            aspects = [gather_licenses_info],
+        ),
+        "out": attr.output(
+            doc = """Output file.""",
+            mandatory = True,
+        ),
+        "warn_on_legacy_licenses": attr.bool(default = False),
+    },
+)
+
+def manifest(name, deps, out = None, **kwargs):
+    if not out:
+        out = name + ".manifest"
+    _manifest(name = name, deps = deps, out = out, **kwargs)
+
diff --git a/rules/compliance.bzl b/rules/compliance.bzl
index 2fb04ab..4fec505 100644
--- a/rules/compliance.bzl
+++ b/rules/compliance.bzl
@@ -85,40 +85,6 @@ _check_license = rule(
 def check_license(**kwargs):
     _check_license(**kwargs)
 
-def _manifest_impl(ctx):
-    # Gather all licenses and make it available as deps for downstream rules
-    # Additionally write the list of license filenames to a file that can
-    # also be used as an input to downstream rules.
-    licenses_file = ctx.actions.declare_file(ctx.attr.out.name)
-    mappings = get_licenses_mapping(ctx.attr.deps, ctx.attr.warn_on_legacy_licenses)
-    ctx.actions.write(
-        output = licenses_file,
-        content = "\n".join([",".join([f.path, p]) for (f, p) in mappings.items()]),
-    )
-    return [DefaultInfo(files = depset(mappings.keys()))]
-
-_manifest = rule(
-    implementation = _manifest_impl,
-    doc = """Internal tmplementation method for manifest().""",
-    attrs = {
-        "deps": attr.label_list(
-            doc = """List of targets to collect license files for.""",
-            aspects = [gather_licenses_info],
-        ),
-        "out": attr.output(
-            doc = """Output file.""",
-            mandatory = True,
-        ),
-        "warn_on_legacy_licenses": attr.bool(default = False),
-    },
-)
-
-def manifest(name, deps, out = None, **kwargs):
-    if not out:
-        out = name + ".manifest"
-
-    _manifest(name = name, deps = deps, out = out, **kwargs)
-
 def _licenses_used_impl(ctx):
     # Gather all licenses and make it available as JSON
     write_licenses_info(ctx, ctx.attr.deps, ctx.outputs.out)