https://bugzilla.redhat.com/show_bug.cgi?id=827745 --- Comment #6 from Parag AN(पराग) <panemade@xxxxxxxxx> --- Review: + is ok - is Need work + koji scratch build used -> http://koji.fedoraproject.org/koji/taskinfo?taskID=4147740 - rpmlint output is silent for SRPM and for RPM junicode-fonts.src: W: spelling-error %description -l en_US monospace -> mono space, mono-space, aerospace junicode-fonts.noarch: W: spelling-error %description -l en_US monospace -> mono space, mono-space, aerospace junicode-fonts.noarch: W: incoherent-version-in-changelog 1.009-1 ['1.01-5.fc18', '1.01-5'] 2 packages and 0 specfiles checked; 0 errors, 3 warnings. + source files are actual upstream source url - package meets naming guidelines. + specfile is properly named. + Spec file is written in American English. + dist tag is present. + license is open source-compatible. + License text is NOT included in package but is present in font file itself. + %doc is present. + BuildRequires are proper. - Buildroot is present. - %clean is present. + Macro use appears rather consistent. + Package contains content and not code. + no headers or static libraries. + no .pc file present. + no -devel subpackage + no .la files. + translations are not available + Does owns the directories it creates. + no scriptlets present. + no duplicates in %files. + file permissions are appropriate. + Not a GUI application suggestions: 1)If you only want to add this package in Fedora then you can remove the following from spec file a) remove the buildroot tag b) removal of buildroot in %install (i.e. rm -fr %{buildroot} ) c) %clean section d) %defattr(-,root,root,-) 2) version field is wrong. It should be 0.7.6 version. check the version by opening the sfd or ttf file in fontforge application. There click on Elements-> Font Info. Check "PS Names" and to the right of that version of the font. You included all the three variants of junicode font as a source0, source1 and source2 files but this is not correct. When upstream will update any single font then how are you going to update this package. You are using cvs version of font files. Please see https://fedoraproject.org/wiki/Packaging:NamingGuidelines#Snapshot_packages You need to use 20120612cvs%{?dist} as a release tag. Then you need to add comments in spec file just before Source0: that will tell how you created tarball from upstream cvs checkout. you can write following in spec # Following tarball is created using upstream checkout as # cvs -z3 -d:pserver:anonymous@xxxxxxxxxxxxxxxxxxxxxxxxxxxx:/cvsroot/junicode co -P junicode # cd junicode # make dist You need to use this created tarball as a source0 and create a new srpm. Remove all other sources. So you should be including single tarball file instead to add individual font files. 3) the comments which are irrelevant should be removed like in this package #Obsoletes: junicode-fonts < 1.009-3 4) Use %description from upstream website as A font family especially for medieval scholars, but containing an extensive enough selection of Unicode characters to be widely useful. 5) Fonts packaging guidelines also suggests to add font information by creating page on wiki See http://fedoraproject.org/wiki/Packaging:FontsPolicy -- You are receiving this mail because: You are on the CC list for the bug. _______________________________________________ package-review mailing list package-review@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/package-review