Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.

https://bugzilla.redhat.com/show_bug.cgi?id=583716

LI Rui Bin cheeseli@hotmail.com changed:

What |Removed |Added ---------------------------------------------------------------------------- Keywords| |EasyFix, Improvement, Patch Depends on| |583545

Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.

https://bugzilla.redhat.com/show_bug.cgi?id=583716

fujiwara tfujiwar@redhat.com changed:

What |Removed |Added ---------------------------------------------------------------------------- CC| |tfujiwar@redhat.com

--- Comment #2 from fujiwara tfujiwar@redhat.com 2010-04-19 23:09:58 EDT --- I also guess a enable/disable shell environment value might be useful while I'm not sure if all people always want to use Kimpanel when it's installed.

Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.

https://bugzilla.redhat.com/show_bug.cgi?id=583716

LI Rui Bin cheeseli@hotmail.com changed:

What |Removed |Added ---------------------------------------------------------------------------- Keywords| |FutureFeature Version|13 |rawhide

--- Comment #4 from LI Rui Bin cheeseli@hotmail.com 2010-04-20 00:31:01 EDT --- I propose to make things out of box. And people who use KDE and do not want to use Kimpanel, then can just remove the Kimpanel package.

More discussion in KDE SIG may be necessary, and as my personal experience, the IBus backend of Kimpanel is not yet stable enough. So this bug may be considered as FutureFeature by now.

