Add new Apache or MIT license rule #3738#3750
Add new Apache or MIT license rule #3738#3750AyanSinhaMahapatra merged 3 commits intoaboutcode-org:developfrom
Conversation
Signed-off-by: Vasily Pozdnyakov <[email protected]>
|
Hi, I am new to new rules creation and have a couple of questions:
|
|
@vasily-pozdnyakov welcome and thanks for the PR!
When you get SCTK from a git checkout and install scancode to run locally (or get scancode from github releases or pip install) you have the license indexes at
That's mostly correct, this could (and does also) generate a lot of false positives in a lot of cases. See #2878 We are working on resolving this and I have a pending PR at #3254 testing out adding required phrases automatically and massively across all the rules from the following:
|
Signed-off-by: Vasily Pozdnyakov <[email protected]>
6d1b58a to
df8cd3d
Compare
pombredanne
left a comment
There was a problem hiding this comment.
Thanks. Just a minor but important nit wrt. to the license expression. The norm for MIT and Pache in the Rust ecosystem is to be a choice and buck2 and this new rules is not an exception.
We could also later add more rules for derived notices such as https://github.com/facebook/buck2?tab=readme-ov-file#license
Signed-off-by: Vasily Pozdnyakov <[email protected]>
|
Thanks++ @vasily-pozdnyakov, merging! |
Fixes #3738
Tasks