Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
Summary: Review Request: navilu-fonts - Free Kannada Opentype sans-serif font
https://bugzilla.redhat.com/show_bug.cgi?id=771842
Summary: Review Request: navilu-fonts - Free Kannada Opentype sans-serif font Product: Fedora Version: 16 Platform: Unspecified OS/Version: Unspecified Status: NEW Severity: unspecified Priority: unspecified Component: Package Review AssignedTo: nobody@fedoraproject.org ReportedBy: psatpute@redhat.com QAContact: extras-qa@fedoraproject.org CC: notting@redhat.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: --- Type: --- Regression: --- Mount Type: --- Documentation: ---
SPEC Url: http://pravins.fedorapeople.org/navilu-fonts.spec SRPM Url: http://pravins.fedorapeople.org/navilu-fonts-1.1-1.fc16.src.rpm
Scratch build : http://koji.fedoraproject.org/koji/taskinfo?taskID=3621205
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=771842
Pravin Satpute psatpute@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Keywords| |i18n CC| |i18n-bugs@lists.fedoraproje | |ct.org, panemade@gmail.com, | |svenkate@redhat.com Flag| |fedora-review?
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=771842
Parag AN(पराग) panemade@gmail.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED AssignedTo|nobody@fedoraproject.org |panemade@gmail.com Flag|fedora-review? |fedora-review+
--- Comment #1 from Parag AN(पराग) panemade@gmail.com 2012-01-06 03:30:35 EST --- Review:
+ koji scratch build ->http://koji.fedoraproject.org/koji/taskinfo?taskID=3621205
+ rpmlint on rpms gave navilu-fonts.noarch: W: spelling-error Summary(en_US) opentype -> open type, open-type, phenotype navilu-fonts.noarch: W: spelling-error %description -l en_US opentype -> open type, open-type, phenotype navilu-fonts.src: W: spelling-error Summary(en_US) opentype -> open type, open-type, phenotype navilu-fonts.src: W: spelling-error %description -l en_US opentype -> open type, open-type, phenotype 2 packages and 0 specfiles checked; 0 errors, 4 warnings.
+ source verified with upstream as (sha1sum) eb71807bdfcc479ad5eac07d18da82100a2fd349 navilu-1.1.tar.gz eb71807bdfcc479ad5eac07d18da82100a2fd349 ../SOURCES/navilu-1.1.tar.gz
+ follows fonts packaging guidelines
Suggestions 1) you should drop the versioned dependency. Change BuildRequires: fontforge >= 20080429 to BuildRequires: fontforge
2)you don't need to clean buildroot now. remove following from spec BuildRequires: fontforge >= 20080429
APPROVED.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=771842
Pravin Satpute psatpute@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag| |fedora-cvs?
--- Comment #2 from Pravin Satpute psatpute@redhat.com 2012-01-09 02:31:32 EST --- Thank you Parag for review, i will do the suggested changes during git import.
New Package SCM Request ======================= Package Name: navilu-fonts Short Description: Free Kannada Opentype sans-serif font Owners: pravins Branches: f16 InitialCC: i18n-team
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=771842
--- Comment #3 from Jon Ciesla limburgher@gmail.com 2012-01-09 08:08:23 EST --- Git done (by process-git-requests).
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=771842
Pravin Satpute psatpute@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |CLOSED Resolution| |RAWHIDE Last Closed| |2012-01-10 02:13:11
--- Comment #4 from Pravin Satpute psatpute@redhat.com 2012-01-10 02:13:11 EST --- Thanks Jon for Git process. done built for rawhide and F16
package-review@lists.fedoraproject.org