-
Notifications
You must be signed in to change notification settings - Fork 560
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
Time::HiRes test failure on threaded darwin build #15401
Comments
From @dur-randirCreated by @dur-randirThe following test consistently fails on threaded build (and passes on non-threaded): ./perl -Ilib -Idist/Time-HiRes dist/Time-HiRes/t/clock.t Perl Info
|
From @tonycozOn Tue Jun 21 05:31:08 2016, randir wrote:
This appears to have been fixed in blead, probably by commit c4a535a Upgrade Time::HiRes from version 1.9734 to 1.9739 which is confusing, since it's blead upstream. Tony |
The RT System itself - Status changed from 'new' to 'open' |
From @dur-randirOn Tue Jul 19 22:45:43 2016, tonyc wrote:
Yes, it passes tests now. |
From mojca.miklavec.lists@gmail.comDne Wed, 20 Jul 2016 04:42:48 -0700 je randir napisal:
I'm probably experiencing the same problem with 5.24.1 (on OS X 10.7), but I checked 5.25.9 after finding this thread and all the tests pass on 5.25.9. Is there any chance to backport the fix? Thank you. Some additional information below. Just in case here's the relevant chuck from the tests (to make sure it's the same issue?): dist/Time-HiRes/t/clock ....................................... panic: MUTEX_LOCK (22) [HiRes.xs:773] at t/clock.t line 40. This might be related to: Also relevant. I'm using the following patch for 5.24, else the build would fail on Sierra: --- dist/Time-HiRes/HiRes.xs.orig It could also be that the test fails because of this patch. https://trac.macports.org/ticket/52012 |
From mojca.miklavec.lists@gmail.comThe tests succeed with perl5.25.9. One of our developers says that the problem is in the included version of Time::HiRes: v5.22.3 1.9726 Citing David:
|
From @LeontOn Mon, Jan 30, 2017 at 12:59 PM, Mojca Miklavec via RT
Yeah, this does sound like a sensible thing to backport. Leon |
From @xsawyerxOn 02/17/2017 11:15 AM, Leon Timmermans wrote:
I agree. |
From @steve-m-hayNote: Backporting a newer Time::HiRes for 5.24.3 is under discussion in https://rt-archive.perl.org/perl5/Ticket/Display.html?id=128427. |
Migrated from rt.perl.org#128445 (status was 'open')
Searchable as RT128445$
The text was updated successfully, but these errors were encountered: