Merge lp:~mniess/ubuntu/trusty/ibus/fix-lp1240198 into lp:ubuntu/trusty/ibus
Status: | Needs review |
---|---|
Proposed branch: | lp:~mniess/ubuntu/trusty/ibus/fix-lp1240198 |
Merge into: | lp:ubuntu/trusty/ibus |
Diff against target: |
48 lines (+28/-0) 3 files modified
debian/changelog (+8/-0) debian/patches/ibus-use-system-keyboard.patch (+19/-0) debian/patches/series (+1/-0) |
To merge this branch: | bzr merge lp:~mniess/ubuntu/trusty/ibus/fix-lp1240198 |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Jamie Strandboge | 2014-06-18 | Needs Fixing on 2014-08-11 | |
Aron Xu (community) | Approve on 2014-07-27 | ||
Review via email:
|
Description of the change
This fixes the bug of randomly changing keyboard layouts (LP: #1240198)
Matthias Niess (mniess) wrote : | # |
> Same here, the files added need to be installed in Makefile.
There was only one file added to debian/patches which should not need to be added to the Makefile.
Jamie Strandboge (jdstrand) wrote : | # |
At first glance, this seems ok, however, I have two questions:
1. is this something suitable for upstreaming and if so, has it been upstreamed to ibus or Debian? I think I can see why it would not be, but it would be good to have someone explicitly comment
2. the patch is against trusty, but per SRU policies (https:/
Matthias Niess (mniess) wrote : | # |
> At first glance, this seems ok, however, I have two questions:
> 1. is this something suitable for upstreaming and if so, has it been
> upstreamed to ibus or Debian? I think I can see why it would not be, but it
> would be good to have someone explicitly comment
I'm pretty sure this is related to the way Ubuntu uses ibus (which also leads to custom keyboard shortcuts not working) so I would say this is not suitable for upstreaming.
> 2. the patch is against trusty, but per SRU policies
> (https:/
> development release and then applied to a stable release. Can you adjust the
> packaging and test that it works ok in utopic?
I currently have no machine to put utopic on so I cannot test this. I just wanted to help out get this resolved quickly and have this ready for 14.04.2 as it is a huge problem for non-english users. This is especially bad for corporate users who rely on Ubuntu for their work. It is only one dconf setting that was changed in 13.10 and immediately lead to the issue. It should have never made it into 14.04 but it did.
Bruno Nova (brunonova) wrote : | # |
@Matthias, you could install Utopic on a virtual machine (like Virtualbox) and test the patch in there.
(I'm not an Ubuntu developer.)
Matthias Niess (mniess) wrote : | # |
> @Matthias, you could install Utopic on a virtual machine (like Virtualbox) and
> test the patch in there.
> (I'm not an Ubuntu developer.)
Will do this weekend!
Bruno Nova (brunonova) wrote : | # |
I tried to test this in Utopic, but every time I run debuild, the file data/dconf/
It seems data/dconf/
Patching that file seems to generate the desired data/dconf/
@Matthias, do you want to work on this?
Matthias Niess (mniess) wrote : | # |
@Bruno sorry I didn't get to it, yet. As utopic is released the whole thing needs to be based on vivid (15.04) now, right? My time-consuming day job prevented me from installing anything but trusty on my machines until now.
So the only thing really is changing the default for "use-system-
I would love to work on this but I cannot promise I'll find the time to install vivid soon, to test it there.
Bruno Nova (brunonova) wrote : | # |
@Matthias, the development branch of the package will still be the same in Vivid as it was in Utopic: lp:ubuntu/ibus.
I'll work on this then, either today or next Monday.
Setting the "use-system-
I did that in my laptop using "ibus-setup" and I never had any problems again, and I also haven't noticed any regressions.
Bruno Nova (brunonova) wrote : | # |
Editing data/dconf/
But, after compiling the package and installing it using "dpkg -i", running ibus-setup fails, printing:
(ibus-
Trace/
I have no idea why. I doubt this small change is causing this error.
Also, I tested this in Utopic, since Vivid wouldn't boot in Virtualbox.
I'm going to provide this change as a patch in the bug report.
And I'm not sure that this bug affect Utopic (fresh installs).
/desktop/
Unmerged revisions
- 50. By Matthias Niess on 2014-06-18
-
added dep-3 headers
- 49. By Matthias Niess on 2014-06-18
-
* debian/
patches/ ibus-use- system- keyboard. patch:
- change dconf default of use-system-keyboard- layout to true
(LP: #1240198)
Same here, the files added need to be installed in Makefile.