Bug 906248 (libkkc) - Review Request: libkkc - Japanese Kana Kanji conversion library
Summary: Review Request: libkkc - Japanese Kana Kanji conversion library
Keywords:
Status: CLOSED RAWHIDE
Alias: libkkc
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Parag AN(पराग)
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On: 906245
Blocks: ibus-kkc
TreeView+ depends on / blocked
 
Reported: 2013-01-31 10:01 UTC by Daiki Ueno
Modified: 2013-02-23 01:04 UTC (History)
4 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2013-02-11 23:36:05 UTC
Type: Bug
Embargoed:
panemade: fedora-review+
gwync: fedora-cvs+


Attachments (Terms of Use)
mock build log (764.79 KB, text/plain)
2013-02-03 12:45 UTC, Mamoru TASAKA
no flags Details

Description Daiki Ueno 2013-01-31 10:01:11 UTC
Spec URL: http://ueno.fedorapeople.org/libkkc/libkkc.spec
SRPM URL: http://ueno.fedorapeople.org/libkkc/libkkc-0.1.0-1.fc18.src.rpm
Description:
libkkc provides a converter from Kana-string to
Kana-Kanji-mixed-string.  It was named after kkc.el in GNU Emacs, a
simple Kana Kanji converter, while libkkc tries to convert sentences
in a bit more complex way using N-gram language models.

Comment 1 Mamoru TASAKA 2013-02-03 12:45:05 UTC
Created attachment 692302 [details]
mock build log

Just tried mockbuild.
Please make build log more verbose so that we can check if Fedora specific compilation flags are honored correctly.

Comment 2 Mamoru TASAKA 2013-02-03 14:26:38 UTC
Note that for this package I just tried mockbuild.

Comment 3 Daiki Ueno 2013-02-04 09:58:03 UTC
Updated the latest release and disabled silent rules.

Spec URL: http://ueno.fedorapeople.org/libkkc/libkkc.spec
SRPM URL: http://ueno.fedorapeople.org/libkkc/libkkc-0.1.1-1.fc18.src.rpm

Comment 4 Daiki Ueno 2013-02-05 09:42:42 UTC
Fixed some issues reported by Parag off-line.

Spec URL: http://ueno.fedorapeople.org/libkkc/libkkc.spec
SRPM URL: http://ueno.fedorapeople.org/libkkc/libkkc-0.1.2-1.fc18.src.rpm

Comment 5 Parag AN(पराग) 2013-02-05 13:10:07 UTC
Review:-
+ Koji scratch build -> http://koji.fedoraproject.org/koji/taskinfo?taskID=4929607

+ rpmlint on rpms gave
libkkc.src: W: spelling-error %description -l en_US kkc -> kc, k kc, KC
libkkc.src: W: spelling-error %description -l en_US el -> e, l, eel
libkkc.x86_64: W: spelling-error %description -l en_US kkc -> kc, k kc, KC
libkkc.x86_64: W: spelling-error %description -l en_US el -> e, l, eel
libkkc-data.x86_64: W: no-documentation
libkkc-devel.x86_64: W: no-documentation
libkkc-tools.x86_64: W: no-documentation
libkkc-tools.x86_64: W: no-manual-page-for-binary kkc
6 packages and 0 specfiles checked; 0 errors, 8 warnings.

+ Source verified with upstream as
https://bitbucket.org/libkkc/libkkc/downloads/libkkc-0.1.2.tar.gz :
  CHECKSUM(SHA256) this package     : 885c34780b8f5318bb8a4276e6f331fad37f020442d5f720dfea75da3f100da6
  CHECKSUM(SHA256) upstream package : 885c34780b8f5318bb8a4276e6f331fad37f020442d5f720dfea75da3f100da6
https://bitbucket.org/libkkc/libkkc-data/downloads/libkkc-data-0.1.1.tar.gz :
  CHECKSUM(SHA256) this package     : eeaaf2a4dfc9e051da044874169be5385d2cd4f2a0785b1de705d9955f3f10dc
  CHECKSUM(SHA256) upstream package : eeaaf2a4dfc9e051da044874169be5385d2cd4f2a0785b1de705d9955f3f10dc

+ License is GPLv3+  and included in COPYING as well as some source files in libkkc tarball


Suggestions:
1) you can add ChangeLog from data tarball to -data as %doc

2) I see that -data installing just models and keymaps. This should be noarch package then.

Comment 6 Daiki Ueno 2013-02-06 04:26:35 UTC
Thanks, added ChangeLog of libkkc-data.  However, there is no assumption that the marisa format is arch-independent.  So I would like to leave the arch as it is. 

