Bug 407779 – app-i18n/ibus-chewing-1.3.10: fails to save configuration

• Status: RESOLVED FIXED
• Product: Gentoo Linux

[Description Jack Chen 2012-03-11 11:09:50 UTC]

Created attachment 304905 [details]
Patch for app-i18n/ibus-chewing-1.3.10 to fix the problem.

Problem and possible cause:
The internal configuration of app-i18n/ibus-chewing-1.3.10 are not saved successfully.
This problem starts from app-i18n/ibus-1.4.0, where IBus changed their configuration value type from GValue to GVariant.
ibus-chewing adapts the change by writing functions to convert between GValue and GVariant.  But it seems that the GValue->GVariant conversion function dealt with some types incorrectly, which caused g_value_get_uint() type checking failure.


Steps to reproduce:
1. Install >=app-i18n/ibus-1.4.0.
2. Install =app-i18n/ibus-chewing-1.3.10.
3. Start IBus normally, switch to ibus-chewing input method.
4. On ibus-chewing language panel, click "configure" button (the second button from right).  Or if you have language panel embedded in menu, left-click on the IBus icon and choose the third item (Chinese: 設定).
5. Adjust the 5th setting (a spinbox with range 8-40, Chinese: 容納中文字數).
6. Click save (button at right-bottom corner, Chinese: 儲存).
7. Start another application, then repeat step 3 and 4.  You'll find that the setting has gone back to 8, not the value configured in step 5.

All three fields with a spinbox are affected.


