Commit Graph

7 Commits (ec23ea2c79cea23d75807f1bf1324e8f352a685b)

Author SHA1 Message Date
Jordan Brown ec23ea2c79 EXTENDED_LICENSE_CLASSIFIERS for non-SPDX license validation
Add support for validating licenses that aren't in the official SPDX license list but are commonly used in our codebase. These are the values output by the license classifier. This introduces a new EXTENDED_LICENSE_CLASSIFIERS set that contains these custom license identifiers, organized by restrictiveness level similar to ALLOWED_SPDX_LICENSES.


Bug: 379977497
Change-Id: I18e0b38572ee4df783573ea338b55ac237d8134d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/6107859
Reviewed-by: Rachael Newitt <renewitt@google.com>
Commit-Queue: Jordan Brown <rop@google.com>
7 months ago
Jordan Brown fcb6178914 Adding notice (and less restrictive) licenses to ALLOWED_SPDX_LICENSES
These are the licenses that are currently in chromium and are 'unencumbered', 'permissive', or 'notice' licenses

Change-Id: I40281cdb2e8b7e1af7a06e0ff234e2005903d8c1
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/6055002
Reviewed-by: Jordan Brown <rop@google.com>
Reviewed-by: Rachael Newitt <renewitt@google.com>
Commit-Queue: Jordan Brown <rop@google.com>
7 months ago
Jordan Brown 3c68cd01f4 Restructuring 'ALLOWED_SPDX_LICENSES'
.. to show license classifications via comments

Change-Id: Id18ee6a597457ecaaafbd24373d78aca09ba7058
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/6055001
Commit-Queue: Jordan Brown <rop@google.com>
Reviewed-by: Jordan Brown <rop@google.com>
Reviewed-by: Rachael Newitt <renewitt@google.com>
7 months ago
Joshua Pawlicki f548b21cd3 Add LZMA-SDK-9.22 to the license allowlist.
https://spdx.org/licenses/LZMA-SDK-9.22.html

chromium //third_party/lzma_sdk currently calls the license
"Public Domain":
https://source.chromium.org/chromium/chromium/src/+/main:third_party/lzma_sdk/README.chromium;l=6;drc=51397c51193b75500c2ae5ef5dcbc90638b69dc4

but LZMA-SDK-9.22 seems to be a better identifier for the license.

Change-Id: I0c0c237bbc52a0a5912d8f41b1f3c6a4a9663d77
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/6093870
Auto-Submit: Joshua Pawlicki <waffles@chromium.org>
Commit-Queue: Jordan Brown <rop@google.com>
Reviewed-by: Jordan Brown <rop@google.com>
7 months ago
Jordan Brown 833f488142 Revert "Adding Supplement:Apache-2.0 to chromium license allowlist."
This reverts commit 9dd0755b69.

Reason for revert: 'Supplement:Apache-2.0' is considered to be allowed, but is not technically a valid SPDX identifier.

Original change's description:
> Adding Supplement:Apache-2.0 to chromium license allowlist.
>
> The 'Supplement:Apache-2.0' contains the full license text and is the preferred version over just 'Apache-2.0'.
>
> Change-Id: Ie4566ffdcd20771a0170190270d8d45e1b77980d
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/6040532
> Reviewed-by: Jordan Brown <rop@google.com>
> Reviewed-by: Rachael Newitt <renewitt@google.com>
> Commit-Queue: Rachael Newitt <renewitt@google.com>
> Auto-Submit: Jordan Brown <rop@google.com>

Change-Id: Ic6529514304b826dbc8840a468bab47321c30471
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/6040754
Commit-Queue: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
Bot-Commit: Rubber Stamper <rubber-stamper@appspot.gserviceaccount.com>
Auto-Submit: Jordan Brown <rop@google.com>
8 months ago
Jordan 9dd0755b69 Adding Supplement:Apache-2.0 to chromium license allowlist.
The 'Supplement:Apache-2.0' contains the full license text and is the preferred version over just 'Apache-2.0'.

Change-Id: Ie4566ffdcd20771a0170190270d8d45e1b77980d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/6040532
Reviewed-by: Jordan Brown <rop@google.com>
Reviewed-by: Rachael Newitt <renewitt@google.com>
Commit-Queue: Rachael Newitt <renewitt@google.com>
Auto-Submit: Jordan Brown <rop@google.com>
8 months ago
Jordan b5eb54d5c1 Updating presubmit to check license is a valid spdx identifier.
Further details are available in https://docs.google.com/document/d/1x4GLly7KJ2xmlJRvsswJlmVSLLjBjk5NOxWZ-Dryg_A

Bug: 358504615
Change-Id: I76997454140ee63aea6b6f492669800efce271be
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/6021876
Reviewed-by: Joey Scarr <jsca@google.com>
Commit-Queue: Jordan Brown <rop@google.com>
Reviewed-by: Andrew Grieve <agrieve@chromium.org>
Reviewed-by: Rick Byers <rbyers@chromium.org>
Reviewed-by: Rachael Newitt <renewitt@google.com>
8 months ago