Skip Menu |

This queue is for tickets about the Forks-Super CPAN distribution.

Report information
The Basics
Id: 118474
Status: stalled
Priority: 0/
Queue: Forks-Super

People
Owner: Nobody in particular
Requestors: calvin.schwenzfeier [...] gmail.com
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: 0.88



Subject: Major timeout failure in tests resulting in 100% CPU usage and tests that never return without being forcibly exited (this is on MacOS).
Date: Sun, 23 Oct 2016 06:58:32 +0100
To: bug-Forks-Super [...] rt.cpan.org
From: Calvin Schwenzfeier <calvin.schwenzfeier [...] gmail.com>

Message body is not shown because it is too large.

Message body is not shown because it is too large.

Hi Calvin, Thank you for this report. I wonder if you might have better luck if you set $PREFER_ALTERNATE_ALARM = 1; in lib/Forks/Super/SysInfo.pm. If that helps, I could make that the default setting in MacOS. Best, Marty O'Brien
In the test report, t/40j-timeout.t passes but t/40a-timeout.t, t/40d-timeout.t, and t/40g-timeout.t all fail. This indicates that your system would benefit from using the "poor man's alarm", so I will enable it for all darwin users in v0.88. -- Marty
Subject: Re: [rt.cpan.org #118474] Major timeout failure in tests resulting in 100% CPU usage and tests that never return without being forcibly exited (this is on MacOS).
Date: Mon, 12 Dec 2016 22:20:55 +0000
To: bug-Forks-Super [...] rt.cpan.org
From: Calvin Schwenzfeier <calvin.schwenzfeier [...] gmail.com>
Oh wow! You responded! Sorry, given my previous record on reporting bugs to RT, I was expecting (at best) to hear something a few years from now. So, changing `if ($^O =~ /bsd/) {` to `if ($^O =~ /bsd|darwin/) {` in .../system-limits.PL and doing `make && make test` did the trick. B-) Thus, if it helps, I can confirm setting `$info{prefer_alt_alarm}`/`$PREFER_ALTERNATE_ALARM` to 1 on darwin fixes the timeout test failures I originally reported. Below is the resulting output for the .../t/40*-timeout.t tests: t/40a-timeout.t ............... ok t/40b-timeout.t ............... ok t/40c-timeout.t ............... Forks::Super::Job::_config_timeout_child(): quick timeout at /Users/cal/build/Forks-Super-0.87/blib/lib/Forks/Super/Job.pm line 1947. t/40c-timeout.t ............... ok t/40d-timeout.t ............... ok t/40e-timeout.t ............... ok t/40f-timeout.t ............... Forks::Super::Job::_config_timeout_child(): quick timeout at /Users/cal/build/Forks-Super-0.87/blib/lib/Forks/Super/Job.pm line 1947. t/40f-timeout.t ............... ok t/40g-timeout.t ............... ok t/40h-timeout.t ............... ok t/40i-timeout.t ............... ok t/40j-timeout.t ............... ok Thanks for supporting your module! ~Cal On Wed, Nov 23, 2016 at 9:29 PM, Marty O'Brien via RT < bug-Forks-Super@rt.cpan.org> wrote: Show quoted text
> <URL: https://rt.cpan.org/Ticket/Display.html?id=118474 > > > In the test report, t/40j-timeout.t passes but t/40a-timeout.t, > t/40d-timeout.t, > and t/40g-timeout.t all fail. This indicates that your system would > benefit from > using the "poor man's alarm", so I will enable it for all darwin users in > v0.88. > > -- Marty > > >
Enabled poor man's alarm by default for MacOS/darwin in 0.88. But there are still no CPAN tester reports from darwin systems, so I don't know what to think.