https://bugzilla.redhat.com/show_bug.cgi?id=1758487
--- Comment #2 from Kentaro Hayashi <[email protected]> --- Thank you for reviewing. I've updated spec and SRPM Spec URL: https://packages.groonga.org/tmp/fasttext/fasttext.spec SRPM URL: https://packages.groonga.org/tmp/fasttext/fasttext-0.9.1-1.fc30.src.rpm Here is the summary for updates. * Removed unused Group * Removed unused ldconfig scriptlet * Removed needless %defattr, %clean * Updated Source0: URL correctly * Added %license tag * Added missing BR to gcc-c++ * Added comments for patches * Fixed to unintentional soname bump (less globbing in %files) * Fixed %description, add missing . in end of sentences * Fixed to use %autosetup instead of series of %patch * Added a patch to enable hardening with -fPIC -- 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 -- [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]
