[Bug 963074] Review Request: librime - Rime Input Method Engine

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

 



Product: Fedora
https://bugzilla.redhat.com/show_bug.cgi?id=963074

--- Comment #1 from Parag AN(पराग) <panemade@xxxxxxxxx> ---
Review:-

+ Koji scratch build f20 ->
http://koji.fedoraproject.org/koji/taskinfo?taskID=5383022

+ rpmlint on rpms gave
librime.src: W: invalid-url Source0:
http://rimeime.googlecode.com/files/librime-0.9.8.tar.gz HTTP Error 404: Not
Found
librime.x86_64: W: no-manual-page-for-binary rime_dict_manager
librime.x86_64: W: no-manual-page-for-binary rime_deployer
librime-devel.x86_64: W: no-documentation
4 packages and 0 specfiles checked; 0 errors, 4 warnings.
==> Source is downloadable so no issues.

+ Source verified with upstream as
http://rimeime.googlecode.com/files/librime-0.9.8.tar.gz :
  CHECKSUM(SHA256) this package     :
94092a7d1990b949efe6b4bcf8e8351cee64a3abe7838770a775ecb02a1ef065
  CHECKSUM(SHA256) upstream package :
94092a7d1990b949efe6b4bcf8e8351cee64a3abe7838770a775ecb02a1ef065


+ License is valid and its text is include in LICENSE file

Suggestions:-
1) Add -tools subpackage which will install rime_deployer and 
rime_dict_manager and let it 
Requires:       %{name}%{?_isa} = %{version}-%{release}


2) package need to own directory %{_datadir}/cmake/rime

3) I don't think you need following for this package
BuildRequires:  xorg-x11-proto-devel, zlib-devel

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=h3mp85u42J&a=cc_unsubscribe
_______________________________________________
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]