Bug 1231165

Summary: FTFBS: perl-Coro against perl-5.22
Product: [Fedora] Fedora Reporter: Nils Philippsen <nphilipp>
Component: perl-CoroAssignee: Petr Pisar <ppisar>
Status: CLOSED RAWHIDE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: rawhideCC: mmaslano, perl-devel, ppisar
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
URL: http://www.nntp.perl.org/group/perl.perl5.porters/2015/05/msg228257.html
Whiteboard:
Fixed In Version: perl-Coro-6.43-3.fc23 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2015-06-18 14:08:49 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1231260    
Attachments:
Description Flags
Do not edit Perl's vtable none

Description Nils Philippsen 2015-06-12 10:13:03 UTC 
Description of problem:
I just found that a couple of packages didn't update on my Rawhide machine, because perl-Coro hasn't been rebuilt against perl-5.22 yet. The rebuild failed[1], apparently because the module uses perl internals that have changed between 5.20 and 5.22[2].

[1] http://koji.fedoraproject.org/koji/taskinfo?taskID=9996895
[2] https://groups.google.com/forum/#!topic/perl.perl5.porters/B0YPUPtyOMo

Version-Release number of selected component (if applicable):
perl-Coro-6.43-2.fc23
perl-5.22.0-344.fc23
Comment 1 Petr Pisar 2015-06-12 13:00:10 UTC 
Thank you for the report. I'm aware about it. There was an opinion to fix perl instead of Coro on p5p list. However, I don't believe anything of it happens soon. I recommend you to drop perl(Coro) from your dependencies if you can.
Comment 2 Petr Pisar 2015-06-12 13:34:56 UTC 
Reading the commit that changed  the constness (c910fead7893fe9700031ee59de6b904260b5d69) and the linked discussion, I think that reverting the commit would move the data structure from read-only TEXT segment to BSS/DATA writable segment. Question is if would break ABI or wouldn't.
Comment 3 Petr Pisar 2015-06-18 12:53:40 UTC 
I will revert the c910fead7893fe9700031ee59de6b904260b5d69 temporarily as it should not break anything. Only the perl thread RSS will be few kilobytes larger. I will also apply post-5.22.0 73949fca082fe50bf47755c5ffa328259057ae36 commit which restores another part of API.

These changes will allow perl-Coro to work in Fedora 23. I hope Coro upstream will develop a fix so we will be able to drop the revert. Otherwise we will drop Coro.
Comment 4 Petr Pisar 2016-05-24 10:56:10 UTC 
Created attachment 1160980 [details]
Do not edit Perl's vtable

This is a proposed fix from <http://www.nntp.perl.org/group/perl.perl5.porters/2016/05/msg236178.html> thread.