https://bugzilla.redhat.com/show_bug.cgi?id=2189985 Fabio Valentini <decathorpe@xxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|nobody@xxxxxxxxxxxxxxxxx |decathorpe@xxxxxxxxx Doc Type|--- |If docs needed, set a value Flags| |fedora-review? CC| |decathorpe@xxxxxxxxx Status|NEW |ASSIGNED --- Comment #2 from Fabio Valentini <decathorpe@xxxxxxxxx> --- The license information from the %cargo_license_summary should be in the "-n %{crate}" subpackage, with a separate License tag. For Rust packages, the License tag of the main package usually reflects the license of the crate. It doesn't make much of a difference for this package (since it only provides a binary but no library interface), but it will make it easier to re-apply these changes for version updates (and it's also in line with other Rust packages). You also shouldn't "simplify" the license that much anymore ... according to latest guidance from Red Hat legal, these licenses: # (MIT OR Apache-2.0) AND Unicode-DFS-2016 # Apache-2.0 OR MIT # Apache-2.0 WITH LLVM-exception OR Apache-2.0 OR MIT # MIT # MIT OR Apache-2.0 # Unlicense OR MIT Result in this license tag: License: MIT AND Unicode-DFS-2016 AND (Apache-2.0 OR MIT) AND (Apache-2.0 WITH LLVM-exception OR Apache-2.0 OR MIT) AND (Unlicense OR MIT) (i.e. the only thing you can de-duplicate is reordered (A OR B) / (B OR A), and I tend to sort the rest of the AND clauses alphabetically for reproducibility) -- 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=2189985 _______________________________________________ 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
