[Pkg-ime-devel] Bug#862814: ibus-ui (systray icon) mis-detects Qt/KDE version

[Ralf Jung]

Hi,

> On Wed, May 17, 2017 at 02:03:22PM +0200, Ralf Jung wrote:
>> I can confirm that replacing the aforementioned piece of code by just
>>
>>   m_indicator.set_icon_full(icon_name, "");
>>
>> (i.e., removing the preprocessor conditional entirely) fixes the
>> problem:  The icon is properly shown in the systray.
> 
> Please send us a patch against current package

I attached what I dropped in debian/patches.

Kind regards,
Ralf
-------------- next part --------------
From: Ralf Jung <post at ralfj.de>
Date: Wed, 17 May 2017 13:56:19 +0200
X-Dgit-Generated: 1.5.14-2+local 854e73f39b613fda23e95975cf867d90827bd7fd
Subject: assume we always have a new enough Plasma+Qt


---

--- ibus-1.5.14.orig/ui/gtk3/panel.vala
+++ ibus-1.5.14/ui/gtk3/panel.vala
@@ -1372,15 +1372,7 @@ class Panel : IBus.PanelService {
                 m_status_icon.set_from_file(icon_name);
             }
             else if (m_icon_type == IconType.INDICATOR) {
-#if INDICATOR_ENGINE_ICON
                 m_indicator.set_icon_full(icon_name, "");
-#else
-                warning("plasma-workspace 5.2 or later is required to " +
-                        "show the absolute path icon %s. Currently check " +
-                        "qtbase 5.4 since there is no way to check " +
-                        "the version of plasma-workspace.", icon_name);
-                m_indicator.set_icon_full("ibus-engine", "");
-#endif
             }
         } else {
             string language = null;