Bug 295331
Summary: | Return does not work with XIM | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Warren Togami <wtogami> |
Component: | xchat | Assignee: | Kevin Kofler <kevin> |
Status: | CLOSED ERRATA | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | rawhide | CC: | caillon, dominik, eng-i18n-bugs, fedora, kevin, petersen, peterzelezny, phuang |
Target Milestone: | --- | Keywords: | i18n, Reopened |
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | 2.8.4-6.fc7 | Doc Type: | Bug Fix |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2007-11-29 01:39:24 UTC | Type: | --- |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: | |||
Bug Depends On: | |||
Bug Blocks: | 235704 |
Description
Warren Togami
2007-09-18 18:59:26 UTC
--enable-spell=none --enable-spell=static Tried building with these options, they no longer fix the problem. Seems that upstream made it worse. =( Adding i18n group. They might be interested in knowing that IM is broken in xchat in both F7 and F8. CCing upstream too. From https://bugzilla.redhat.com/process_bug.cgi#c16 : I had the update tested by someone using the Korean SCIM input method and it worked for him. (He just tried again with 2.8.4-2.fc7 and it still works for him.) I wouldn't have pushed it otherwise. So "completely" is probably the wrong word, still this doesn't sound good. Does this break only some input methods and not others? Interesting... something else weird is going on. /desktop/gnome/interface/show_input_method_menu Set this to on in order to see and change the current IM setting. Starting GTK+ apps in F8 defaults with "X Input Method" instead of "SCIM Bridge Input Method". If you manually select the latter, then xchat works as expected. There might actually be two bugs going on here: 1) GNOME or GTK+ isn't using scim-bridge by default on F8. 2) xchat is broken with XIM specifically. #2 is quite bad itself. scim-bridge problem filed in Bug #295371. Is this really a high-priority F8Blocker now that things are being fixed so SCIM is picked by default again? Isn't XIM obsolescent anyway? I'm not saying it shouldn't be fixed, but I'm wondering how urgent this really is. Downgrading to F8Target. I don't know enough about IMs to help much, but it's probably related to: http://sf.net/tracker/index.php?func=detail&aid=896968&group_id=239&atid=100239 I just tried with current rawhide and I can use scim ok in xchat. Is this ok now? See Warren's comment #4, SCIM works, XIM doesn't. (I haven't confirmed this myself though.) Ah, yes - XIM looks ok to me too though. No, still broken. Procedure ========= 1) LANG=en_US.UTF-8 xchat 2) Right-click in the text entry box, switch to X Input Method 3) Type a line 4) Hit ENTER 5) ENTER didn't happen. Ok. How about with "GTK_IM_MODULE=xim xchat"? That seems to work for me. Ah I see what you mean: reproduced completely. Please ignore comment 13. Anyway I suspect input is broken with XIM anyway so probably not a good idea to use it with xchat currently, but that doesn't help users using xim only input methods of course... I found xchat-2.4.3-im_context_filter_keypress.patch causes this problem. What is this patch for? It seems that this patch is for xchat-2.4.3. Maybe we could drop it now. I also found xchat-2.4.4-unrealize.patch can be dropped too. This patch has been committed to. I think we should review every old patches now! (In reply to comment #16) > I also found xchat-2.4.4-unrealize.patch can be dropped too. This patch has been > committed to upstream. I think we should review every old patches now! ~~~~~~~~~~ Sorry for typo. :P (In reply to comment #15) > I found xchat-2.4.3-im_context_filter_keypress.patch causes this problem. What > is this patch for? It seems that this patch is for xchat-2.4.3. Maybe we could > drop it now. See Bug#144588. it might be applied wrongly because it was made for the older release. please make sure if xchat without that patch works now for #144588 too. I'll drop xchat-2.4.4-unrealize.patch, it's indeed obsolete, though having it applied a second time should be harmless in that case. As for xchat-2.4.3-im_context_filter_keypress.patch: I see there's a return breaking out of a while loop. Can one of you test what happens if you change the return TRUE; in the patch to a continue;? Uh, just a continue won't work, because it would also skip the move to the next key: last = kb; kb = kb->next; I'm also not sure it's the right thing to do anyway. I don't think the patch is being wrongly applied, I looked at the history of fkeys.c in the XChat CVS and I don't see anything changed in the context of the patch. (In reply to comment #18) > > See Bug#144588. it might be applied wrongly because it was made for the older > release. please make sure if xchat without that patch works now for #144588 too. > We don't use iiimf now. I can not reproduce this problem. It seems work fine with SCIM. And XIM? (I know it's not the default, but breaking one thing for XIM while fixing another would be bad. ;-) ) I'm going ahead and killing the broken xchat-2.4.3-im_context_filter_keypress.patch, because the patch apparently breaks more than it fixes when applied to the current XChat. xchat-2.8.4-6.fc7 has been pushed to the Fedora 7 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update xchat' xchat-2.8.4-6.fc7 has been pushed to the Fedora 7 stable repository. If problems still persist, please make note of it in this bug report. 2.8.4-6.fc7 breaks scim-anthy for me. I could correct the converted-but-not-committed string with cursor keys (up/down) before, but now, if I press up or down I just move backward or forward in irc input history. This needs to be fixed, please. Return key worked fine before, too. Hmmm, the update has been in Rawhide for over a month, in F8 since its release and in F7 updates-testing for almost a month and it's now that the problem reports show up? :-( I don't blame you in particular, I know you may be too busy or lack the hardware to test updates-testing and/or Rawhide, but the general lack of testers. All I can offer to you for now is to reinstall 2.8.4-5.fc7 with rpm -Uvh --oldpackage. (You know where to get it from, right?) I hope we can make a working im_context_filter_keypress patch (i.e. one which fixes this without breaking XIM again) as soon as possible. I'd really appreciate any help I can get from the RH i18n team there. Well it's not your fault, either. It's my fault for not following updates-testing. :) I just thought you should know. It's been confirmed by one of my friends on IRC, too. Feel free to let me know if you have something new to test regarding this issue. (In reply to comment #27) > 2.8.4-6.fc7 breaks scim-anthy for me. I could correct the > converted-but-not-committed string with cursor keys (up/down) before, but now, > if I press up or down I just move backward or forward in irc input history. > This needs to be fixed, please. Return key worked fine before, too. I just tried with 2.8.4-11.fc9 in rawhide with scim-bridge and that works fine. Which immodule are you using? He already said it, it's scim-anthy:
> 2.8.4-6.fc7 breaks scim-anthy for me.
(In reply to comment #31) > He already said it, it's scim-anthy: scim-anthy is IMEngine used by scim. I am testing with that too. 2.8.4-11 is still broken, but differently: I still can't choose between various anthy's matches using up/down, but it I don't move backward or forward in irc input history anymore when in anthy mode. I think we should move this discussion to the bug report I've opened specifically for this new issue: bug 429104. |