Bug 1231165 - FTFBS: perl-Coro against perl-5.22
Summary: FTFBS: perl-Coro against perl-5.22
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: perl-Coro
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Petr Pisar
QA Contact: Fedora Extras Quality Assurance
URL: http://www.nntp.perl.org/group/perl.p...
Whiteboard:
Depends On:
Blocks: 1231260
TreeView+ depends on / blocked
 
Reported: 2015-06-12 10:13 UTC by Nils Philippsen
Modified: 2016-05-24 10:56 UTC (History)
3 users (show)

Fixed In Version: perl-Coro-6.43-3.fc23
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2015-06-18 14:08:49 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)
Do not edit Perl's vtable (7.18 KB, patch)
2016-05-24 10:56 UTC, Petr Pisar
no flags Details | Diff


Links
System ID Private Priority Status Summary Last Updated
CPAN 101063 0 None None None Never

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.


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