https://bugzilla.redhat.com/show_bug.cgi?id=1765730 --- Comment #2 from Jerry James <loganjerry@xxxxxxxxx> --- (In reply to Ankur Sinha (FranciscoD) from comment #1) > - I've not been able to find any source files that say GPLv2+. The GPLv2 > license file is included. Could you double check where they say GPLv2 "or > newer" just to be sure that the "+" is needed please? Oh, sorry, I forgot to mention that that was resolved in git after the 2.6.1 release. This is the justification: https://github.com/gap-packages/grpconst/commit/b3224915ab870d5ac94ffe61cdfb5030d7931357 Hopefully upstream releases a new version at some point that includes that commit. > - incorrect-fsf-address /usr/share/licenses/gap-pkg-grpconst/LICENSE: Please > let upstream know about this. Will do. > - Do the Requires need to be versioned? The package's web page says: > * autpgrp >=1.6 > * irredsol >=1.2 > > Versioning them may help catch issues later when individual packages are > updated. True. I didn't add the versions because the very first version of autpgrp built for Fedora was 1.6, and the very first version of irredsol built for Fedora was 1.3.1, so it is not possible to get older versions from Fedora repositories, not even for EOL Fedora releases. If those values are updated in a future release, I will add them to the Requires. Thanks for the review! -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component _______________________________________________ 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
