[Bug 1208842] Review Request: gdouros-symbola-fonts - A symbol font

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



https://bugzilla.redhat.com/show_bug.cgi?id=1208842

Parag AN(पराग) <panemade@xxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |panemade@xxxxxxxxx



--- Comment #1 from Parag AN(पराग) <panemade@xxxxxxxxx> ---
1) When I ran fedora-review on this package, I see
Diff spec file in url and in SRPM
---------------------------------
---
/home/parag/rev/1208842-gdouros-symbola-fonts/srpm/gdouros-symbola-fonts.spec  
    2015-04-07 06:15:33.505471601 +0530
+++
/home/parag/rev/1208842-gdouros-symbola-fonts/srpm-unpacked/gdouros-symbola-fonts.spec
     2015-04-02 16:15:34.000000000 +0530
@@ -68,6 +68,5 @@
 * Fri Mar 13 2015 Alexander Ploumistos <alexpl@xxxxxxxxxxxxxxxxx> - 7.21-2
 - Rebuilt for 20150308 internal update
-- Remove font with hinting as discussed on fedora-devel mailing list
-- Change license to Public Domain
+- Remove font with hinting as discussed on fedora-devel mailing list

 * Thu Mar 05 2015 Alexander Ploumistos <alexpl@xxxxxxxxxxxxxxxxx> - 7.21-1


Can you fix this and have the same spec packaged in srpm?

2) Also, its not good to package a single doc file separately. Add it in main
package only. Just add it next to %_font_pkg line.

3) Change other packages also if they have -doc subpackage then drop it.

4) You need sponsorship in packager group. Can you do some informal reviews on
other packages waiting for their package review?

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review





[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]