Bug 1000583 - Review Request: fcitx-anthy - Anthy Engine for Fcitx
Summary: Review Request: fcitx-anthy - Anthy Engine for Fcitx
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Christopher Meng
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
: 887907 (view as bug list)
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2013-08-23 18:23 UTC by Robin Lee
Modified: 2013-09-23 00:24 UTC (History)
3 users (show)

Fixed In Version: fcitx-anthy-0.2.0-2.fc18
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2013-09-23 00:24:17 UTC
Type: ---
Embargoed:
i: fedora-review+
gwync: fedora-cvs+


Attachments (Terms of Use)

Description Robin Lee 2013-08-23 18:23:23 UTC
Spec URL: http://cheeselee.fedorapeople.org/fcitx-anthy.spec
SRPM URL: http://cheeselee.fedorapeople.org/fcitx-anthy-0.2.0-1.fc19.src.rpm
Description: Fcitx-anthy is an Anthy engine wrapper for Fcitx. It provides a Japanese input method.
Fedora Account System Username: cheeselee

Comment 1 Robin Lee 2013-08-23 18:25:12 UTC
*** Bug 887907 has been marked as a duplicate of this bug. ***

Comment 2 Christopher Meng 2013-09-05 14:03:59 UTC
Directories without known owners:

%{_datadir}/fcitx/anthy/*.sty

--->

%{_datadir}/fcitx/anthy


===========

Suggestion:

Name:			fcitx-anthy
Version:		0.2.0
Release:		1%{?dist}
Summary:		Anthy Engine for Fcitx
Group:			System Environment/Libraries
License:		GPLv2+
URL:			https://fcitx-im.org/wiki/Anthy
Source0:		http://download.fcitx-im.org/fcitx-anthy/%{name}-%{version}.tar.xz
BuildRequires:	cmake
BuildRequires:	fcitx-devel
BuildRequires:	gettext
BuildRequires:	intltool
BuildRequires:	anthy-devel
Requires:		fcitx

Auh, a bit ugly...

Name:		fcitx-anthy
Version:	0.2.0
Release:	1%{?dist}
Summary:	Anthy Engine for Fcitx
Group:		System Environment/Libraries
License:	GPLv2+
URL:		https://fcitx-im.org/wiki/Anthy
Source0:	http://download.fcitx-im.org/fcitx-anthy/%{name}-%{version}.tar.xz
BuildRequires:	anthy-devel
BuildRequires:	cmake
BuildRequires:	fcitx-devel
BuildRequires:	gettext
BuildRequires:	intltool
Requires:	fcitx

Also, %description is too short.

Orig:

Fcitx-anthy is an Anthy engine wrapper for Fcitx. It
provides a Japanese input method.


Mine:

Fcitx-anthy is an Anthy engine wrapper for Fcitx based on scim-anthy. It provides a Japanese input method. Users can input Japanese hiragana and katakana by inputting romaji via fcitx-anthy. Fcitx-anthy can also convert hiragana or katakana to Japanese kanji after inputting hiragana or katakana. 

*************************************

Please FIX the issue before SCM.

APPROVED.

Comment 3 Robin Lee 2013-09-10 03:00:12 UTC
Spec URL: http://cheeselee.fedorapeople.org/fcitx-anthy.spec
SRPM URL: http://cheeselee.fedorapeople.org/fcitx-anthy-0.2.0-2.fc19.src.rpm

Changes:
- Own the diretory %%{_datadir}/fcitx/anthy/
- description enlarged


(In reply to Christopher Meng from comment #2)
> Directories without known owners:
> 
> %{_datadir}/fcitx/anthy/*.sty
> 
> --->
> 
> %{_datadir}/fcitx/anthy
> 
> 
> ===========
> 
> Suggestion:
> 
> Name:			fcitx-anthy
> Version:		0.2.0
> Release:		1%{?dist}
> Summary:		Anthy Engine for Fcitx
> Group:			System Environment/Libraries
> License:		GPLv2+
> URL:			https://fcitx-im.org/wiki/Anthy
> Source0:		http://download.fcitx-im.org/fcitx-anthy/%{name}-%{version}.tar.xz
> BuildRequires:	cmake
> BuildRequires:	fcitx-devel
> BuildRequires:	gettext
> BuildRequires:	intltool
> BuildRequires:	anthy-devel
> Requires:		fcitx
> 
> Auh, a bit ugly...
> 
> Name:		fcitx-anthy
> Version:	0.2.0
> Release:	1%{?dist}
> Summary:	Anthy Engine for Fcitx
> Group:		System Environment/Libraries
> License:	GPLv2+
> URL:		https://fcitx-im.org/wiki/Anthy
> Source0:	http://download.fcitx-im.org/fcitx-anthy/%{name}-%{version}.tar.xz
> BuildRequires:	anthy-devel
> BuildRequires:	cmake
> BuildRequires:	fcitx-devel
> BuildRequires:	gettext
> BuildRequires:	intltool
> Requires:	fcitx
The specfile is aligned with tabs. If you set tab as 4 spaces in you editor, you will get a cleanly aligned view.

> 
> Also, %description is too short.
> 
> Orig:
> 
> Fcitx-anthy is an Anthy engine wrapper for Fcitx. It
> provides a Japanese input method.
> 
> 
> Mine:
> 
> Fcitx-anthy is an Anthy engine wrapper for Fcitx based on scim-anthy. It
> provides a Japanese input method. Users can input Japanese hiragana and
> katakana by inputting romaji via fcitx-anthy. Fcitx-anthy can also convert
> hiragana or katakana to Japanese kanji after inputting hiragana or katakana.
OK. I have revised it with respect to your suggestion.
 
> 
> *************************************
> 
> Please FIX the issue before SCM.
> 
> APPROVED.

Comment 4 Robin Lee 2013-09-10 03:04:23 UTC
Thanks for your review!

New Package SCM Request
=======================
Package Name: fcitx-anthy
Short Description: Anthy Engine for Fcitx
Owners: cheeselee
Branches: f18 f19 f20
InitialCC: i18n-team

Comment 5 Gwyn Ciesla 2013-09-10 11:54:59 UTC
Git done (by process-git-requests).

Comment 6 Fedora Update System 2013-09-11 02:30:31 UTC
fcitx-anthy-0.2.0-2.fc18 has been submitted as an update for Fedora 18.
https://admin.fedoraproject.org/updates/fcitx-anthy-0.2.0-2.fc18

Comment 7 Fedora Update System 2013-09-12 02:01:54 UTC
fcitx-anthy-0.2.0-2.fc18 has been pushed to the Fedora 18 testing repository.

Comment 8 Fedora Update System 2013-09-23 00:24:17 UTC
fcitx-anthy-0.2.0-2.fc18 has been pushed to the Fedora 18 stable repository.


Note You need to log in before you can comment on or make changes to this bug.