https://bugzilla.redhat.com/show_bug.cgi?id=2081964 Troy Dawson <tdawson@xxxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |tdawson@xxxxxxxxxx --- Comment #2 from Troy Dawson <tdawson@xxxxxxxxxx> --- Despite what the plasmatube home page says, there are three licenses in the code. They are all found in the LICENSES directory. CC0-1.0.txt CC-BY-SA-4.0.txt GPL-3.0-or-later.txt Please add the licenses to %files using the %license tag Also, please update the Licenses: section to add the other two licenses. Another problem is that the Summary is too long. Please put what is in the Summary into the Description and pull one of the short descriptions from the code page. I like "YouTube video player based on QtMultimedia and youtube-dl" There is also locale issues when I run a review, but I'm not sure what to do with those since that is all scripted. -- 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=2081964 _______________________________________________ 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
