erabo/spiegel-keyman
1
0
Fork 0
mirror of https://github.com/keymanapp/keyman.git synced 2026-11-02 09:07:41 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 4

fix(linux): use ibus command to restart ibus 🍒

Browse source 
When using Kubuntu exiting ibus sometimes used to hang (#10178). This
was because ibus' `bus.exit()` doesn't shutdown properly
(see https://github.com/ibus/ibus/issues/2816#issuecomment-3410473708).
The recommended way is to use the `ibus exit` command. Since we no longer
use the API we can directly use the `ibus restart` command, which this
PR implements.

Fixes: #10178
Cherry-pick-of: #14971
This commit is contained in:
Eberhard Beilharz 2025-10-16 16:58:18 +02:00
parent 46e22c8398
commit 285a8463e7
No known key found for this signature in database
GPG key ID: E9140597606020D3
1 changed files with 2 additions and 14 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

16
linux/keyman-config/keyman_config/ibus_util.py
View file

@ -187,20 +187,8 @@ def restart_ibus(bus=None):
logging.info('restarting IBus by subprocess for user %s', realuser)
subprocess.run(['sudo', '-u', realuser, 'ibus', 'restart'], check=False)
else:
logging.info('restarting IBus through API')
try:
if not bus:
bus = get_ibus_bus()
if bus:
logging.info("restarting IBus")
# we no longer try to restart since we sometimes ended up with more than one
# ibus-daemon process (#6237). Instead we only exit ibus here, and start
# ibus again below.
bus.exit(False)
bus.destroy()
except Exception as e:
logging.warning("Failed to restart IBus")
logging.warning(e)
logging.info('restarting IBus by subprocess')
subprocess.run(['ibus', 'restart'], check=False)
# give ibus a chance to shutdown (#6237)
time.sleep(1) # 1s
verify_ibus_daemon(True)