https://bugzilla.redhat.com/show_bug.cgi?id=1981048 Petr Menšík <pemensik@xxxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |pemensik@xxxxxxxxxx --- Comment #2 from Petr Menšík <pemensik@xxxxxxxxxx> --- First of all, I think somewhere it should explain what is this package for and what it does. For me, linhpsdr sounds almost like someone's cat walked over a keyboard and typed just random stuff. Please explain something about the package. I think it should be clear to anyone interested whether he needs the package and what it might do. I understand this it for tech-friendly people, but even upstream does not care to explain what the project does and how it is used (in README). I think that should be improved both in package and upstream. I think I am not the only one, who does not have any idea what might HPSDR stand for. Well, when this is linux application for that, it is not really helping. I expect it would not violate any trademark, if the package spends few more words to at least say: I am package for working with Software Defined Radio on linux. Just including abbreviation is not sufficient without any kind of category description. I think desktop-file-validate should belong to %check section Otherwise the package seems reasonable. -- 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. _______________________________________________ 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 on the list, report it: https://pagure.io/fedora-infrastructure