Bleadperl v5.25.0-71-g5d32d26 breaks ZEFRAM/Array-Base-0.005.tar.gz #15335
Comments
From @andkbisect commit 5d32d26 enable PERL_OP_PARENT by default. diagnostics Compilation fails due missing op_sibling also affected ZEFRAM/Array-Base-0.005.tar.gz perl -V Summary of my perl5 (revision 5 version 25 subversion 1) configuration: Characteristics of this binary (from libperl): -- |
From @jkeenanOn Wed May 18 12:20:25 2016, andreas.koenig.7os6VVqR@franz.ak.mind.de wrote:
Is this change to the internals the kind that we feel strongly enough to want to ask the authors below to revise their code?
-- |
|
The RT System itself - Status changed from 'new' to 'open' |
From @cpansproutOn Thu May 19 18:01:57 2016, jkeenan wrote:
I certainly think so, but that is just one opinion. -- Father Chrysostomos |
From @ilmari"Father Chrysostomos via RT" <perlbug-followup@perl.org> writes:
I think so too. Conversion should be as trivial as adding conditional http://perl5.git.perl.org/perl.git/blob/v5.25.0:/op.h#l1005 And then replacing uses of ->op_sibling with calls to the appropriate -- |
From @andkAlso affected: CLKAO/invoker-0.34.tar.gz |
From @iabynOn Fri, May 20, 2016 at 09:12:45AM +0100, Dagfinn Ilmari Mannsåker wrote:
Note that I had earlier solicited opinions as to whether this should be http://nntp.perl.org/group/perl.perl5.porters/235635 -- |
From @demerphqOn 20 May 2016 at 10:12, Dagfinn Ilmari Mannsåker <ilmari@ilmari.org> wrote:
I plan to do this to Sereal soon. Thanks for the instructions! Yves -- |
From @ilmaridemerphq <demerphq@gmail.com> writes:
FYI: These macros have been added to Devel::PPPort as of version 3.33. -- |
From @jkeenanOn Wed, 18 May 2016 19:20:25 GMT, andreas.koenig.7os6VVqR@franz.ak.mind.de wrote:
Contributors: Can we get an update on the status of this BBC ticket? Thank you very much. |
From @iabynOn Tue, Dec 06, 2016 at 01:48:15PM -0800, James E Keenan via RT wrote:
Of the distributions listed in this ticket, the following appear to have CHOCOLATE/autobox-2.83.tar.gz ADUITSIS/IPC-Mmap-0.21.tar.gz while the following haven't had a new release yet: columns are: 02/2012 #117385 ZEFRAM/Array-Base-0.005.tar.gz Several belong to list members, so would be good to hear whether they The original intention was to enable PERL_OP_PARENT by default early in the -- |
From [email protected]I have mine mostly updated in my private repos, such that they compile -zefram |
From @karenetheridgere List-Gather (and also Cond-Expr and Sub-Call-Tail, which are not listed On Wed, Dec 7, 2016 at 4:07 AM, Dave Mitchell <davem@iabyn.com> wrote:
|
From @iabynOn Fri, Dec 09, 2016 at 12:13:12PM +0000, Dave Mitchell wrote:
I don't think this made it to the p5p list since the diffs included two -- |
From @iabynOn Wed, Dec 07, 2016 at 12:07:25PM +0000, Dave Mitchell wrote:
[snip]
01/2015 #117254 ETHER/Cond-Expr-0.04.tar.gz
If we're to back out PERL_OP_PARENT by default for 5.26.0, we need to do I privately emailed to Karen fixes for List-Gather and Cond-Expr, and -- |
From @pjcjOn Mon, Mar 13, 2017 at 12:29:24PM +0000, Dave Mitchell wrote:
I *will* get a new release of Devel::Cover out soon incorporating this -- |
From @xsawyerxOn 03/13/2017 01:29 PM, Dave Mitchell wrote:
Devel-MAT has a fix and just needs a merge and release. UNIVERSAL-ref's last release is from 2011. (September 2016, half a year Would having Devel-MAT, Devel-Cover, and ZEFRAM and ETHER dists be |
From @jkeenanOn Tue, 14 Mar 2017 12:08:05 GMT, xsawyerx@gmail.com wrote:
All the CPAN authors have been notified and many already have a fix in place or in development. I see no need to revert this change. We have more contentious changes in the works, so we shouldn't spend any more tuits on this one. Thank you very much. -- |
From @iabynOn Tue, Mar 14, 2017 at 05:35:32AM -0700, James E Keenan via RT wrote:
This thread concerns whether PERL_OP_PARENT should be kept enabled by Karen, are the patches I emailed you a while ago suitable? Are you likely 01/2015 #116694 ETHER/List-Gather-0.12.tar.gz in time for 5.26.0? Zefram, you said a while ago that you had fixes which would be released as 02/2012 #117385 ZEFRAM/Array-Base-0.005.tar.gz Paul, I understand a fix is in for Devel-MAT. Is there likely to be a As for the remaining distributions broken by PERL_OP_PARENT, here is 05/2015 github #4 TOKUHIROM/B-Tap-0.15.tar.gz 02/2012 #117387 RURBAN/optimizer-0.08.tar.gz 08/2012 #117338 RUZ/Sub-StopCalls-0.02.tar.gz 01/2011 #118008 JJORE/UNIVERSAL-ref-0.14.tar.gz 02/2013 #117215 CLKAO/invoker-0.34.tar.gz Does anyone consider any of those distros important enough that we should -- |
From @leonerdOn Mon, 20 Mar 2017 16:55:14 +0000
Hello. Hmm - this is the first I'm aware of an issue here. I do see a similarish-sounding issue at https://rt.cpan.org/Ticket/Display.html?id=114581 perchance this is the same? However - I believe I'm record as saying a number of times and I'll -- leonerd@leonerd.org.uk | https://metacpan.org/author/PEVANS |
From @demerphqOn 20 March 2017 at 18:41, Paul "LeoNerd" Evans <leonerd@leonerd.org.uk> wrote:
I wonder if this is something we should get the various people in our I mean, if Paul could have gone to his pause account, for example, and Yves -- |
From @iabynOn Mon, Mar 20, 2017 at 05:41:02PM +0000, Paul "LeoNerd" Evans wrote:
Yes, that's it.
Ok thanks. -- |
From @jkeenanOn Mon, 20 Mar 2017 16:56:12 GMT, davem wrote:
Perhaps you should attach them to these bug tickets: List-Gather: https://rt.cpan.org/Ticket/Display.html?id=116694 Cond-Expr: https://rt.cpan.org/Ticket/Display.html?id=117254 [snip] -- |
From @iabynOn Tue, Mar 28, 2017 at 07:18:13AM -0700, James E Keenan via RT wrote:
Now attached. -- |
From @iabynThis is the ticket that concerns distributions broken by enabling I think I've done about all I can for this, and propose removing it from -- |
From @pjcjOn Mon, Mar 13, 2017 at 01:43:39PM +0100, Paul Johnson wrote:
Not sure whether it can be classified as soon - in the grand scheme of -- |
From @ikegamiAlso breaks IKEGAMI/Syntax-Feature-Loop-v1.6.0.tar.gz Will fix this weekend. |
1 similar comment
From @ikegamiAlso breaks IKEGAMI/Syntax-Feature-Loop-v1.6.0.tar.gz Will fix this weekend. |
From @jkeenanOn Sat, 20 May 2017 00:47:15 GMT, ikegami@adaelis.com wrote:
Were these fixed? -- |
|
@jkeenan this is an old BBC ticket. Are we ok to close it? |
toddr
added
the
Closable?
I see newer versions for these modules, but there's a dependency on Lexical-Sub which is broken for newer perls (https://rt.cpan.org/Dist/Display.html?Name=Lexical-Var). |
|
From the modules mentioned in this ticket (except the revdeps) the following do not have a fixed stable release:
|
|
@eserte IMO the point of this ticket was to track and decide if we needed to revert/fix problems prior to the release of 5.30. Now 5.30 is out, I think we should be closing this ticket. If tickets should be opened as a result of changes in core, I'm unclear that it is the responsibility of perl/perl5 to do so. My concern is that we would have to leave A LOT of tickets open indefinitely if the policy was that all of CPAN had to be working before we could close anything. That is my opinion though, how do you see this should be handled? |
|
Formally, if the breaking change is documented in perldelta and all affected modules are notified, then there's no need to keep the issue open. I just wanted to give a small overview of the still broken modules — maybe somebody here finds one of these (or the unlisted revdeps) important enough to fix it? |
|
It was documented. These were reported.
This one is @karenetheridge 's module. I believe it is broken on something earlier so its kinda moot that this is broken there: https://rt.cpan.org/Public/Bug/Display.html?id=112244
|
Yes. |
Migrated from rt.perl.org#128179 (status was 'open')
Searchable as RT128179$
The text was updated successfully, but these errors were encountered: