Bug 963078

Summary: Review Request: ibus-rime - Rime Input Method Engine for IBus
Product: [Fedora] Fedora Reporter: Peng Wu <pwu>
Component: Package ReviewAssignee: Parag AN(पराग) <panemade>
Status: CLOSED ERRATA QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: i18n-bugs, notting, package-review
Target Milestone: ---Flags: panemade: fedora-review+
gwync: fedora-cvs+
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: ibus-rime-0.9.9-2.fc19 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2013-05-28 00:59:26 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On: 963074, 963077    
Bug Blocks:    

Description Peng Wu 2013-05-15 06:08:54 UTC
Spec URL: http://pwu.fedorapeople.org/rime/ibus-rime.spec
SRPM URL: http://pwu.fedorapeople.org/rime/ibus-rime-0.9.9-1.fc18.src.rpm
Description: ibus-rime: Rime Input Method Engine for Linux/IBus
Fedora Account System Username:pwu

Comment 1 Jens Petersen 2013-05-21 08:32:05 UTC
Please mention in the summary/description what this engine is for.

The current 
---
Summary:        Rime Input Method Engine for Linux/IBus
%description
ibus-rime: Rime Input Method Engine for Linux/IBus
----
does not really explain what it is...

I would be good to include zh translations of the Summary and Description too.

https://fedoraproject.org/wiki/Packaging:Guidelines?rd=Packaging/Guidelines#summary

Comment 2 Parag AN(पराग) 2013-05-21 16:47:13 UTC
Review:

+ mock build succeeded for rawhide

+ rpmlint on rpms gave
ibus-rime.src: W: invalid-url Source0: http://rimeime.googlecode.com/files/ibus-rime-0.9.9.tar.gz HTTP Error 404: Not Found
3 packages and 0 specfiles checked; 0 errors, 1 warnings.

+ Source verified with upstream as 
http://rimeime.googlecode.com/files/ibus-rime-0.9.9.tar.gz :
  CHECKSUM(SHA256) this package     : 5bbb76abc763bdd3c83a5fe594f1488b34377b34b3f6e3c8c7b5cf86ab550cad
  CHECKSUM(SHA256) upstream package : 5bbb76abc763bdd3c83a5fe594f1488b34377b34b3f6e3c8c7b5cf86ab550cad

suggestions:
1) remove unnecessary comments from spec

2) fix the issue raised in comment#1

3) Add ChangeLog to %doc

Comment 3 Peng Wu 2013-05-22 05:41:44 UTC
Fixed, please review it again, thanks.
Spec URL: http://pwu.fedorapeople.org/rime/ibus-rime.spec
SRPM URL: http://pwu.fedorapeople.org/rime/ibus-rime-0.9.9-2.fc18.src.rpm

Comment 4 Parag AN(पराग) 2013-05-23 04:06:00 UTC
only thing remaining is to own the directory /usr/share/ibus-rime/ and /usr/share/ibus-rime/icons

you can simply write in %files which owns both directories
%{_datadir}/ibus-rime/

APPROVED.

Comment 5 Peng Wu 2013-05-23 06:00:17 UTC
Okay, I will fix it when importing.

Comment 6 Peng Wu 2013-05-23 06:04:29 UTC
New Package SCM Request
=======================
Package Name: ibus-rime
Short Description: Rime Input Method Engine for IBus
Owners: pwu
Branches: f18 f19
InitialCC: i18n-team petersen

Comment 7 Gwyn Ciesla 2013-05-23 12:57:52 UTC
Git done (by process-git-requests).

Comment 8 Fedora Update System 2013-05-25 01:49:01 UTC
ibus-rime-0.9.9-2.fc18,brise-0.22-2.fc18,librime-0.9.8-2.fc18 has been submitted as an update for Fedora 18.
https://admin.fedoraproject.org/updates/ibus-rime-0.9.9-2.fc18,brise-0.22-2.fc18,librime-0.9.8-2.fc18

Comment 9 Fedora Update System 2013-05-25 01:49:21 UTC
ibus-rime-0.9.9-2.fc19,brise-0.22-2.fc19,librime-0.9.8-2.fc19 has been submitted as an update for Fedora 19.
https://admin.fedoraproject.org/updates/ibus-rime-0.9.9-2.fc19,brise-0.22-2.fc19,librime-0.9.8-2.fc19

Comment 10 Fedora Update System 2013-05-25 19:02:42 UTC
ibus-rime-0.9.9-2.fc19, brise-0.22-2.fc19, librime-0.9.8-2.fc19 has been pushed to the Fedora 19 testing repository.

Comment 11 Fedora Update System 2013-05-28 00:59:26 UTC
ibus-rime-0.9.9-2.fc18, brise-0.22-2.fc18, librime-0.9.8-2.fc18 has been pushed to the Fedora 18 stable repository.

Comment 12 Fedora Update System 2013-05-28 02:16:38 UTC
ibus-rime-0.9.9-2.fc19, brise-0.22-2.fc19, librime-0.9.8-2.fc19 has been pushed to the Fedora 19 stable repository.