Skip Menu |

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

Report information
The Basics
Id: 75178
Status: resolved
Priority: 0/
Queue: Net-Daemon

People
Owner: Nobody in particular
Requestors: TODDR [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: Normal
Broken in: 0.48
Fixed in: (no value)



Subject: Net::Daemon emitting warnings in perl 5.16 (no threads)
This will emit copious re-define warnings in perl 5.15+. It didn't in 5.14 as best I can tell perl -cw -MNet::Daemon -e1 The reason as best I can tell is because threads and threads::shared are pulled in even if perl was not compiled with threads. I suggest renaming the variable $this_is_510 to $this_is_510_threads The change this: my $this_is_510 = $^V ge v5.10.0; To This: $this_is_510_threads = $^V ge v5.10.0 && $Config{'usethreads'};
Malcom, I'd like your take on this idea. I'll do a release if you have nothing in the works ATM. Todd
Subject: Re: [rt.cpan.org #75178] Net::Daemon emitting warnings in perl 5.16 (no threads)
Date: Tue, 21 Feb 2012 06:49:55 -0500
To: bug-Net-Daemon [...] rt.cpan.org
From: Malcolm Nooning <m.nooning [...] comcast.net>
Hello Todd, I just started a new major task. I cannot look at this until the weekend. There were no problems on Windows XP when I checked last week. However, there were problems on my CentOS Linux. Do you have both Linux and Windows that you can test on? On 2/20/2012 8:06 PM, Todd Rinaldo via RT wrote: Show quoted text
> Queue: Net-Daemon > Ticket<URL: https://rt.cpan.org/Ticket/Display.html?id=75178> > > Malcom, I'd like your take on this idea. I'll do a release if you have nothing in the works ATM. > > Todd >
More info on this. It looks like when forks is installed, it installs it's own version of threads over top of threads.pm, probably when perl is built without threads support. We probably need to detect that use threads is using forks and behave accordingly.
If after loading threads, you notice that $forks::threads is true, it means threads is actually forks masquerading as threads.
Ticket migrated to github as https://github.com/toddr/Net-Daemon/issues/16