Merge lp:~hennekn/ubuntu/trusty/ibus/bug-1240198 into lp:ubuntu/trusty/ibus

Proposed by Hendrik Schrieber
Status: Work in progress
Proposed branch: lp:~hennekn/ubuntu/trusty/ibus/bug-1240198
Merge into: lp:ubuntu/trusty/ibus
Diff against target: 48 lines (+15/-0)
4 files modified
.pc/applied-patches (+1/-0)
debian/changelog (+6/-0)
debian/patches/04_enable_system_layout.patch (+7/-0)
debian/patches/series (+1/-0)
To merge this branch: bzr merge lp:~hennekn/ubuntu/trusty/ibus/bug-1240198
Reviewer Review Type Date Requested Status
Aron Xu (community) Needs Fixing
Ubuntu branches Pending
Review via email: mp+220843@code.launchpad.net

Description of the change

This patch is supposed to fix Bug#1240198 by adding an additional config file to set the default value of "use-system-keyboard-layout" to "true" and therefore prevent ibus from falling back to the en-us keyboard layout.

For the setting to take effect "dconf update" has to be run but based on other patches to IBus I suppose this happens automatically via triggers on install.

This is not easily testable since the bug is semi-random and ibus not working as a standalone app. Therefore it should be tested by a wider audience.

This is my first patch and I am not certain if this will work so any guidance or help would be appreciated.

To post a comment you must log in.
Revision history for this message
William Hua (attente) wrote :

Hi, I haven't been able to replicate the original bug, but looking at the diff, I think you need to add 04_enable_system_layout.patch to EXTRA_DIST in data/dconf/Makefile.am.

Revision history for this message
Aron Xu (happyaron) wrote :

Agree with William, and also please clean up those .pc noise. Thanks!

review: Needs Fixing
Revision history for this message
Sebastien Bacher (seb128) wrote :

Changing to "work in progress" so it's out of the sponsoring queue until the pointed issues are resolved, change back to "needs review" then

Unmerged revisions

49. By Hendrik Knackstedt <hendrik@hendrik-Aspire-5750G>

Add patch to fix Bug #1240198 by adding a custom config to enable
the use of the system's keyboard layout by default.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== added directory '.pc/04_enable_system_layout.patch'
2=== added file '.pc/04_enable_system_layout.patch/.timestamp'
3=== added directory '.pc/04_enable_system_layout.patch/data'
4=== added directory '.pc/04_enable_system_layout.patch/data/dconf'
5=== added file '.pc/04_enable_system_layout.patch/data/dconf/04_enable_system_layout'
6=== modified file '.pc/applied-patches'
7--- .pc/applied-patches 2014-03-19 14:08:29 +0000
8+++ .pc/applied-patches 2014-05-23 18:56:50 +0000
9@@ -11,3 +11,4 @@
10 03_dont_run_dconf_update.patch
11 ubuntu-disable-trigger.patch
12 ubuntu-forward-panel-properties.patch
13+04_enable_system_layout.patch
14
15=== modified file 'debian/changelog'
16--- debian/changelog 2014-03-19 14:08:29 +0000
17+++ debian/changelog 2014-05-23 18:56:50 +0000
18@@ -1,3 +1,9 @@
19+ibus (1.5.5-1ubuntu4) UNRELEASED; urgency=medium
20+
21+ * Add patch to fix Bug #1240198
22+
23+ -- Hendrik Knackstedt <hendrik@hendrik-Aspire-5750G> Fri, 23 May 2014 20:36:30 +0200
24+
25 ibus (1.5.5-1ubuntu3) trusty; urgency=medium
26
27 * debian/patches/ubuntu-forward-panel-properties.patch:
28
29=== added file 'debian/patches/04_enable_system_layout.patch'
30--- debian/patches/04_enable_system_layout.patch 1970-01-01 00:00:00 +0000
31+++ debian/patches/04_enable_system_layout.patch 2014-05-23 18:56:50 +0000
32@@ -0,0 +1,7 @@
33+Index: ibus/data/dconf/04_enable_system_layout
34+===================================================================
35+--- /dev/null 1970-01-01 00:00:00.000000000 +0000
36++++ ibus/data/dconf/04_enable_system_layout 2014-05-23 20:30:21.944418735 +0200
37+@@ -0,0 +1,2 @@
38++[desktop/ibus/general]
39++use-system-keyboard-layout=true
40
41=== modified file 'debian/patches/series'
42--- debian/patches/series 2014-03-19 14:08:29 +0000
43+++ debian/patches/series 2014-05-23 18:56:50 +0000
44@@ -99,3 +99,4 @@
45 03_dont_run_dconf_update.patch
46 ubuntu-disable-trigger.patch
47 ubuntu-forward-panel-properties.patch
48+04_enable_system_layout.patch

Subscribers

People subscribed via source and target branches