https://bugzilla.redhat.com/show_bug.cgi?id=2240458
Bug ID: 2240458 Summary: Perl 5.38 breaks Irssi locale Product: Fedora Version: 39 Hardware: x86_64 URL: https://github.com/irssi/scripts.irssi.org/issues/857 OS: Linux Status: NEW Component: perl Severity: medium Assignee: jplesnik@redhat.com Reporter: mjuszkie@redhat.com QA Contact: extras-qa@fedoraproject.org CC: iarnell@gmail.com, jplesnik@redhat.com, kasal@ucw.cz, mmaslano@redhat.com, mspacek@redhat.com, perl-devel@lists.fedoraproject.org, ppisar@redhat.com, psabata@redhat.com, rhughes@redhat.com, spotrh@gmail.com Target Milestone: --- Classification: Fedora
After upgrade to Fedora 39 beta one of scripts in irssi (irc client) broke. After checking what is going on (I do not know Perl) I found out that length of string was reported with negative sign.
Opened a ticket at Irssi github account https://github.com/irssi/scripts.irssi.org/issues/857 and went through debugging a bit.
Turned out that Perl 5.38 drops locale settings when embedded in C. There is a bug for it already: https://github.com/Perl/perl5/issues/21366
I do not know which solution should be taken to make it work properly. For irssi script I have hacky workaround. But would be nice to not have such bug on release of new Fedora version.
Reproducible: Always
Steps to Reproduce: 1. set locale to any UTF-8 one (I use pl_PL.UTF-8) 2. run irssi 3. run "/script exec print Text::CharWidth::mbswidth("─")" command
Actual Results: Should return "1"
Expected Results: Returns "-1"
irssi-1.4.4-5.fc39.x86_64 perl-5.38.0-500.fc39.x86_64
https://bugzilla.redhat.com/show_bug.cgi?id=2240458
Jitka Plesnikova jplesnik@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED
--- Comment #1 from Jitka Plesnikova jplesnik@redhat.com --- Thank you for report.
The bug is fixed in perl development version and it is requested to be included in perl-5.38.1.
There was lots of changes in 'locale.c'. I will check how to backported it to Perl 5.38.
https://bugzilla.redhat.com/show_bug.cgi?id=2240458
--- Comment #2 from Marcin Juszkiewicz mjuszkie@redhat.com --- Fedora 39 is not released yet, why not bump to 5.38.1 instead of backporting?
https://bugzilla.redhat.com/show_bug.cgi?id=2240458
Petr Pisar ppisar@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Link ID| |Github | |Perl/perl5/issues/21366
--- Comment #3 from Petr Pisar ppisar@redhat.com --- Because 5.38.1 does not exist yet:
$ git branch -a |grep origin/maint-5\. remotes/origin/maint-5.004 remotes/origin/maint-5.005 remotes/origin/maint-5.10 remotes/origin/maint-5.12 remotes/origin/maint-5.14 remotes/origin/maint-5.16 remotes/origin/maint-5.18 remotes/origin/maint-5.20 remotes/origin/maint-5.20-votes remotes/origin/maint-5.22 remotes/origin/maint-5.24 remotes/origin/maint-5.26 remotes/origin/maint-5.28 remotes/origin/maint-5.28-sep18 remotes/origin/maint-5.30 remotes/origin/maint-5.32 remotes/origin/maint-5.34 remotes/origin/maint-5.36 remotes/origin/maint-5.6 remotes/origin/maint-5.8
https://bugzilla.redhat.com/show_bug.cgi?id=2240458
--- Comment #4 from Jitka Plesnikova jplesnik@redhat.com --- It seems that reverting patch mention in comment [1] should be used as workaround. I tried it and irssi which I rebuilt with this perl worked as expected.
[1] https://github.com/Perl/perl5/issues/21366#issuecomment-1732560904
https://bugzilla.redhat.com/show_bug.cgi?id=2240458
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |MODIFIED
--- Comment #5 from Fedora Update System updates@fedoraproject.org --- FEDORA-2023-b182207d72 has been submitted as an update to Fedora 39. https://bodhi.fedoraproject.org/updates/FEDORA-2023-b182207d72
https://bugzilla.redhat.com/show_bug.cgi?id=2240458
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|MODIFIED |ON_QA
--- Comment #6 from Fedora Update System updates@fedoraproject.org --- FEDORA-2023-b182207d72 has been pushed to the Fedora 39 testing repository. Soon you'll be able to install the update with the following command: `sudo dnf upgrade --enablerepo=updates-testing --refresh --advisory=FEDORA-2023-b182207d72` You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2023-b182207d72
See also https://fedoraproject.org/wiki/QA:Updates_Testing for more information on how to test updates.
https://bugzilla.redhat.com/show_bug.cgi?id=2240458
Dennis Brendel dbrendel@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |mfabian@redhat.com
--- Comment #7 from Dennis Brendel dbrendel@redhat.com --- *** Bug 2241148 has been marked as a duplicate of this bug. ***
https://bugzilla.redhat.com/show_bug.cgi?id=2240458
Dennis Brendel dbrendel@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |kkretschmer@gmail.com
--- Comment #8 from Dennis Brendel dbrendel@redhat.com --- *** Bug 2240460 has been marked as a duplicate of this bug. ***
https://bugzilla.redhat.com/show_bug.cgi?id=2240458
Fedora Update System updates@fedoraproject.org changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ON_QA |CLOSED Fixed In Version| |perl-5.38.0-501.fc39 Resolution|--- |ERRATA Last Closed| |2023-11-03 18:26:42
--- Comment #9 from Fedora Update System updates@fedoraproject.org --- FEDORA-2023-b182207d72 has been pushed to the Fedora 39 stable repository. If problem still persists, please make note of it in this bug report.
perl-devel@lists.fedoraproject.org