Start removing licenses(notice) and replacing with license()

bazelbuild / rules_pkg

Bazel rules for creating packages of many types (zip, tar, deb, rpm, ...)

Apache License 2.0

216 stars 171 forks source link

Start removing licenses(notice) and replacing with license() #742

Closed aiuto closed 4 months ago

aiuto commented 1 year ago

licenses() does not do anything.

aiuto commented 12 months ago

@teeler - can we do this?

aiuto commented 9 months ago

Lowered in urgency. It's going to make my imports harder if I do this today.

aiuto commented 4 months ago

friendly ping from reviewers. license() is a no-op in Bazel. It is not used in Blaze when applicable_licenses is used.

aiuto commented 4 months ago

The failing test is unrelated to this. It is that --noenable_bzlmod does not work with bazel at head, and bazelci is ignoreing the explicit bazel version.