FS#12356 - Please apply patch to SCIM for Java clients

Attached to Project: Arch Linux
Opened by Wenzhi Liang (lang2) - Friday, 05 December 2008, 23:36 GMT
Last edited by Eric Belanger (Snowman) - Monday, 16 March 2009, 19:37 GMT
Task Type Bug Report
Category Packages: Extra
Status Closed
Assigned To Eric Belanger (Snowman)
Aaron Griffin (phrakture)
Architecture All
Severity Medium
Priority Normal
Reported Version None
Due in Version Undecided
Due Date Undecided
Percent Complete 100%
Votes 0
Private No

Details

Description:
This is exactly the same bug as Ubuntu Bug #178742:

https://bugs.launchpad.net/ubuntu/+source/scim/+bug/178742

Basically, scim doesn't work in Java applications and the following patch is needed:

+diff -urNad scim-1.4.7~/modules/FrontEnd/scim_x11_frontend.cpp scim-1.4.7/modules/FrontEnd/scim_x11_frontend.cpp
+--- scim-1.4.7~/modules/FrontEnd/scim_x11_frontend.cpp 2007-06-26 14:31:52.000000000 +0000
++++ scim-1.4.7/modules/FrontEnd/scim_x11_frontend.cpp 2008-03-15 06:36:15.000000000 +0000
+@@ -1086,7 +1086,8 @@
+ // If the ic is not focused, then return.
+ if (!is_focused_ic (ic)) {
+ SCIM_DEBUG_FRONTEND(1) << "IC " << call_data->icid << " is not focused, focus it first.\n";
+- ims_set_ic_focus_handler (ims, (IMChangeFocusStruct *) call_data);
++ // ims_set_ic_focus_handler (ims, (IMChangeFocusStruct *) call_data);
++ return 1;
+ }
+
+ XKeyEvent *event = (XKeyEvent*) &(call_data->event);

Additional info:
* package version(s)
* config and/or log files etc.


Steps to reproduce:
This task depends upon

Closed by  Eric Belanger (Snowman)
Monday, 16 March 2009, 19:37 GMT
Reason for closing:  Fixed
Comment by Andreas Radke (AndyRTR) - Monday, 22 December 2008, 16:01 GMT
i'm not sure if we should keep maintaining the pkg in extra, it's orphaned for a while. maybe it's better maintained by native speakers in community/AUR. but usage is quiet high >3% acording Pierre stats.

following our patching rules i would let it up to the user to decide if they want to add it or not. it's not a major bug and not (yet) upstream included in a release. removing me from the assignies...
Comment by Aaron Griffin (phrakture) - Tuesday, 23 December 2008, 23:15 GMT
I'd be fine with this. scim is kind of important for some users, so I think we should keep it around. I just wish we had a maintainer that actually used it.
Comment by Andreas Radke (AndyRTR) - Wednesday, 24 December 2008, 10:15 GMT
So we can close this one with "won't implement" and it's documented here how to fix this minor issue?
Comment by Aaron Griffin (phrakture) - Wednesday, 24 December 2008, 17:46 GMT
Hmm. Would someone be willing to add a wiki page regarding "scim and java" that indicates these directions? The wiki is easier than the bug tracker, and it would allow us to get translations, which would be important for those users actually needing scim.
Comment by Eric Belanger (Snowman) - Tuesday, 30 December 2008, 03:00 GMT
I don't use scim but I could just go ahead and apply the patch if there's no objections.
Comment by Eric Belanger (Snowman) - Saturday, 28 February 2009, 04:03 GMT
Please test with scim-1.4.8-1 in [testing] to see if it's still a problem. See http://www.archlinux.org/pipermail/arch-general/2009-February/003805.html for more details.
Comment by Wenzhi Liang (lang2) - Saturday, 28 February 2009, 11:27 GMT
1.4.8 doesn't work with either jre or openjdk. Is it supposed to: fixed in upstream?
Comment by Eric Belanger (Snowman) - Sunday, 08 March 2009, 05:42 GMT
I've applied the patch to scim-1.4.8-2 in testing repo.
Comment by Wenzhi Liang (lang2) - Tuesday, 10 March 2009, 21:58 GMT
Tested 1.4.8-1 again with a fresh user acount and it works. So it was a setup issue I had. Please revert back to 1.4.8-1 as it was not patched and this can be close. Thanks!
Comment by Eric Belanger (Snowman) - Wednesday, 11 March 2009, 00:11 GMT
I removed that patch in scim-1.4.8-3. I'll close this bug report once it'll be moved to extra. BTW, did you also tested the other scim-related and arphic fonts packages in testing?
Comment by Wenzhi Liang (lang2) - Wednesday, 11 March 2009, 00:46 GMT
By scim-related, you meant:
testing/scim-anthy
testing/scim-chewing
testing/scim-tables
testing/scim-uim
?
Comment by Eric Belanger (Snowman) - Wednesday, 11 March 2009, 00:54 GMT Comment by Wenzhi Liang (lang2) - Wednesday, 11 March 2009, 20:41 GMT
Here's what I tested:

scim-anthy, scim-uim, scim-chewing: installed and checked the corresponding option is present in the menu.
arphic fonts: installed and verified that they work (as in can be used to display stuff). No way of telling whether it includes firefly or not.
Comment by Eric Belanger (Snowman) - Thursday, 12 March 2009, 00:21 GMT
Thanks.
We can keep firefly in the repo at least until it needs to be updated/fixed. We could check at that time it it's still useful or not.

Loading...