Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the Daemon-Control CPAN distribution.

Report information
The Basics
Id: 77385
Status: resolved
Priority: 0/
Queue: Daemon-Control
People
Owner: Nobody in particular
Requestors: RWSTAUNER [...] cpan.org
Cc:
AdminCc:
Bug Information
Severity: Wishlist
Broken in: (no value)
Fixed in: (no value)

History Show all quoted text
  Wed May 23 14:12:53 2012 RWSTAUNER [...] cpan.org - Ticket created
Subject: Option for configuring sleep duration between kill signals
It would be nice if the length of sleep in between kills was configurable (in the do_stop() method). I've had some complicated shut downs (on some sad hardware) that can take longer than a second. If you'd be open to the feature I'd be willing to whip up a pull req.
  Wed May 23 14:44:38 2012 symkat [...] cpan.org - Correspondence added
On Wed May 23 14:12:53 2012, RWSTAUNER wrote: Show quoted text
> It would be nice if the length of sleep in between kills was configurable > (in the do_stop() method). > > I've had some complicated shut downs (on some sad hardware) > that can take longer than a second. > > If you'd be open to the feature I'd be willing to whip up a pull req.
Hi, I've created this issue at https://github.com/symkat/Daemon-Control/issues/18 with suggestions for the implementation. Any feedback there on the name of the accessor or anything else is welcome. Feel free to submit a pull request, I'll likely be taking some time this weekend to resolve the open issues listed on github either way. =) -Kat
  Wed May 23 14:44:38 2012 The RT System itself - Status changed from 'new' to 'open'
  Wed May 23 15:05:42 2012 RWSTAUNER [...] cpan.org - Correspondence added
Looks great, thanks! If you prefer github you can specify an alternate bugtracker in the dist META files (https://metacpan.org/module/CPAN::Meta::Spec#resources) though I don't know how to do it with Module::Install (I'm a dist-zilla user). Anyway, the steps you listed look about the same as I had determined from reviewing the code, and that sounds as good an accessor name as any. Thanks again!
  Mon Sep 03 15:50:35 2012 symkat [...] cpan.org - Correspondence added
This has been commited on GitHub, and a new CPAN dist will be released at or before Wednesday night.
  Mon Sep 03 15:50:36 2012 symkat [...] cpan.org - Status changed from 'open' to 'resolved'