(In reply to comment #2)

I also guess a enable/disable shell environment value might be useful while I'm not sure if all people always want to use Kimpanel when it's installed.

Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.

https://bugzilla.redhat.com/show_bug.cgi?id=583716

LI Rui Bin cheeseli@hotmail.com changed:

What |Removed |Added ---------------------------------------------------------------------------- Attachment #407724|0 |1 is obsolete| |

--- Comment #6 from LI Rui Bin cheeseli@hotmail.com 2010-04-20 00:41:52 EDT --- Created an attachment (id=407725) --> (https://bugzilla.redhat.com/attachment.cgi?id=407725) Additional runtime detecting in ibus.conf for Kimpanel

Test for any kind of KDE session including something like 'openbox-kde'.

Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.

https://bugzilla.redhat.com/show_bug.cgi?id=583716

LI Rui Bin cheeseli@hotmail.com changed:

What |Removed |Added ---------------------------------------------------------------------------- Attachment #407725|0 |1 is obsolete| |

--- Comment #8 from LI Rui Bin cheeseli@hotmail.com 2010-04-20 01:15:27 EDT --- Created an attachment (id=407728) --> (https://bugzilla.redhat.com/attachment.cgi?id=407728) Additional runtime detecting in ibus.conf for Kimpanel

Redirect output to /dev/null instead

Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.

https://bugzilla.redhat.com/show_bug.cgi?id=583716

Peng Huang phuang@redhat.com changed:

What |Removed |Added ---------------------------------------------------------------------------- Flag| |needinfo?(cheeseli@hotmail. | |com)

--- Comment #10 from Peng Huang phuang@redhat.com 2010-04-20 04:12:07 EDT --- which package kimpanel is included in?

Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.

https://bugzilla.redhat.com/show_bug.cgi?id=583716

LI Rui Bin cheeseli@hotmail.com changed:

What |Removed |Added ---------------------------------------------------------------------------- Flag|needinfo?(cheeseli@hotmail. | |com), needinfo? |

--- Comment #12 from LI Rui Bin cheeseli@hotmail.com 2010-04-20 04:40:11 EDT --- Refer to: https://bugzilla.redhat.com/show_bug.cgi?id=583545 This bug depends on that one. Kimpanel is not yet included in any package by now. (In reply to comment #10)

which package kimpanel is included in?

Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.

https://bugzilla.redhat.com/show_bug.cgi?id=583716

Chen Lei supercyper@163.com changed:

What |Removed |Added ---------------------------------------------------------------------------- CC| |supercyper@163.com

--- Comment #14 from Chen Lei supercyper@163.com 2010-04-20 05:31:06 EDT --- test -f /usr/share/ibus/ui/panel.py -a "x$DESKTOP_SESSION" = "xkde"

is enough.

Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.

https://bugzilla.redhat.com/show_bug.cgi?id=583716

--- Comment #16 from LI Rui Bin cheeseli@hotmail.com 2010-04-20 06:37:01 EDT --- Because there are some kde variations like openbox-kde, so we should test sub-string and not identity. (In reply to comment #14)

test -f /usr/share/ibus/ui/panel.py -a "x$DESKTOP_SESSION" = "xkde"

is enough.

Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.

https://bugzilla.redhat.com/show_bug.cgi?id=583716

LI Rui Bin cheeseli@hotmail.com changed:

What |Removed |Added ---------------------------------------------------------------------------- Attachment #407728|0 |1 is obsolete| |

--- Comment #18 from LI Rui Bin cheeseli@hotmail.com 2010-04-21 10:38:22 EDT --- Created an attachment (id=408091) --> (https://bugzilla.redhat.com/attachment.cgi?id=408091) Additional runtime detecting in ibus.conf for Kimpanel

Patch updated.

Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.

https://bugzilla.redhat.com/show_bug.cgi?id=583716

Robin Lee robinlee.sysu@gmail.com changed:

What |Removed |Added ---------------------------------------------------------------------------- Attachment #408091|0 |1 is obsolete| |

--- Comment #20 from Robin Lee robinlee.sysu@gmail.com 2010-04-22 11:09:12 EDT --- Created an attachment (id=408353) --> (https://bugzilla.redhat.com/attachment.cgi?id=408353) Additional runtime detecting in ibus.conf for Kimpanel

panel.py moved

Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.

https://bugzilla.redhat.com/show_bug.cgi?id=583716

--- Comment #22 from fujiwara tfujiwar@redhat.com 2010-04-22 21:00:25 EDT --- (In reply to comment #21)

I think 'ibus-*.conf' may not be a good pattern for third-party additional setting files, because this pattern has special meaning for imsettings.

I don't understand what is the special meaning. There are many updates so it looks wrong to update ibus.conf directly to describe a specific UI(kimpanel) in upstream because it means you may ask to update it in the future again. The previous idea means to have a fix to support general ibus arguments. The /etc/X11/xinit/xinput.d could be for XIM but not a specific IM setting so the idea is the main conf specifies the XIM name and I also think it might be a similar way with xim.conf. The directory of the sub conf is an idea but also /usr/share/ibus might be another option.

or I also think just simply will not fix. It would be no big problem to add an additional main conf for kimpanel. Users also can add $HOME/.xinputrc instead.

Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.

https://bugzilla.redhat.com/show_bug.cgi?id=583716

--- Comment #24 from Kevin Kofler kevin@tigcc.ticalc.org 2010-04-23 11:44:35 EDT --- The idea of the pluggable ibus-*.conf scriptlets for things like kimpanel to plug into is interesting. I'd suggest putting them somewhere other than /etc/X11/xinit/xinput.d/ so they don't interfere with toplevel Xinput scriptlets though. Maybe /usr/share/ibus/ibus.conf.d/*.conf?

What's sure is that we need some solution, whether it's pluggable scriptlets or the patch to ibus.conf itself, for kimpanel to work as expected.

Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.

https://bugzilla.redhat.com/show_bug.cgi?id=583716

--- Comment #26 from fujiwara tfujiwar@redhat.com 2010-05-13 23:16:45 EDT --- BTW, I think the bugfix timeline will be set up after bug 583545 is fixed.

Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.

https://bugzilla.redhat.com/show_bug.cgi?id=583716

Fedora Admin XMLRPC Client fedora-admin-xmlrpc@redhat.com changed:

What |Removed |Added ---------------------------------------------------------------------------- AssignedTo|shawn.p.huang@gmail.com |tfujiwar@redhat.com

--- Comment #27 from Fedora Admin XMLRPC Client fedora-admin-xmlrpc@redhat.com 2010-08-02 02:17:32 EDT --- This package has changed ownership in the Fedora Package Database. Reassigning to the new owner of this component.

https://bugzilla.redhat.com/show_bug.cgi?id=583716 Bug 583716 depends on bug 583545, which changed state.

Bug 583545 Summary: Enable building Kimpanel and make a subpackage for it https://bugzilla.redhat.com/show_bug.cgi?id=583545

What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |CLOSED Resolution|--- |CURRENTRELEASE

https://bugzilla.redhat.com/show_bug.cgi?id=583716

Rex Dieter rdieter@math.unl.edu changed:

What |Removed |Added ---------------------------------------------------------------------------- Blocks|1135101 (F22Target-kde) |1250207 (F23Target-kde)

Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1135101 [Bug 1135101] Fedora 22 Target KDE Tracker https://bugzilla.redhat.com/show_bug.cgi?id=1250207 [Bug 1250207] Fedora 23 Target KDE Tracker