Fedora Merge Review: fonts-indic http://cvs.fedora.redhat.com/viewcvs/devel/fonts-indic/ Initial Owner: pnemade
Before any reviewer will pick this package, I made it as per FE guidelines and updated a new RPM of this package today. So i think i will need now a formal official review here :)
However, I will be happy to see some missing packaging improvement here.
Initial Owner: rbhalera
Created attachment 160096 [details] rpmlint silent SPEC file update cvs with this SPEC and build new package
Review: + package builds in mock (development i386). + rpmlint is silent for SRPM and for RPM. + source files match upstream url bb9497ee772062b97ff00a1a68b17c98 fonts-indic-2.1.5.tar.gz + package meets naming and packaging guidelines. + specfile is properly named, is cleanly written + Spec file is written in American English. + Spec file is legible. + dist tag is present. + build root is correct. + license is open source-compatible. + License text is included in package. + %doc is present. + BuildRequires are proper. + %clean is present. + package installed properly. + Macro use appears rather consistent. + Package contains content. + no headers or static libraries. + no .pc file present. + no -devel subpackage + no .la files. + no translations are available + Does owns the directories it creates. + fonts scriptlets present. + no duplicates in %files. + file permissions are appropriate. + Not a GUI App. APPROVED.