-
Notifications
You must be signed in to change notification settings - Fork 571
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
Blead Breaks CPAN: MAROS/DateTime-Format-CLDR-1.19.tar.gz #16429
Comments
From @eserteThe t/011_nanosecond.t of DateTime-Format-CLDR-1.19 test started to Test log:
Perl Info
|
From @eserteLooked into my fail report list again, and it seems that the same error may happen also with older perl versions. And the author has already put a note in the test script: explain("This test might fail on some plattforms due to unknown reasons"); |
From zefram@fysh.orgslaven@rezic.de wrote:
Fails for me in that manner on both 5.27.8 and 5.27.9. On my test perls, On older perls I see it failing on some versions and passing on other I'll look some more. -zefram |
The RT System itself - Status changed from 'new' to 'open' |
From @eserteDana Wed, 21 Feb 2018 13:44:48 -0800, zefram@fysh.org re�e:
Does not seem so --- I run the test script with Module::PrintUsed on the failing linux and passing freebsd system, and the only difference is: - Test::NoWarnings 1.04 ... which is expected, as Test2::API::Breakage is loaded only if a failure occurs.
|
From maros@cpan.orgMight be related to houseabsolute/DateTime.pm#29 |
From zefram@fysh.orgThe Perl version difference that I see between 5.21.3 and The perl version differences that I see on older perls (on Linux -zefram |
From @eserteDana Thu, 22 Feb 2018 11:23:24 -0800, zefram@fysh.org re�e:
Link to CPAN RT ticket: https://rt.cpan.org/Ticket/Display.html?id=124691 |
This was reported as a BBC ticket but we appear to have not made a decision to change anything in perl for 5.28.0's final release. Given this has been reported upstream, I propose we close this unless we can identify a bug in Perl that needs to be addressed. |
See houseabsolute/DateTime.pm#29 (comment) for more analysis. It appears that @autarch spotted this problem in @maros 's distro several years ago. I have two inferences. First, this is a problem likely to occur only with certain configurations of perl. I've demonstrated it with Second, this is a problem which will have to be addressed first by the CPAN module author. The failing test needs to be made more flexible to accommodate rounding errors with different perl configurations. Note that that may not be the entire problem, but it's definitely part of the problem. I hope we hear back from the module author and from Dave Rolsky. Thank you very much. |
This seems like a bug in the module (DT::F::CLDR), not the perl core. |
Agreed. I cloned the github repo, intending to prepare a p.r. along the same lines as the one you applied to DateTime in v1.51 (houseabsolute/DateTime.pm#89). However, right out-of-the-box I got a Module::Install-related failure in Makefile.PL, reported in houseabsolute/DateTime.pm#89. I think the core has made every reasonable effort to address this issue and that it will be up to the module author to take the next step. So I'm closing it as a core issue. Thank you very much. |
I think your second link was meant to be: https://rt.cpan.org/Public/Bug/Display.html?id=131908 |
Migrated from rt.perl.org#132894 (status was 'open')
Searchable as RT132894$
The text was updated successfully, but these errors were encountered: