I tried to fix this by replacing:

    if os.getenv("XDG_CURRENT_DESKTOP") == "Unity":
    
with:

    desktop = os.environ.get('XDG_CURRENT_DESKTOP')
    if desktop and 'Unity' in desktop.split(':'):

But then ibus-setup didn't start at all under Unity:

$ ibus-setup
Traceback (most recent call last):
  File "/usr/share/ibus/setup/main.py", line 710, in <module>
    setup = Setup()
  File "/usr/share/ibus/setup/main.py", line 120, in __init__
    self.__init_ui()
  File "/usr/share/ibus/setup/main.py", line 413, in __init_ui
    self.__init_hotkeys()
  File "/usr/share/ibus/setup/main.py", line 130, in __init_hotkeys
    self.__init_hotkey(name, label)
  File "/usr/share/ibus/setup/main.py", line 144, in __init_hotkey
    entry.set_text("; ".join(shortcuts))
AttributeError: 'NoneType' object has no attribute 'set_text'
$

Probably the affected upstream files have changed since the patch was
written.

So does anyone have time/interest to maintain this patch? Or should we
simply drop it? After all IBus seems to work fine (in focal) without it.

** Also affects: unity
   Importance: Undecided
       Status: New

** Changed in: ibus (Ubuntu)
   Importance: Medium => Low

-- 
You received this bug notification because you are a member of Ubuntu
Touch seeded packages, which is subscribed to ibus in Ubuntu.
https://bugs.launchpad.net/bugs/1870945

Title:
  ubuntu-disable-trigger.patch is effectively not enabled

Status in Unity:
  New
Status in ibus package in Ubuntu:
  In Progress

Bug description:
  ubuntu-disable-trigger.patch has effectively not been enabled since
  XDG_CURRENT_DESKTOP was turned into a list of values instead of a
  single value.

To manage notifications about this bug go to:
https://bugs.launchpad.net/unity/+bug/1870945/+subscriptions

-- 
Mailing list: https://launchpad.net/~touch-packages
Post to     : touch-packages@lists.launchpad.net
Unsubscribe : https://launchpad.net/~touch-packages
More help   : https://help.launchpad.net/ListHelp

Reply via email to