Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: gfs-porson-fonts - GFS Porson Greek font https://bugzilla.redhat.com/show_bug.cgi?id=398701 ------- Additional Comments From nicolas.mailhot@xxxxxxxxxxx 2007-11-26 08:14 EST ------- (In reply to comment #1) > looks correct SRPM is > http://nim.fedorapeople.org/gfs-porson-fonts-20060908-2.fc9.src.rpm Yes I fixed a small problem after filing the review request > rpmlint gave me > gfs-porson-fonts.noarch: I: checking > gfs-porson-fonts.noarch: E: description-line-too-long In England, during the > 1790’s, Cambridge University Press decided to procure a > Your description lines must not exceed 79 characters. If a line is exceeding > this number, cut it to fit in two lines. The lines have been resized using the standard "fold" tool. So they are all 80 column max. rpmlint targets a non-standard terminal width, 79 column and I don't believe this width is actually in use anywhere or that any terminal user will actually be inconvenienced if we use the full 80 column. If you really insist I may resize the text but that seems a waste of time to me. > gfs-porson-fonts.noarch: W: no-version-in-last-changelog > The last changelog entry doesn't contain a version. Please insert the > version that is coherent with the version of the package and rebuild it. rpmlint wants to force you to put the version at the end of the first changelog line, however the official approved Fedora changelog format list includes a style where the version is not on this line (http://fedoraproject.org/wiki/Packaging/Guidelines#Changelogs, 3rd style) so rpmlint's warning is bogus -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug, or are watching someone who is. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review