New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Bleadperl v5.19.5-40-g2e73d70 breaks DFH/App-JobLog-1.031.tar.gz #13373
Comments
From @andkgit bisect commit 2e73d70 Propagate lvalue context to && and || diagnostics http://www.cpantesters.org/cpan/report/a6669ff4-3e22-11e3-8065-20e7f24aa4a3 perl -V Summary of my perl5 (revision 5 version 19 subversion 6) configuration: Characteristics of this binary (from libperl): -- |
From @jkeenanOn Sat Oct 26 06:05:20 2013, andreas.koenig.7os6VVqR@franz.ak.mind.de wrote:
At http://www.cpantesters.org/distro/A/App-JobLog.html, I see App-JobLog v1.031 as having a very strong record of PASSes on Perl 5.18 and 5.19. So I wonder whether this ticket is still valid. Comments? Thank you very much. |
The RT System itself - Status changed from 'new' to 'open' |
From @andk"James E Keenan via RT" <perlbug-followup@perl.org> writes:
I think the URL you chose filters bleadperl releases out. Please try the http://matrix.cpantesters.org/?dist=App-JobLog%201.031 Or configure your access to the other url at prefs.cpantesters.org -- |
From @jkeenanOn Wed Dec 25 15:22:25 2013, andreas.koenig.7os6VVqR@franz.ak.mind.de wrote:
Thanks for that lead. And that confirms that this is a recent failure. I just tested the module against an installed Perl 5.18.1 and it passed (albeit with a load of those "when is experimental" warnings). jimk |
From @jkeenanOn Wed Dec 25 17:38:21 2013, jkeenan wrote:
Have notified distro maintainer via https://rt.cpan.org/Ticket/Display.html?id=91660 |
From @tonycozOn Wed Dec 25 15:22:25 2013, andreas.koenig.7os6VVqR@franz.ak.mind.de wrote:
This turns out to be caused by a change in perl, previously: $ perl -MData::Dumper -e 'my %h; for ($h{modifier} || "") { 1 } print Dumper(\%h)' with blead: $ ./perl -Ilib -MData::Dumper -e 'my %h; for ($h{modifier} || "") { 1 } print Dumper(\%h)' bisected to: commit 2e73d70 Propagate lvalue context to && and || The author in this case is I think doing the obvious thing to avoid lvalue context from modifying the hash, but this breaks that. Tony |
From @demerphqOn 6 February 2014 12:28, Tony Cook via RT <perlbug-followup@perl.org> wrote:
Personally I consider this a pretty serious regression. Lots of code Yves -- |
From @rgarciaOn 6 February 2014 06:08, demerphq <demerphq@gmail.com> wrote:
Totally agreed. Moreover that means that upgrading perl could make an |
From perl5-porters@perl.orgTony Cook wrote:
I have a fix in my head that will make this work the way it used to I just have not had a chance to do it yet. It will involve propagat- |
From @cpansproutFixed in 375879a. -- Father Chrysostomos |
@cpansprout - Status changed from 'open' to 'resolved' |
Migrated from rt.perl.org#120374 (status was 'resolved')
Searchable as RT120374$
The text was updated successfully, but these errors were encountered: