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.21.5-77-g51a82aa breaks TIMB/Devel-NYTProf-5.06.tar.gz #14210
Comments
From @andkgit bisect commit 51a82aa [perl #122695] Fix line number for else{foo} sample fail report http://www.cpantesters.org/cpan/report/57794ab6-5b4c-11e4-8613-91e8bb36d621 perl -V Summary of my perl5 (revision 5 version 21 subversion 6) configuration: Characteristics of this binary (from libperl): |
From @cpansproutOn Wed Nov 05 11:30:52 2014, andreas.koenig.7os6VVqR@franz.ak.mind.de wrote:
Changes to line numbers (even fixes) are bound to break such sensitive tests. I don’t understand the test output, so I am cc’ing Tim Bunce with the hope that he can shed light on whether there really is a Perl bug here. -- Father Chrysostomos |
The RT System itself - Status changed from 'new' to 'open' |
From @timbunceThe NYTProf tests are very sensitive to line number changes. I've not looked in-depth yet but I've looked closely enough to say that this looks like it's detecting the improved line numbering and that the tests need adjusting to handle that. So this shouldn't be a blocker for perl. And thanks, better line number accuracy is good! |
From @jkeenanOn Thu Nov 06 03:47:38 2014, timbo wrote:
Tim: I see that we have a passing cpantesters report for Devel-NYTProf-5.06_80 (Developer version) at: http://www.cpantesters.org/cpan/report/d23be2dc-6582-11e4-aef9-bc9bbb36d621 Does that mean that this ticket is closable? Thank you very much. -- |
From @timbunceOn Fri, Dec 12, 2014 at 05:12:46PM -0800, James E Keenan via RT wrote:
I hope to look into the state of this over the Christmas break. Tim. |
From @timbunceI've uploaded trial release (Devel-NYTProf-5.07-TRIAL1.tar.gz) that's Once that smokes ok on cpantesters then I'll release it and we can close Tim. |
From @timbunceI've uploaded Devel-NYTProf-5.07 now. |
@iabyn - Status changed from 'open' to 'resolved' |
Migrated from rt.perl.org#123134 (status was 'resolved')
Searchable as RT123134$
The text was updated successfully, but these errors were encountered: