Bug 1031569

Summary: Review Request: cdac-sakal-marathi-fonts - Marathi language font from CDAC
Product: [Fedora] Fedora Reporter: Pravin Satpute <psatpute>
Component: Package ReviewAssignee: Christopher Meng <i>
Status: CLOSED ERRATA QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: rawhideCC: i, package-review, panemade, psatpute, sochotni
Target Milestone: ---Flags: i: fedora-review+
gwync: fedora-cvs+
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: cdac-sakal-marathi-fonts-9.21-3.fc20 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2014-02-08 05:05:43 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Comment 1 Stanislav Ochotnicky 2013-11-18 11:22:35 UTC
Next time please use review template[1] for creating package review bugs. For now you should add missing information

[1] https://bugzilla.redhat.com/enter_bug.cgi?product=Fedora&format=fedora-review

Comment 2 Christopher Meng 2013-11-18 11:27:05 UTC
%build
echo "Nothing to do in Build."

???

This is fantastic but I think you don't have to echo it, just leaving comments there is enough.

Comment 3 Pravin Satpute 2013-11-18 13:33:16 UTC
(In reply to Stanislav Ochotnicky from comment #1)
> Next time please use review template[1] for creating package review bugs.
> For now you should add missing information
> 
> [1]
> https://bugzilla.redhat.com/enter_bug.cgi?product=Fedora&format=fedora-review

Spec URL: http://pravins.fedorapeople.org/cdac-sakal-marathi-fonts.spec
SRPM URL: http://pravins.fedorapeople.org/cdac-sakal-marathi-fonts-9.21-1.fc19.src.rpm
Description: Open type font for Devanagari script for Marathi language released by CDAC.
Fedora Account System Username: pravins

Comment 4 Pravin Satpute 2013-11-18 13:35:03 UTC
(In reply to Christopher Meng from comment #2)
> %build
> echo "Nothing to do in Build."
> 
> ???
> 
> This is fantastic but I think you don't have to echo it, just leaving
> comments there is enough.

Sure, will do it.

Comment 5 Pravin Satpute 2013-11-20 05:49:00 UTC
Updated specs and srpm is as followed:

Spec URL: http://pravins.fedorapeople.org/cdac-sakal-marathi-fonts.spec
SRPM URL: http://pravins.fedorapeople.org/cdac-sakal-marathi-fonts-9.21-2.fc19.src.rpm

Comment 6 Christopher Meng 2013-11-28 11:14:15 UTC
PACKAGE APPROVED.

Comment 7 Parag AN(पराग) 2013-11-29 07:06:26 UTC
Ah! this is already approved.

Pravin should be happy now :)

but my suggestions are create a font page on fedora wiki and remove Group tag and also mention http://sakalmarathi.cdac.in/fr/licences.php in comment for LICENSE file in spec.

Then this package is ready to go in Fedora :)

Comment 8 Pravin Satpute 2013-12-02 07:02:06 UTC
New Package SCM Request
=======================
Package Name:  cdac-sakal-marathi-fonts
Short Description: Marathi language font from CDAC
Owners: pravins
Branches: f20
InitialCC: i18n-team fonts-sig

Comment 9 Gwyn Ciesla 2013-12-02 13:17:40 UTC
Git done (by process-git-requests).

Comment 10 Fedora Update System 2013-12-03 04:52:08 UTC
cdac-sakal-marathi-fonts-9.21-3.fc20 has been submitted as an update for Fedora 20.
https://admin.fedoraproject.org/updates/cdac-sakal-marathi-fonts-9.21-3.fc20

Comment 11 Fedora Update System 2013-12-03 18:19:54 UTC
cdac-sakal-marathi-fonts-9.21-3.fc20 has been pushed to the Fedora 20 testing repository.

Comment 12 Christopher Meng 2013-12-23 15:54:55 UTC
I think that you can push it to stable now.

Comment 13 Pravin Satpute 2014-02-07 09:54:21 UTC
Somehow i missed that email :(
Thanks Christopher for ping.

Comment 14 Fedora Update System 2014-02-08 05:05:43 UTC
cdac-sakal-marathi-fonts-9.21-3.fc20 has been pushed to the Fedora 20 stable repository.

Comment 15 Pravin Satpute 2014-09-09 08:16:45 UTC
Package Change Request
======================
Package Name: cdac-sakal-marathi-fonts
New Branches: el5 el6 epel7
Owners: pravins

Comment 16 Gwyn Ciesla 2014-09-09 11:48:08 UTC
Git done (by process-git-requests).