Bug #75931 for App-Daemon: Proc::ProcessTable Prerequisite

Tue Mar 20 23:23:46 2012 SHAW [...] cpan.org - Ticket created

Subject:

Proc::ProcessTable Prerequisite

Hi, as you may know Proc::ProcessTable 0.45 does not compile on perl's where usei?threads=undef. This is do to a bugfix that makes things threadsafe. 0.44 still "works" (compiles). Could you add a note in the POD about this, and maybe a link to this fix: https://rt.cpan.org/Ticket/Display.html?id=58236 -Skye

Wed Mar 21 02:46:29 2012 MSCHILLI [...] cpan.org - Correspondence added

I think this should be in Proc::ProcessTable's bug queue, not in App::Daemon's. Modules warning of problems that are entirely in the realm of their dependencies isn't a scalable solution. If there was a relation between how App::Daemon uses Proc::ProcessTable and the bug, that'd be a different story, but from what I hear it's just that Proc::ProcessTable is broken and needs to be fixed, right?

Wed Mar 21 02:46:30 2012 The RT System itself - Status changed from 'new' to 'open'

Wed Mar 21 02:46:30 2012 MSCHILLI [...] cpan.org - Status changed from 'open' to 'resolved'

Wed Mar 21 14:39:37 2012 SHAW [...] cpan.org - Correspondence added

On Wed Mar 21 02:46:29 2012, MSCHILLI wrote: Show quoted text

> I think this should be in Proc::ProcessTable's bug queue, not in > App::Daemon's.

It has been; for 4 years. Show quoted text

> Modules warning of problems that are entirely in the realm of their > dependencies isn't a scalable solution.

We're talking about highlighting problems one might encounter when installing a library and/or application on a particular platform/ configuration. This is quite common. Show quoted text

> ... but from what I hear it's just that Proc::ProcessTable > is broken and needs to be fixed, right?

Yes. -Skye

Wed Mar 21 14:39:37 2012 The RT System itself - Status changed from 'resolved' to 'open'

Mon Oct 08 17:53:48 2012 BBYRD [...] cpan.org - Correspondence added

More like "Proc::ProcessTable is broken and shouldn't be used". It hasn't been updated since 2008. It has a CPAN Tester score of 1345 / 851, along with several bad reviews.

Mon Oct 08 22:08:39 2012 MSCHILLI [...] cpan.org - Correspondence added

Do you know an alternative to Proc::ProcessTable? App::Daemon is using it to find a running process by name, the module provides a nice abstraction on top of many *nix-specific implementations.

Tue Oct 30 13:52:02 2012 BBYRD [...] cpan.org - Correspondence added

On Mon Oct 08 22:08:39 2012, MSCHILLI wrote: Show quoted text

> Do you know an alternative to Proc::ProcessTable? App::Daemon is using

it Show quoted text

> to find a running process by name, the module provides a nice

abstraction Show quoted text

> on top of many *nix-specific implementations.

How about P9Y::ProcessTable?

Wed Oct 31 02:08:50 2012 MSCHILLI [...] cpan.org - Correspondence added

I've changed process_is_running() to use POSIX methods only. Proc::ProcessTable is now optional and only required when using process_running_by_name(): https://github.com/mschilli/app- daemon/commit/dce8330fca0a0f72670bf8eacf2f9dd164dc2086 Let me know if this works for you. Thanks for reporting the problem. -- Mike

Wed Oct 31 02:08:50 2012 MSCHILLI [...] cpan.org - Status changed from 'open' to 'resolved'

Thu Nov 01 01:32:42 2012 MSCHILLI [...] cpan.org - Correspondence added

On Tue Oct 30 13:52:02 2012, BBYRD wrote: Show quoted text

> How about P9Y::ProcessTable?

I like the idea (support a limited number of OSes, no XS code), but it requires 22 additional CPAN modules (!) on my box and then stops with a syntax error: syntax error at /Users/mschilli/.cpanm/work/1351701388.50313/P9Y- ProcessTable-0.93/blib/lib/P9Y/ProcessTable.pm line 14, near ") {"

Thu Nov 01 01:32:42 2012 The RT System itself - Status changed from 'resolved' to 'open'

Thu Apr 18 15:07:58 2013 BBYRD [...] cpan.org - Correspondence added

On Thu Nov 01 01:32:42 2012, MSCHILLI wrote: Show quoted text

> On Tue Oct 30 13:52:02 2012, BBYRD wrote:

> > How about P9Y::ProcessTable?

> > I like the idea (support a limited number of OSes, no XS code), but it > requires 22 additional CPAN modules (!) on my box and then stops with a > syntax error: > > syntax error at /Users/mschilli/.cpanm/work/1351701388.50313/P9Y- > ProcessTable-0.93/blib/lib/P9Y/ProcessTable.pm line 14, near ") {"

Is this still a problem with the latest _DEV release?

Thu Apr 18 22:25:31 2013 MSCHILLI [...] cpan.org - Correspondence added

On Thu Apr 18 15:07:58 2013, BBYRD wrote: Show quoted text

> Is this still a problem with the latest _DEV release?

Test suite is failing now because it can't find List::AllUtils.

Tue Apr 29 02:27:01 2014 MSCHILLI [...] cpan.org - Correspondence added

Closing due to inactivity.

Tue Apr 29 02:27:02 2014 MSCHILLI [...] cpan.org - Status changed from 'open' to 'resolved'