[Bug 693950] Review Request: yagf - Graphical front-end for cuneiform

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

 



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=693950

--- Comment #15 from Dmitrij S. Kryzhevich <krege@xxxxxxx> 2011-08-14 23:21:16 EDT ---
(In reply to comment #14)
> http://fedoraproject.org/wiki/Packaging:ScriptletSnippets#Icon_Cache is the
> wrong scriptlet to run, as you don't install files "into one of the
> subdirectories in %{_datadir}/icons/ ..."

It does install. /usr/share/icons/hicolor/96x96/apps/yagf.png.

> Please drop the version constraint for qt4-devel. There is no version that old
> in Fedora.

OK.


> Instead, your spec file meets the requirements for
> http://fedoraproject.org/wiki/Packaging:ScriptletSnippets#desktop-database
> 
> "Comment[pl]=Grafinė teksto atpažinimo programos Cuneiform sąsaja" -- That line
> in the desktop file is wrong as well. Just delete it.

Missed that. Fixed as well as Name section.

> (line-break doesn't spell line-brake)
+
> (You can cut out "on the Linux platform" from the description.)
+
> (I also suggest to put each BuildRequirement on a separate line, as it is
> easier to overlook.)
+

Spec URL: http://krege.fedorapeople.org/yagf/yagf.spec
SRPM URL: http://krege.fedorapeople.org/yagf/yagf-0.8.6-6.fc15.src.rpm

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
_______________________________________________
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]