https://bugzilla.redhat.com/show_bug.cgi?id=2279853 Fabio Valentini <decathorpe@xxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- Depends On|182235 (FE-Legal) | Status|NEW |ASSIGNED Assignee|nobody@xxxxxxxxxxxxxxxxx |decathorpe@xxxxxxxxx --- Comment #6 from Fabio Valentini <decathorpe@xxxxxxxxx> --- > And just to understand this better, `tables.rs` should be under `Unicode-DFS-2016` license because it used UCD files to generate it? This is exactly it, if I understand correctly: Code generated from Unicode data files is covered by the Unicode-DFS-2016 license. So the resulting license expression should be `(MIT OR Apache-2.0) AND Unicode-DFS_2016`. See also the unicode-ident crate for a very similar case. Until upstream implements this change, you can patch package.license to fix this downstream for now. This is also what we do for crates where the developers disagree with this assessment and / or disagree about whether the Unicode license needs to be represented in Cargo.toml (like bstr: https://src.fedoraproject.org/rpms/rust-bstr/blob/rawhide/f/bstr-fix-metadata.diff, or regex-syntax: https://src.fedoraproject.org/rpms/rust-regex-syntax/blob/rawhide/f/regex-syntax-fix-metadata.diff). Side note: The same applies to the Rust standard library as well (which is why bstr and regex doesn't do it either), but Rust Foundation lawyers seem to disagree with Red Hat lawyers here about whether `AND Unicode-DFS-2016` needs to be added to the license expression in Cargo.toml. Since we need to placate Red Hat lawyers here, we need to apply their standard, even if the upstream projects disagree :) Other than that, the package looks good to me. Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=182235 [Bug 182235] Fedora Legal Tracker -- You are receiving this mail because: You are always notified about changes to this product and component You are on the CC list for the bug. https://bugzilla.redhat.com/show_bug.cgi?id=2279853 Report this comment as SPAM: https://bugzilla.redhat.com/enter_bug.cgi?product=Bugzilla&format=report-spam&short_desc=Report%20of%20Bug%202279853%23c6 -- _______________________________________________ package-review mailing list -- package-review@xxxxxxxxxxxxxxxxxxxxxxx To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx Fedora Code of Conduct: https://docs.fedoraproject.org/en-US/project/code-of-conduct/ List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines List Archives: https://lists.fedoraproject.org/archives/list/package-review@xxxxxxxxxxxxxxxxxxxxxxx Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
