Bug 2226507 - weechat: FTBFS in Fedora rawhide/f39
Summary: weechat: FTBFS in Fedora rawhide/f39
Keywords:
Status: ASSIGNED
Alias: None
Product: Fedora
Classification: Fedora
Component: weechat
Version: 39
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Paul P Komkoff Jr
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: F39FTBFS 2222649 F40FTBFS, RAWHIDEFTBFS
TreeView+ depends on / blocked
 
Reported: 2023-07-25 20:26 UTC by Fedora Release Engineering
Modified: 2023-08-17 15:11 UTC (History)
7 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed:
Type: ---
Embargoed:


Attachments (Terms of Use)
build.log (32.00 KB, text/plain)
2023-07-25 20:26 UTC, Fedora Release Engineering
no flags Details
root.log (32.00 KB, text/plain)
2023-07-25 20:26 UTC, Fedora Release Engineering
no flags Details
state.log (1003 bytes, text/plain)
2023-07-25 20:26 UTC, Fedora Release Engineering
no flags Details

Description Fedora Release Engineering 2023-07-25 20:26:18 UTC
weechat failed to build from source in Fedora rawhide/f39

https://koji.fedoraproject.org/koji/taskinfo?taskID=103752472


For details on the mass rebuild see:

https://fedoraproject.org/wiki/Fedora_39_Mass_Rebuild
Please fix weechat at your earliest convenience and set the bug's status to
ASSIGNED when you start fixing it. If the bug remains in NEW state for 8 weeks,
weechat will be orphaned. Before branching of Fedora 40,
weechat will be retired, if it still fails to build.

For more details on the FTBFS policy, please visit:
https://docs.fedoraproject.org/en-US/fesco/Fails_to_build_from_source_Fails_to_install/

Comment 1 Fedora Release Engineering 2023-07-25 20:26:28 UTC
Created attachment 1979852 [details]
build.log

file build.log too big, will only attach last 32768 bytes

Comment 2 Fedora Release Engineering 2023-07-25 20:26:31 UTC
Created attachment 1979853 [details]
root.log

file root.log too big, will only attach last 32768 bytes

Comment 3 Fedora Release Engineering 2023-07-25 20:26:34 UTC
Created attachment 1979854 [details]
state.log

Comment 4 Tim Flink 2023-08-15 21:58:21 UTC
I've been working on this and I have a package for the latest 4.0.3 that builds but fails the tests under F39 and F40. From the upstream bug report [1], it sounds like any non-ascii characters will not display correctly even if I disabled the tests. It seems like there was a change in perl 5.38 which forces a locale change when embedding perl in C [2] like weechat does.

I've confirmed this by building in F38 and F37 (perl 5.36) - the test suite passes with no issues. It only fails with F39 and F40 as those have perl 5.38. If I build with -DENABLE_PERL=NO, the build and test also finish under F40.

It seems like we're left with some less than ideal options:

1. Leave the weechat package as FTBFS until the upstream issue is solved
2. Solve the issue in weechat code even though they seem to want to wait for upstream perl to fix it
3. Ask the perl packagers to patch out the upstream change [3] and hope that doesn't break anything.
4. Disable the tests and ship something that can't handle more than ASCII and will fail in strange ways if weechat ever sees a non-ASCII character
5. Disable the perl bindings until the upstream issue has been resolved

My instinct is to go with 5 and try to figure out how to announce this change in the build for anyone who uses the perl bindings. Are there other thoughts or suggestions? I still need to do more testing of the perl-disabled build to be more sure there aren't big side-effects but wanted to get the question out there for more opinions.

[1] https://github.com/weechat/weechat/issues/1996
[2] https://github.com/Perl/perl5/issues/21366
[3] https://github.com/Perl/perl5/commit/7af2d2037375d58e700f9e1b217efb2c4db66133

Comment 5 Fedora Release Engineering 2023-08-16 08:05:32 UTC
This bug appears to have been reported against 'rawhide' during the Fedora Linux 39 development cycle.
Changing version to 39.

Comment 6 Mamoru TASAKA 2023-08-16 09:38:24 UTC
Note that there is perl5_38 issue (wrt LC_CTYPE) on F-40/39, even if trying to build weechat 4.0.3 on F-38,
I see test failure on i686 and aarch64.

Tim, does your srpm builds on i686 / aarch64?

Comment 7 Tim Flink 2023-08-16 15:42:08 UTC
(In reply to Mamoru TASAKA from comment #6)
> Note that there is perl5_38 issue (wrt LC_CTYPE) on F-40/39, even if trying
> to build weechat 4.0.3 on F-38,
> I see test failure on i686 and aarch64.
> 
> Tim, does your srpm builds on i686 / aarch64?

It turns out that my srpm does not pass tests on i686 or aarch64 [1]. Is this something you're tracking or working on?


[1] https://copr.fedorainfracloud.org/coprs/tflink/weechat-testing/build/6306016/

Comment 8 Mamoru TASAKA 2023-08-17 03:10:53 UTC
Yes, these failures on i686 or aarch64 in [1] are the same failures I am also seeing,
however I have not tried to investigate the cause yet.

Comment 9 Mamoru TASAKA 2023-08-17 15:11:27 UTC
Current status:

* For F-40/F-39 perl5_38 issue, if we only care for making test pass, I wrote the workaround as https://github.com/weechat/weechat/issues/1996#issuecomment-1680146238
* For F-38 aarch64 test failure, the cause is now identified, with possible solution: https://github.com/weechat/weechat/issues/1997
* For F-38 i686 test failure, well, firstly I am not sure if weechat on Fedora is going to updated to 4.0.3 on stable branches (i.e. on F-38 and below).
  Then if we are to upgrade to 4.0.3 on Fedora on F-39/40 only, then I am not sure if we should investigate i686 test failure.
  Currently Fedora encourages to drop i686 support:
  https://fedoraproject.org/wiki/Changes/EncourageI686LeafRemoval
  and the upstream probably is not interested on 32bit.


Note You need to log in before you can comment on or make changes to this bug.