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

[Osamu Aoki]

Hi,

On Sat, Jan 25, 2014 at 02:54:45PM +0530, Vasudev Kamath wrote:
> Osamu Aoki <osamu_aoki_home at nifty.com> writes:
...
> > 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.

Hint: patch the source for the corresponding part to make them more
consostent.
  http://www.debian.org/doc/manuals/maint-guide/modify.en.html
 
> # scriptlist := $(shell sed -n 's/^ime_name.* = \([^.]*\) -.*/\1/p' hunspell-tables/* | sort -u)

Yah, that much I can dig in.  A brute force thing to do is add another sed
script to change "Norwegian" and "(somtext)" to what you wish to be.

> > 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 commented as here based on the build package.

> I can provide you the
> deb file if desired. I'm not able to get ibus working on my system
> without Gnome :(.

I see.  What Desktop you use.

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

Recommend may not be popular and bad idea.

But why not suggest.  You may also think some script so updating them
are semi-automatic. (with good amount of suggestion pointer in
debian/rules or in the commento of control.

I will sponsor you if you convert this to dh.  

Problem with CDBS for me is I have to know and dig deep into CDBS to
find out customization done by the package maintainer (without waiting
for thesolutionfrom CDBS developer.)  If dh's overide system braks,
everyone knows and it will be fixed for sure.  But time spent for
finding DEB_CONFIGURE_EXTRA_FLAGS is the place to do
 
 ./configure --libexecdir=/usr/lib/ibus

was enough for me to hate it. 

Anyway, please consider attached as starter for debian/rules

Osamu 
-------------- next part --------------
#!/usr/bin/make -f
# output every command that modifies files on the build system.
#DH_VERBOSE = 1

# see FEATURE AREAS in dpkg-buildflags(1))
#export DEB_BUILD_MAINT_OPTIONS = hardening=+all
# see ENVIRONMENT in dpkg-buildflags(1))
# package maintainers to append CFLAGS
export DEB_CFLAGS_MAINT_APPEND  = -Wall -pedantic
# package maintainers to append LDFLAGS
export DEB_LDFLAGS_MAINT_APPEND = -Wl,--as-needed

%:
		dh $@ --with "autotools-dev"

# python2 is skipped since this is for sid/testing

override_dh_auto_configure:
	dh_auto_configure -- \
	        --disable-additional \
	        --disable-static \
	        --libdir=/usr/lib \
	        --libexecdir=/usr/lib/ibus

### Following needs to be adjusted #############

pkg = $(DEB_SOURCE_PACKAGE)

DEB_UPSTREAM_URL = https://fedorahosted.org/releases/i/b/ibus-typing-booster
#DEB_UPSTREAM_TARBALL_MD5 = 0c44cd8a9cbe56bf9894732985864700

scriptlist := English Ukrainian Icelandic Russian Portuguese German Yiddish
scriptlist += Thai Sinhala Swedish Polish Norwegian Kurdish Hungarian Greek
scriptlist += Czech Romanian Norwegian Italian Armenian Hebrew Persian Spanish
scriptlist += Bulgarian Belarusian Hindi Marathi Bengali Urdu Farsi Nepali
scriptlist += French Danish Oriya Telugu Tamil  Punjabi Gujarati Maithili
scriptlist += Assamese Kannada

# scripts := $(shell echo ${scriptlist} | tr ' ' '\n' |sort -u | perl -0 -F/\\n/ -ane 'print " * ", join("\$${Newline} * ", at F)')
scripts := $(shell echo ${scriptlist} | sort -u | perl -ane 'print " * ", join("\$${Newline} * ", at F)')


## Upstream build modifies following file
#DEB_UPSTREAM_CRUFT_COPY = hunspell-tables ibus-typing-booster.spec
#DEB_UPSTREAM_CRUFT_COPY = engine/typing-booster.xml.in
#
#install/$(pkg)::
#	echo 'supported:scriptlist=$(scripts)' >> debian/ibus-typing-booster.substvars
#
# Remove the byte-compiled files from the package this should happen
# at the post-installation step
#binary-install/$(pkg)::
#	find debian/$(pkg) -name "*.py[co]" -exec rm -f {} \;