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
lib/File/Copy.t fails mtime test in perl-5.28 built with quadmath #16632
Comments
From @jkeenanWhen perl is compiled with -Dusequadmath, we are prone to an error in ##### $ ./perl -Ilib -V | grep config_args $ cd t;./perl harness -v ../lib/File/Copy.t; cd - Test Summary Report ../lib/File/Copy.t (Wstat: 256 Tests: 466 Failed: 1) Per suggestion from khw on #p5p, I created a branch in which I reverted I next checked out tag v5.26.0 and re-configured with '-Dusequadmath'. I next checked out tag v5.28.0 and re-configured with '-Dusequadmath'. ##### Test Summary Report /home/jkeenan/gitwork/perl2/lib/File/Copy.t (Wstat: 256 Tests: 466 Thank you very much. |
From @jkeenanSummary of my perl5 (revision 5 version 28 subversion 0) configuration: Characteristics of this binary (from libperl): |
From @sisyphusOn Sun, 15 Jul 2018 14:15:29 -0700, jkeenan@pobox.com wrote:
There's some discussion about this at: I placed the first release affected by the bug at 5.27.6. (That probably should be double checked, but.) Cheers, |
The RT System itself - Status changed from 'new' to 'open' |
From @jkeenanOn Mon, 16 Jul 2018 00:31:08 GMT, sisyphus@cpan.org wrote:
Reviewing that thread, I see that a fix was pushed to a smoke-me branch, but all the smoke tests failed due to a porting test failure. I have corrected that error and pushed the result to a new branch: smoke-me/jkeenan/davem/133377-time-hires Can we get that branch smoked on platforms that support quadmath? (And, does that include anything beyond Linux?)
Thank you very much. |
From @sisyphus
"Yes" to both. In fact it's desirable. However, I've just agreed (in #41202) to moving part of my patch into When I've got that change sorted, how about I: 1) update blead to current state Is that the best way for me to proceed ? Cheers, On Thu, Jul 19, 2018 at 10:46 AM, James E Keenan via RT <
|
From @sisyphusOn Thu, 19 Jul 2018 18:10:23 -0700, sisyphus359@gmail.com wrote:
Duh - ignore that previous post of mine as it has no relevance to this ticket. (Using gmail, I thought I was posting to a different thread.) Apologies, |
From @sisyphusOn Wed, 18 Jul 2018 17:46:19 -0700, jkeenan wrote:
I see plenty of evidence that the problem still persists. I don't have a solution, but I do find the following rather interesting: Firstly, the test script does: my $time = 1000000000.12345; If one then immediately checks the value returned by: (stat("copy-$$"))[9] one finds that it is exactly equal to $time - 1e-9. Following on from that, there's a call to: move("copy-$$", "file-$$") and if we then immediately check the value returned by: (stat("file-$$"))[9] we find that it is exactly equal to $time - 1e-9 - 1e-9 These aren't simple rounding errors, and I've no idea of (and even less interest in) what the OS is making of all this. One workaround would be to instead set: $time = 1000000000.123449998; With that in place, all tests pass. Maybe there should also be a TODO test using the original value of 1000000000.12345 ? I was curious as to whether the (stat(...))[9] values would change every time the file was moved, but I found that, having arrived at the value of 1000000000.123449998, that's where it remained. Cheers, |
From @iabynOn Fri, Aug 03, 2018 at 06:12:31AM -0700, sisyphus@cpan.org via RT wrote:
I haven't read this thread closely, but is this same issue that I produced commit 3e15af2 (origin/smoke-me/davem/time_hires_nsec, time_hires_nsec) time::HiRes: don't truncate nanosec utime M dist/Time-HiRes/HiRes.xs -- |
1 similar comment
From @iabynOn Fri, Aug 03, 2018 at 06:12:31AM -0700, sisyphus@cpan.org via RT wrote:
I haven't read this thread closely, but is this same issue that I produced commit 3e15af2 (origin/smoke-me/davem/time_hires_nsec, time_hires_nsec) time::HiRes: don't truncate nanosec utime M dist/Time-HiRes/HiRes.xs -- |
From @sisyphus< I haven't read this thread closely, but is this same issue that I Seems very likely to me that the answer is "yes". Could the patch be applied now, so that we can try it out ? Cheers, On Fri, Aug 3, 2018 at 11:29 PM, Dave Mitchell <davem@iabyn.com> wrote:
|
1 similar comment
From @sisyphus< I haven't read this thread closely, but is this same issue that I Seems very likely to me that the answer is "yes". Could the patch be applied now, so that we can try it out ? Cheers, On Fri, Aug 3, 2018 at 11:29 PM, Dave Mitchell <davem@iabyn.com> wrote:
|
From @khwilliamsonOn 08/03/2018 07:29 AM, Dave Mitchell wrote:
I have pushed this patch to the smoke report currently running. The http://perl.develop-help.com/?b=smoke-me%2Fkhw-locale
|
1 similar comment
From @khwilliamsonOn 08/03/2018 07:29 AM, Dave Mitchell wrote:
I have pushed this patch to the smoke report currently running. The http://perl.develop-help.com/?b=smoke-me%2Fkhw-locale
|
From @khwilliamsonOn 08/03/2018 09:05 AM, Karl Williamson wrote:
The smokes are coming out much better now. But there are still some And there's a small problem with Rob's patch, proto.h:4332:11: warning: 'S_mulexp10' declared 'static' but never There needs to be a #ifdef in embed.fnc to fix this warning
|
1 similar comment
From @khwilliamsonOn 08/03/2018 09:05 AM, Karl Williamson wrote:
The smokes are coming out much better now. But there are still some And there's a small problem with Rob's patch, proto.h:4332:11: warning: 'S_mulexp10' declared 'static' but never There needs to be a #ifdef in embed.fnc to fix this warning
|
From @khwilliamsonOn Fri, 03 Aug 2018 21:56:47 -0700, public@khwilliamson.com wrote:
Is this ticket closable? |
From @jkeenanOn Thu, 18 Apr 2019 14:57:13 GMT, khw wrote:
Just now I built blead v5.29.9-151-g43049430d1 on Linux with -Dusequadmath. I got no errors in lib/File/Copy.t. So the original complaint has been satisfied. (A lot else got discussed in this ticket, though. And with quadmath there are known failures in t/re/uniprops02.t -- but that's outside the scope of this RT.) So, by me at least, it's closable. Thank you very much. |
From @jkeenanOn Thu, 18 Apr 2019 22:11:42 GMT, jkeenan wrote:
No one has spoken in favor of keeping this ticket open, so I'm closing it. In the course of this ticket there was a branch created pertaining to Time::HiRes. However, that module has moved on since that time and the branch will no longer cleanly merge into blead. So any problems with Time::HiRes should go into a new RT. Thank you very much. |
@jkeenan - Status changed from 'open' to 'resolved' |
Migrated from rt.perl.org#133377 (status was 'resolved')
Searchable as RT133377$
The text was updated successfully, but these errors were encountered: