Bug 824061 - Review Request: fcitx-unikey - Vietnamese Engine for Fcitx
Summary: Review Request: fcitx-unikey - Vietnamese Engine for Fcitx
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
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:
: 824063 (view as bug list)
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-05-22 16:26 UTC by Liang Suilong
Modified: 2013-01-12 01:09 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-12-28 03:52:02 UTC
Type: Bug
Embargoed:
panemade: fedora-review+
gwync: fedora-cvs+


Attachments (Terms of Use)

Comment 1 Liang Suilong 2012-05-22 16:31:28 UTC
(In reply to comment #0)
> SPEC: http://liangsuilong.fedorapeople.org/fcitx/fcitx-unikey.spec
> SRPM:
> http://liangsuilong.fedorapeople.org/fcitx/fcitx-hangul-0.1.0-1.fc16.src.rpm
> Koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=4094915
> Description:
> A Vietnamese engine for Fcitx that uses Unikey.

I post a wrong SRPM URL. Here is a correct URL. 

SPEC: http://liangsuilong.fedorapeople.org/fcitx/fcitx-unikey.spec
SRPM: http://liangsuilong.fedorapeople.org/fcitx/fcitx-unikey-0.1.0-1.fc16.src.rpm
Koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=4094915
Description:
A Vietnamese engine for Fcitx that uses Unikey.

Comment 2 Kevin Fenzi 2012-05-22 16:58:10 UTC
*** Bug 824063 has been marked as a duplicate of this bug. ***

Comment 3 Kevin Fenzi 2012-05-22 17:01:06 UTC
Any particular reason you assigned this to me? 

I can try and help review, but I'm pretty busy and likely won't get to it very soon...

Comment 4 Parag AN(पराग) 2012-05-23 03:33:30 UTC
Kevin,
   I will help the submitter with this package review like I did for his other fcitx-* package reviews. Taking this for review.

Comment 5 Parag AN(पराग) 2012-05-24 04:18:50 UTC
Review:-

+ Koji build used ->http://koji.fedoraproject.org/koji/taskinfo?taskID=4094915

+ rpmlint on rpms gave
fcitx-unikey.src: W: invalid-url Source0: http://fcitx.googlecode.com/files/fcitx-unikey-0.1.0.tar.xz HTTP Error 404: Not Found
2 packages and 0 specfiles checked; 0 errors, 1 warnings.
==> This can be ignored

+ Source verified with upstream as (sha1sum)
ef57f512d3ded16f6656be5e07e4465cbf13ffd5  fcitx-unikey-0.1.0.tar.xz
ef57f512d3ded16f6656be5e07e4465cbf13ffd5  ../SOURCES/fcitx-unikey-0.1.0.tar.xz

Suggestions:
1) each pushd command should have corresponding popd. please add it in %build

2) I am not sure about final license tag. As ukengine part is in LGPLv2+, VnConv is in GPLv2+ and your own code is in GPLv3+.

Can you check with upstream? Guess you need to change your code from GPLv3+ to GPLv2+ 

3) The correct way to handle locale files is remove manually generation of %{name}.lang in spec file and use "%find_lang %{name}" at the end of %install section. See https://fedoraproject.org/wiki/Packaging/Guidelines#Handling_Locale_Files

Comment 6 Parag AN(पराग) 2012-05-24 04:45:35 UTC
I asked on #fedora-devel and come to know that without changing source license, this package can have license tag as GPLv3+

Comment 7 Parag AN(पराग) 2012-06-06 03:56:38 UTC
any update here?

Comment 8 Parag AN(पराग) 2012-06-29 14:17:46 UTC
any update here?

Comment 9 Parag AN(पराग) 2012-07-09 05:50:01 UTC
Can you please provide update here so that once package review completes and built in fedora, will be available for testing?

Comment 10 Parag AN(पराग) 2012-07-20 08:54:13 UTC
liangsuilong,  
   If you are busy with your work then please allow me to fix this package and request SCM for this package and build it in Fedora so that people can use this.

Comment 12 Parag AN(पराग) 2012-08-06 05:35:55 UTC
It would have been good if you could have added some changelog and bumped the release tag for above change.

APPROVED.

Comment 13 Parag AN(पराग) 2012-10-25 03:40:51 UTC
Hi Liang,
   If you are busy, can I request you to allow me to import this package and co-own with you this package?

Comment 14 Parag AN(पराग) 2012-12-11 03:15:53 UTC
No response from reporter since last 4 months here.
If you still want this package in fedora then just reopen this and request for git package module.
For now closing...

Comment 15 Liang Suilong 2012-12-11 04:47:28 UTC
New Package SCM Request
=======================
Package Name: fcitx-unikey
Short Description: Vietnamese Engine for Fcitx
Owners: liangsuilong
Branches: f16 f17 f18 el6
InitialCC: i18n-team

Comment 16 Gwyn Ciesla 2012-12-11 13:14:25 UTC
Git done (by process-git-requests).

Comment 17 Fedora Update System 2012-12-13 15:09:59 UTC
fcitx-unikey-0.1.1-2.fc18 has been submitted as an update for Fedora 18.
https://admin.fedoraproject.org/updates/fcitx-unikey-0.1.1-2.fc18

Comment 18 Fedora Update System 2012-12-13 15:10:18 UTC
fcitx-unikey-0.1.1-2.fc17 has been submitted as an update for Fedora 17.
https://admin.fedoraproject.org/updates/fcitx-unikey-0.1.1-2.fc17

Comment 19 Fedora Update System 2012-12-14 06:47:45 UTC
fcitx-unikey-0.1.1-2.fc18 has been pushed to the Fedora 18 testing repository.

Comment 20 Fedora Update System 2012-12-28 03:52:04 UTC
fcitx-unikey-0.1.1-2.fc17 has been pushed to the Fedora 17 stable repository.

Comment 21 Fedora Update System 2013-01-12 01:09:16 UTC
fcitx-unikey-0.1.1-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.