[Bug 1228865] Review Request: gdouros-anaktoria-fonts - A font based on "Grecs du roi" and the "First Folio Edition of Shakespeare"

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

 



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

Zbigniew Jędrzejewski-Szmek <zbyszek@xxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
           Assignee|nobody@xxxxxxxxxxxxxxxxx    |zbyszek@xxxxxxxxx
              Flags|                            |fedora-review?



--- Comment #5 from Zbigniew Jędrzejewski-Szmek <zbyszek@xxxxxxxxx> ---
OK. I guess that's fine: the font is open source and either the author will
re-post it or it will pop up somewhere else. Where is the license specified?

%description seems to contain a private use unicode character (1480󿀄
1561).

[ ]: Each %files section contains %defattr if rpm < 4.4
     Note: %defattr present but not needed

[ ]: Large documentation must go in a -doc subpackage. Large could be size
     (~1MB) or number of files.
     Note: Documentation size is 808960 bytes in 1 files.
That's borderline. A bit too small to create a separate package.

Appdata file should be validated in %check
[https://fedoraproject.org/wiki/Packaging:AppData].

$ appstream-util validate-relax
/usr/share/appdata/gdouros-anaktoria.metainfo.xml
/usr/share/appdata/gdouros-anaktoria.metainfo.xml: FAILED:
• markup-invalid        : <id> does not have correct extension for kind
• tag-missing           : <extends> is not present
Validation of files failed

-- 
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]