[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#1106244: unblock: fcitx-rime/0.3.2+git20230425-7



Package: release.debian.org
Control: affects -1 + src:fcitx-rime
X-Debbugs-Cc: fcitx-rime@packages.debian.org
User: release.debian.org@packages.debian.org
Usertags: unblock
X-Debbugs-Cc: byang@debian.org bird@b1az3.me
Severity: normal

Please unblock package fcitx-rime

[ Reason ]
As in https://bugs.debian.org/1106211 , the current
fcitx-rime/0.3.2+git20230425-6 in Debian Testing cannot be
launched due to an undefined symbol problem. This is due to
recent changes of librime library. The -7 upload
fixes this problem by cherry-pick the patch from Arch Linux
side.

The fcitx-rime upstream is now dead, and thus the patch cannot
be verified by its original author. Manual code review is
performed instead.

[ Impact ]
The fcitx-rime package as an input method cannot be used
by users.

[ Tests ]
Manually reviewed and manually tested in a Debian Sid and
a Debian Testing VM.

[ Risks ]
The risk level is minimum. The package cannot be further broken
anyway.

[ Checklist ]
  [X] all changes are documented in the d/changelog
  [X] I reviewed all changes and I approve them
  [X] attach debdiff against the package in testing

[ Other info ]
Please find the full debdiff in the attachment.

unblock fcitx-rime/0.3.2+git20230425-7


Thanks,
Boyuan Yang
diff -Nru fcitx-rime-0.3.2+git20230425/debian/changelog fcitx-rime-0.3.2+git20230425/debian/changelog
--- fcitx-rime-0.3.2+git20230425/debian/changelog	2025-03-03 12:15:48.000000000 -0500
+++ fcitx-rime-0.3.2+git20230425/debian/changelog	2025-05-21 13:16:57.000000000 -0400
@@ -1,3 +1,11 @@
+fcitx-rime (0.3.2+git20230425-7) unstable; urgency=high
+
+  * Team upload.
+  * debian/patches/fix-rime-process-key.patch: Fix incompatibility
+    with recent librime versions. (Closes: #1106211) 
+
+ -- Boyuan Yang <byang@debian.org>  Wed, 21 May 2025 13:16:57 -0400
+
 fcitx-rime (0.3.2+git20230425-6) unstable; urgency=medium
 
   * Team upload.
diff -Nru fcitx-rime-0.3.2+git20230425/debian/patches/fix-rime-process-key.patch fcitx-rime-0.3.2+git20230425/debian/patches/fix-rime-process-key.patch
--- fcitx-rime-0.3.2+git20230425/debian/patches/fix-rime-process-key.patch	1969-12-31 19:00:00.000000000 -0500
+++ fcitx-rime-0.3.2+git20230425/debian/patches/fix-rime-process-key.patch	2025-05-21 13:16:42.000000000 -0400
@@ -0,0 +1,32 @@
+From 383e6754238dfeaf816afa0f1f2fa5061f835f91 Mon Sep 17 00:00:00 2001
+From: Frederick Zhang <frederick888@tsundere.moe>
+Date: Sat, 8 Feb 2025 18:37:11 +1100
+Subject: [PATCH] Fix undefined symbol RimeProcessKey
+
+[1] https://github.com/rime/librime/pull/877
+
+Origin: https://gitlab.archlinux.org/archlinux/packaging/packages/fcitx-rime/-/blob/main/fix-rime-process-key.patch
+Bug-Debian: https://bugs.debian.org/1106211
+Signed-off-by: Frederick Zhang <frederick888@tsundere.moe>
+---
+ src/fcitx-rime.c | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/src/fcitx-rime.c b/src/fcitx-rime.c
+index 906bd81..c1df460 100644
+--- a/src/fcitx-rime.c
++++ b/src/fcitx-rime.c
+@@ -366,9 +366,9 @@ INPUT_RETURN_VALUE FcitxRimeGetCandWord(void* arg, FcitxCandidateWord* candWord)
+ }
+ 
+ boolean FcitxRimePaging(void* arg, boolean prev) {
+     FcitxRime *rime = (FcitxRime *)arg;
+-    boolean result = RimeProcessKey(rime->session_id, prev ? FcitxKey_Page_Up : FcitxKey_Page_Down, 0);
++    boolean result = rime->api->process_key(rime->session_id, prev ? FcitxKey_Page_Up : FcitxKey_Page_Down, 0);
+     if (result) {
+         FcitxRimeGetCandWords(rime);
+         FcitxUIUpdateInputWindow(rime->owner);
+     }
+-- 
+2.48.1
+
diff -Nru fcitx-rime-0.3.2+git20230425/debian/patches/series fcitx-rime-0.3.2+git20230425/debian/patches/series
--- fcitx-rime-0.3.2+git20230425/debian/patches/series	2024-12-09 13:15:30.000000000 -0500
+++ fcitx-rime-0.3.2+git20230425/debian/patches/series	2025-05-21 13:15:48.000000000 -0400
@@ -1 +1,2 @@
 0001-Fix-compatibility-with-librime-1.12.patch
+fix-rime-process-key.patch

Attachment: OpenPGP_signature.asc
Description: OpenPGP digital signature


Reply to: