Bug 798891 - Review Request: libtranslit - transliteration library with multiple backends
Summary: Review Request: libtranslit - transliteration library with multiple backends
Keywords:
Status: CLOSED RAWHIDE
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:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-03-01 08:08 UTC by Daiki Ueno
Modified: 2014-01-02 10:30 UTC (History)
6 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-03-05 04:42:27 UTC
Type: ---
Embargoed:
panemade: fedora-review+
gwync: fedora-cvs+


Attachments (Terms of Use)

Description Daiki Ueno 2012-03-01 08:08:57 UTC
Spec URL: http://ueno.fedorapeople.org/libtranslit/libtranslit.spec
SRPM URL: http://ueno.fedorapeople.org/libtranslit/libtranslit-0.0.1-1.fc18.src.rpm
Description:
GObject-based Unicode transliteration library with multiple backends.

Comment 1 Parag AN(पराग) 2012-03-02 08:58:06 UTC
Review:
+ koji build (f17) ->http://koji.fedoraproject.org/koji/taskinfo?taskID=3845925

+ rpmlint on rpms gave
libtranslit.x86_64: W: spelling-error Summary(en_US) backends -> back ends, back-ends, backhands
libtranslit.x86_64: W: spelling-error %description -l en_US backends -> back ends, back-ends, backhands
libtranslit.x86_64: W: no-documentation
libtranslit.x86_64: W: devel-file-in-non-devel-package /usr/lib64/libtranslit/modules/libtranslitm17n.so
libtranslit.src: W: spelling-error Summary(en_US) backends -> back ends, back-ends, backhands
libtranslit.src: W: spelling-error %description -l en_US backends -> back ends, back-ends, backhands
2 packages and 0 specfiles checked; 0 errors, 6 warnings.

+ source verified with upstream as (sha1sum)
ea9dbe0b4d2bd9495394df307b385686eaffa7a7  ../SOURCES/libtranslit-0.0.1.tar.gz
ea9dbe0b4d2bd9495394df307b385686eaffa7a7  libtranslit-0.0.1.tar.gz

+ package follows packaging guidelines.

APPROVED.

Comment 2 Daiki Ueno 2012-03-02 09:05:23 UTC
Thanks for the review, Parag.

New Package SCM Request
=======================
Package Name: libtranslit
Short Description: transliteration library with multiple backends
Owners: ueno
Branches: f17 f16 f15
InitialCC: i18n-team

Comment 3 Gwyn Ciesla 2012-03-02 12:59:51 UTC
Git done (by process-git-requests).

Comment 4 Fedora Update System 2012-03-05 04:39:15 UTC
libtranslit-0.0.2-1.fc15 has been submitted as an update for Fedora 15.
https://admin.fedoraproject.org/updates/libtranslit-0.0.2-1.fc15

Comment 5 Fedora Update System 2012-03-05 04:39:24 UTC
libtranslit-0.0.2-1.fc16 has been submitted as an update for Fedora 16.
https://admin.fedoraproject.org/updates/libtranslit-0.0.2-1.fc16

Comment 6 Fedora Update System 2012-03-05 04:39:33 UTC
libtranslit-0.0.2-1.fc17 has been submitted as an update for Fedora 17.
https://admin.fedoraproject.org/updates/libtranslit-0.0.2-1.fc17

Comment 7 Fedora Update System 2012-03-10 02:03:36 UTC
libtranslit-0.0.2-1.fc17 has been pushed to the Fedora 17 stable repository.

Comment 8 Fedora Update System 2012-03-22 01:55:43 UTC
libtranslit-0.0.2-1.fc16 has been pushed to the Fedora 16 stable repository.

Comment 9 Fedora Update System 2012-03-22 01:57:25 UTC
libtranslit-0.0.2-1.fc15 has been pushed to the Fedora 15 stable repository.

Comment 10 Pravin Satpute 2013-12-10 05:43:07 UTC
Hi Daiki,

   We want to make ibus-typing-booster available on RHEL6, for that purpose we need EPEL6 branch for libtranslit. 
  
   Let me know if you are fine with it, we can do the rest..

Comment 11 Daiki Ueno 2013-12-10 05:59:54 UTC
Sure, please.

Comment 12 Pravin Satpute 2013-12-10 06:04:57 UTC
Thanks Daiki :)

Comment 13 Pravin Satpute 2013-12-27 09:49:09 UTC
I was trying to build this for epel6 but unfortunately gobject-introspection is not available in RHEL-6.

and while trying ELEL-6 for gobject-introspection i found 
https://bugzilla.redhat.com/show_bug.cgi?id=468285#c10

Can we build libtranslit with --disable-introspection and will it work as expected?  

Earlier i thought it will be easier but it looks complicated.

Comment 14 Pravin Satpute 2013-12-31 10:06:20 UTC
Finally able to do scratch build but with some alterations:

1.--enable-introspection=no
2. Dropped icu subpackage: EL6 does not have build dependency icu-io

scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=6344468

Comment 15 Mike FABIAN 2014-01-02 07:03:52 UTC
(In reply to Pravin Satpute from comment #13)
> I was trying to build this for epel6 but unfortunately gobject-introspection
> is not available in RHEL-6.

ibus-typing-booster needs gobject-introspection as well.

Comment 16 Pravin Satpute 2014-01-02 10:30:28 UTC
I think good to have that it in BuildRequires of ibus-typing-booster.


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