Fedora Merge Review: words http://cvs.fedora.redhat.com/viewcvs/devel/words/ Initial Owner: kzak
Taking for review.
Partial review: GOOD ==== MUST: source matches upstream (md5sum checked) BAD === MUST: rpmlint output SRPM: W: words summary-ended-with-dot A dictionary of English words for the /usr/share/dict directory. E: words tag-not-utf8 %changelog W: words invalid-license public domain E: words non-utf8-spec-file words.spec E: words no-cleaning-of-buildroot %install All should be fixed: - Change License to "Public Domain" - Remove dot at the end of summart - change SPEC file to UTF-8 - clean the buildroot in %install using rm -rf $RPM_BUILD_ROOT For the RPM, the errors are a subset of above. * Version field is quite arbitrary (3.0) instead of matching upstream. * Separate license file into a different "Source" file. Also explain in the file that the license info in the file readme.txt is deprecated. * Change /usr/share in %files section to %{_datadir} * Change %defattr(-,root,root) to %defattr(-,root,root,-)
> * Version field is quite arbitrary (3.0) instead of matching upstream. The upstream doesn't use any versioning. > * Separate license file into a different "Source" file. Also explain in the > file that the license info in the file readme.txt is deprecated. Good point. Thanks. Fixed. Please, review the package (words-3.0-12.fc7).
As Roozbeh did not continue the review for more than 1.5 years now, I'm now and hereby just silently overtaking the review. Re-assinging the bug report to me first, as it is usual for a Package Review. So rpmlint is completely silent, good job until now. Rest of the package is sane according to my knowledge of being a packager. Some suggestions, I really would like to see, but no must: - Remove "BuildRequires: coreutils", something else anyway pulls that in - Replace "Source: mwords.tar.Z" by "Source: http://web.archive.org/web/20060527013227/http://www.dcs.shef.ac.uk/research/ilash/Moby/mwords.tar.Z" as it is a valid source URL and everybody can verify it Karel, olease close the bug report once you've applied the changes or let me know why you won't do this and close this bug report therewith. APPROVED.
Karel, ping?
(In reply to comment #4) > Some suggestions, I really would like to see, but no must: > - Remove "BuildRequires: coreutils", something else anyway pulls that in Fixed. > - Replace "Source: mwords.tar.Z" by "Source: > http://web.archive.org/web/20060527013227/http://www.dcs.shef.ac.uk/research/ilash/Moby/mwords.tar.Z" > as it is a valid source URL and everybody can verify it OK, good idea. Fixed. > Karel, olease close the bug report once you've applied the changes or let me > know why you won't do this and close this bug report therewith. Closing...