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

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



Hi,

在 2025-05-22四的 08:35 +0200,Sebastian Ramacher写道:
> Control: tags -1 moreinfo
> 
> On 2025-05-21 20:38:16 -0400, Boyuan Yang wrote:
> > 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.
> 
> So librime dropped a symbol without doing a transition?
> 

I reviewed the source code history and it looks like the case. What do you
think would be the best way forward?

* I can prepare an upload of librime to revert the removal of the symbol,
and then request an unblock.

* I am comfortable with having src:fcitx-rime drop out of Debian Testing and
Debian 13, as its upstream is dead now and the whole src:fcitx family will
nevertheless be scheduled for removal in Debian 14.

* It is also okay if you accept this ad-hoc patch for src:fcitx-rime, as
the removed symbol from librime is long deprecated. Other than src:fcitx-rime,
no other consumer, either inside or outside Debian, is known other than
src:fcitx-rime.

Many thanks,
Boyuan Yang


> > 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: signature.asc
Description: This is a digitally signed message part


Reply to: