Skip to content

Omnibus license updates July/Aug 21 - #2626

Merged
pombredanne merged 21 commits into
developfrom
omnibus-license-updates-july-21
Aug 6, 2021
Merged

pombredanne merged 21 commits into
developfrom
omnibus-license-updates-july-21

Conversation

@pombredanne

Copy link
Copy Markdown
Member

This PR makes various improvements to license detection
and includes several new and improved license detection
rules and new licenses.

No license should be detected in "License a Private"
with a rule "license Private" as there is a stopword inside and
this is a short rule.

Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
This notice is vague and matches references to bsd-new or
bsd-simplified when seen in the wild. Therefore we lower the relevance
and reference bsd-new which covers also bsd-simplified.

Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
This is unambiguously a CC0 license reference

Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
XVId is under the gpl-2.0-plus, not the gpl-2.0.

Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
These are corner cases found in handling "ignorables" clues for
the latest licenses and license rules.

Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
"filter_if_only_known_words_rule()" is no longer apply for small
license detection rules. This was discarding too many matches. Instead
the correct approach is to create false positive detection rules for
the few cases that we encounter.

Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
Thse are now aligned with the latest copyright detection

Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
Return whole lines based on requested option, not based on filetype

Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
@pombredanne
pombredanne force-pushed the omnibus-license-updates-july-21 branch from 86f8c47 to 32958ca Compare August 6, 2021 08:26
The Debian copyright tests expected data were just a list of items
without an attribute name telling what each item was.

This fixes this by making the expected YAML results a dictionary rather
than a list. It also adds the full match details to the expected results such that we can better assess what failed and how and can better
update the tests as we improve license detection.

Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
@pombredanne
pombredanne force-pushed the omnibus-license-updates-july-21 branch from 32958ca to 5ca96d1 Compare August 6, 2021 10:39
@pombredanne
pombredanne merged commit ef4232c into develop Aug 6, 2021
@pombredanne
pombredanne deleted the omnibus-license-updates-july-21 branch August 6, 2021 13:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant