https://bugzilla.redhat.com/show_bug.cgi?id=2427087
--- Comment #10 from Fabio Valentini <[email protected]> --- Package looks pretty good now. Only some very small things left: I. It would be great if you could a) use the "SourceLicense" tag to specify which license "phrog" itself is distributed under (right now the "License" tag only applies to the built package) and / or add a short comment to say "phrog license is FOO, licenses of statically linked Rust dependencies are thus: ...". II. The License tag looks correct, though it is not "minimal". In this case, the difference is negligible, so you could just leave it as-is (but in theory, you could drop the "(MIT OR Apache-2.0)" line because that's already covered by "(Apache-2.0 OR MIT)" - but I think this is even the only one that's "duplicated" here, you it wouldn't gain you much in terms of readability, and you would lose out on being able to mostly copy-paste from the %cargo_license_summary output). So I would say whether you want to "minimize" the License tag in this way is entirely up to you in this case. III. The .desktop file validation is missing from the latest version (expected - should be fixed either in the next version or before importing the package). IV. You're generating the "vendor manifest", but that file is not actually used anywhere. You'll need to add `%license cargo-vendor.txt` as a file to the %files list. The RPM generator for `bundled(crate(...))` generator is triggered by the presence (and contents) of this file. -- 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=2427087 Report this comment as SPAM: https://bugzilla.redhat.com/enter_bug.cgi?product=Bugzilla&format=report-spam&short_desc=Report%20of%20Bug%202427087%23c10 -- _______________________________________________ package-review mailing list -- [email protected] To unsubscribe send an email to [email protected] 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/[email protected] Do not reply to spam, report it: https://pagure.io/fedora-infrastructure/new_issue