Actual result:
The saved configuration does not persist and keeps going back to a certain value.
Here's the output found when debugging with "ibus-daemon --verbose":
(ibus-engine-chewing:4097): GLib-GObject-CRITICAL **: g_value_get_uint: assertion `G_VALUE_HOLDS_UINT (value)' failed
(ibus-engine-chewing:4097): GLib-GObject-CRITICAL **: g_value_get_uint: assertion `G_VALUE_HOLDS_UINT (value)' failed
(ibus-engine-chewing:4097): GLib-GObject-CRITICAL **: g_value_get_uint: assertion `G_VALUE_HOLDS_UINT (value)' failed


Expected result:
The saved configure should persist.


How often:
Always reproducible.


Fix:
I've attached a patch for app-i18n/ibus-chewing-1.3.10 to fix the problem.  I'm still testing if this patch affects other parts of ibus-chewing, but it works so far.

When I have spare time, I'll try to report this and bug 407695 to ibus-chewing developers.


PS. I have added a patch in bug 407695.  I'm just making sure that somebody sees the new patch.  If it's enough to just move /gconf into /etc by adding mv commands in ebuild, then it's okay.

[Comment 1 Jack Chen 2012-03-11 11:16:17 UTC]

Portage 2.1.10.44 (default/linux/amd64/10.0, gcc-4.5.3, glibc-2.13-r4, 3.2.1-gentoo-r2 x86_64)
=================================================================
System uname: Linux-3.2.1-gentoo-r2-x86_64-Intel-R-_Core-TM-_i7_CPU_860_@_2.80GHz-with-gentoo-2.0.3
Timestamp of tree: Fri, 09 Mar 2012 16:15:01 +0000
app-shells/bash:          4.1_p9
dev-java/java-config:     2.1.11-r3
dev-lang/python:          2.7.2-r3, 3.2.2
dev-util/cmake:           2.8.6-r4
dev-util/pkgconfig:       0.26
sys-apps/baselayout:      2.0.3
sys-apps/openrc:          0.9.8.4
sys-apps/sandbox:         2.5
sys-devel/autoconf:       2.13, 2.68
sys-devel/automake:       1.9.6-r3, 1.11.1
sys-devel/binutils:       2.21.1-r1
sys-devel/gcc:            4.5.3-r2, 4.6.2
sys-devel/gcc-config:     1.5-r2
sys-devel/libtool:        2.4-r1
sys-devel/make:           3.82-r1
sys-kernel/linux-headers: 3.1 (virtual/os-headers)
sys-libs/glibc:           2.13-r4
Repositories: gentoo gentoo-zh sunrise my_dev
ACCEPT_KEYWORDS="amd64"
ACCEPT_LICENSE="* -@EULA"
CBUILD="x86_64-pc-linux-gnu"
CFLAGS="-march=native -O2 -pipe"
CHOST="x86_64-pc-linux-gnu"
CONFIG_PROTECT="/etc /usr/share/config /usr/share/gnupg/qualified.txt /usr/share/themes/oxygen-gtk/gtk-2.0"
CONFIG_PROTECT_MASK="/etc/ca-certificates.conf /etc/env.d /etc/env.d/java/ /etc/fonts/fonts.conf /etc/gconf /etc/gentoo-release /etc/revdep-rebuild /etc/sandbox.d /etc/terminfo"
CXXFLAGS="-march=native -O2 -pipe"
DISTDIR="/usr/portage/distfiles"
FEATURES="assume-digests binpkg-logs distlocks ebuild-locks fixlafiles news parallel-fetch protect-owned sandbox sfperms strict unknown-features-warn unmerge-logs unmerge-orphans userfetch"
FFLAGS=""
GENTOO_MIRRORS="http://ftp.twaren.net/Linux/Gentoo/ http://ftp.ncnu.edu.tw/Linux/Gentoo/ http://gentoo.cs.nctu.edu.tw/gentoo/"
LANG="en_US.UTF-8"
LDFLAGS="-Wl,-O1 -Wl,--as-needed"
LINGUAS="en en_US zh_TW ja zh_CN"
MAKEOPTS="-j8"
PKGDIR="/usr/portage/packages"                                                                                                                                                        
PORTAGE_CONFIGROOT="/"                                                                                                                                                                
PORTAGE_RSYNC_OPTS="--recursive --links --safe-links --perms --times --compress --force --whole-file --delete --stats --timeout=180 --exclude=/distfiles --exclude=/local --exclude=/packages"                                                                                                                                                                              
PORTAGE_TMPDIR="/var/tmp"                                                                                                                                                             
PORTDIR="/usr/portage"                                                                                                                                                                
PORTDIR_OVERLAY="/var/lib/layman/gentoo-zh /var/lib/layman/sunrise /usr/local/portage/overlay"                                                                                        
SYNC="rsync://rsync6.tw.gentoo.org/gentoo-portage"                                                                                                                                    
USE="X acl acpi alsa amd64 bash-completion berkdb branding bzip2 cairo cdr cjk cli consolekit cracklib crypt cups cxx dbus dri dvd dvdr exif fam fortran gdbm gif gpm iconv ipv6 jpeg kde lcms libnotify lm_sensors mmx mmxext mng modules mudflap multilib ncurses nls nptl nptlonly opengl openmp pam pcre pdf png policykit pppd pulseaudio qt3support qt4 readline semantic-desktop session sse sse2 sse3 sse4_1 ssl ssse3 startup-notification subversion svg sysfs tcpd threads tiff truetype udev unicode vim-pager vim-syntax xml xorg xpm zlib" ALSA_CARDS="hda-intel" ALSA_PCM_PLUGINS="adpcm alaw asym copy dmix dshare dsnoop empty extplug file hooks iec958 ioplug ladspa lfloat linear meter mmap_emul mulaw multi null plug rate route share shm softvol" APACHE2_MODULES="actions alias auth_basic authn_alias authn_anon authn_dbm authn_default authn_file authz_dbm authz_default authz_groupfile authz_host authz_owner authz_user autoindex cache cgi cgid dav dav_fs dav_lock deflate dir disk_cache env expires ext_filter file_cache filter headers include info log_config logio mem_cache mime mime_magic negotiation rewrite setenvif speling status unique_id userdir usertrack vhost_alias" CALLIGRA_FEATURES="kexi words flow plan stage tables krita karbon braindump" CAMERAS="ptp2" COLLECTD_PLUGINS="df interface irq load memory rrdtool swap syslog" ELIBC="glibc" GPSD_PROTOCOLS="ashtech aivdm earthmate evermore fv18 garmin garmintxt gpsclock itrax mtk3301 nmea ntrip navcom oceanserver oldstyle oncore rtcm104v2 rtcm104v3 sirf superstar2 timing tsip tripmate tnt ubx" INPUT_DEVICES="evdev wacom" KERNEL="linux" LCD_DEVICES="bayrad cfontz cfontz633 glk hd44780 lb216 lcdm001 mtxorb ncurses text" LINGUAS="en en_US zh_TW ja zh_CN" PHP_TARGETS="php5-3" RUBY_TARGETS="ruby18" USERLAND="GNU" VIDEO_CARDS="fglrx" XTABLES_ADDONS="quota2 psd pknock lscan length2 ipv4options ipset ipp2p iface geoip fuzzy condition tee tarpit sysrq steal rawnat logmark ipmark dhcpmac delude chaos account"
Unset:  CPPFLAGS, CTARGET, EMERGE_DEFAULT_OPTS, INSTALL_MASK, LC_ALL, PORTAGE_BUNZIP2_COMMAND, PORTAGE_COMPRESS, PORTAGE_COMPRESS_FLAGS, PORTAGE_RSYNC_EXTRA_OPTS

[Comment 2 Samuli Suominen (RETIRED) 2012-03-11 11:23:26 UTC]

This is now at:

http://code.google.com/p/ibus/issues/detail?id=1428

[Comment 3 Samuli Suominen (RETIRED) 2012-03-11 11:29:14 UTC]

Applied to tree

+*ibus-chewing-1.3.10-r1 (11 Mar 2012)
+
+  11 Mar 2012; Samuli Suominen <[e-mail]>
+  +ibus-chewing-1.3.10-r1.ebuild,
+  +files/ibus-chewing-1.3.10-fix_save_setting.patch:
+  Fix saving of configuration wrt #407779 by Jack Chen