Spec URL: http://ueno.fedorapeople.org/libkkc/libkkc.spec
SRPM URL: http://ueno.fedorapeople.org/libkkc/libkkc-0.1.2-2.fc18.src.rpm

Comment 7 Mamoru TASAKA 2013-02-06 04:50:21 UTC
Well,

(In reply to comment #6)
> However, there is no assumption
> that the marisa format is arch-independent.

So treating "-data" rpms (like this kind) needs attention.
http://www.pathname.com/fhs/pub/fhs-2.3.html#USRSHAREARCHITECTUREINDEPENDENTDATA
says files under /usr/share should be "arch-independent", and https://fedoraproject.org/wiki/Packaging:Guidelines?rd=Packaging/Guidelines#Filesystem_Layout says packages in Fedora must follow FHS.

So if you are unsure data files are really arch-independent, you must move them to arch-dependent directory (i.e. %_libdir) and apply patches if needed.

Comment 8 Daiki Ueno 2013-02-06 11:54:16 UTC
I asked the author of marisa-trie and confirmed that the format might be arch-dependent on big endian systems.  So I'll do:

- re-package libkkc-data to generate marisa files when building
- enable python binding of marisa in Fedora, to get the generation script working
- modify libkkc to pick the files from %_libdir

Comment 10 Parag AN(पराग) 2013-02-08 07:16:09 UTC
Suggestions:
1)python versioned BR should be used. You need BR: python2-devel
2)can we add further more packaging for -data?
 let noarch files be as -data noarch subpackage which will pull arch-ed files in -data-models

Comment 11 Daiki Ueno 2013-02-08 07:43:57 UTC
Updated, thanks.

Spec URL: http://ueno.fedorapeople.org/libkkc/libkkc.spec
SRPM URL: http://ueno.fedorapeople.org/libkkc/libkkc-0.1.3-2.fc18.src.rpm

(In reply to comment #10)
> 2)can we add further more packaging for -data?
>  let noarch files be as -data noarch subpackage which will pull arch-ed
> files in -data-models

I moved noarch data files to -common subpackage (noarch).

Comment 12 Parag AN(पराग) 2013-02-08 14:27:44 UTC
spec file looks good.

APPROVED.

Comment 13 Daiki Ueno 2013-02-09 07:37:22 UTC
New Package SCM Request
=======================
Package Name: libkkc
Short Description: Japanese Kana Kanji conversion library
Owners: ueno
Branches: f18 f17
InitialCC: i18n-team

Comment 14 Gwyn Ciesla 2013-02-09 20:52:24 UTC
Git done (by process-git-requests).

Comment 15 Fedora Update System 2013-02-11 23:35:14 UTC
libkkc-0.1.5-1.fc18 has been submitted as an update for Fedora 18.
https://admin.fedoraproject.org/updates/libkkc-0.1.5-1.fc18

Comment 16 Fedora Update System 2013-02-11 23:35:27 UTC
libkkc-0.1.5-1.fc17 has been submitted as an update for Fedora 17.
https://admin.fedoraproject.org/updates/libkkc-0.1.5-1.fc17

Comment 17 Fedora Update System 2013-02-12 02:09:02 UTC
ibus-kkc-1.5.6-1.fc18, marisa-0.2.1-3.fc18, libkkc-0.1.5-1.fc18 has been submitted as an update for Fedora 18.
https://admin.fedoraproject.org/updates/marisa-0.2.1-3.fc18,libkkc-0.1.5-1.fc18,ibus-kkc-1.5.6-1.fc18

Comment 18 Fedora Update System 2013-02-12 02:15:41 UTC
ibus-kkc-1.5.6-1.fc17, marisa-0.2.1-3.fc17, libkkc-0.1.5-1.fc17 has been submitted as an update for Fedora 17.
https://admin.fedoraproject.org/updates/marisa-0.2.1-3.fc17,libkkc-0.1.5-1.fc17,ibus-kkc-1.5.6-1.fc17

Comment 19 Fedora Update System 2013-02-23 00:52:17 UTC
ibus-kkc-1.5.6-1.fc18, marisa-0.2.1-3.fc18, libkkc-0.1.5-1.fc18 has been pushed to the Fedora 18 stable repository.

Comment 20 Fedora Update System 2013-02-23 01:04:43 UTC
ibus-kkc-1.5.6-1.fc17, marisa-0.2.1-3.fc17, libkkc-0.1.5-1.fc17 has been pushed to the Fedora 17 stable repository.


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