[Pkg-ime-devel] RFS: ibus-typing-booster -- [second try]

[Vasudev Kamath]

Osamu Aoki <osamu_aoki_home at nifty.com> writes:

> Hi,
>
> On Wed, Jan 22, 2014 at 10:22:25PM +0530, Vasudev Kamath wrote:
>> Osamu Aoki <osamu_aoki_home at nifty.com> writes:
> ...
>> > When ever ask sponsor, it is good idea to post full information.
>> 
>> Sorry for not mentioning the URL, I just mentioned its in the team
>> repository (a habbit carried from other team I interact with).
>
> OK I see.
>  
>> Sorry for not asking earlier but what is the way to ask for sponsor in
>> this team? Do I need to upload to mentors and give the URL or upload to
>> team repository and give its URL?
>
> Enough information for us not to do second guessing.  No particular
> rule.  Just common sense.  Usually people putting package to mentor tell
> us URL to *.dsc file.  For your case:
>
> url = ssh://git.debian.org/git/pkg-ime/ibus-typing-booster
>
> this should be OK.  It is quite obvious from there.

Noted.

>
>> I want to make one thing clear I really don't want to break the
>> tradition of the team, if some one is willing to take care of this
>> package I will be happy to convert it to dh and give it over. Please let
>> me know on this.
>
> If you upload it to archive, you are the primary the one to take care.
> We are very thin in resource.  That does not mean we will not step in.
> We do our best though.
>  
>> This is not really the right place to discuss this, but Jonas has some
>> different point on your thoughts and I agree with him, if you are
>> interested to take up this discussion I would suggest join us on
>> build-common-hackers at list.alioth.debian.org.
>
> I do not think I qualify to discuss.  That was more of my sentiment to
> explain why I chose dh.  I know GNOME team uses CDBS.
>
> Let's discuss more important thing.

Alright :).

>
> Lintian
> W: ibus-typing-booster source: out-of-date-standards-version 3.9.4 (current is 3.9.5)
>

Ah fixed this.

> Also it is not needed but here is an idea.
>
> I am no perl hacker so the one liner script got my attention.
>
> The scalability of "scriptlist" to adopt new languages as upstream
> accumulates may be a concern.  scriptlist=... line ... requiring
> constant care.  But as I see:
>
>  $ sed -n 's/^description.* for \([^.]*\).*/\1/p' hunspell-tables/*
>
> This extracts most languages.  With a bit of massaging to patch
> description of *.conf in consistent style, ... that what I though maybe
> an idea if you find time later.

Yeah that is really nice idea, instead of description I changed it to
ime_name and its working fine, only problem is there is 2 variation of
Norwegian which is like Norwegian (sometext) which splits like

* Norwegian
* (somtext)

This is not desired, but I don't  know how to over come this so I've
made it as comment for now. If you can suggest modification for fixing
this let me know.

# scriptlist := $(shell sed -n 's/^ime_name.* = \([^.]*\) -.*/\1/p' hunspell-tables/* | sort -u)

>
> Anyway, this is certainly serious CDBS package which I can not touch.
> It is too convoluted for me to dig in.  But it looks generally in good
> shape.

Can you do me a favour and try the built package? I can provide you the
deb file if desired. I'm not able to get ibus working on my system
without Gnome :(.

Also is it okay to suggest/recommend huge number of hunspell
dictionaries with this package?

Thanks for the review.
-- 
Vasudev Kamath
http://copyninja.info
Connect on ~friendica: copyninja@{frndk.de | vasudev.homelinux.net}
IRC nick: copyninja | vasudev {irc.oftc.net | irc.freenode.net}
GPG Key: C517 C25D E408 759D 98A4  C96B 6C8F 74AE 8770 0B7E
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 818 bytes
Desc: not available
URL: <http://lists.alioth.debian.org/pipermail/pkg-ime-devel/attachments/20140125/57bde7a1/attachment.sig>