Skip Menu |

This queue is for tickets about the IO-Lambda CPAN distribution.

Report information
The Basics
Id: 76863
Status: rejected
Priority: 0/
Queue: IO-Lambda
People
Owner: Nobody in particular
Requestors: ribasushi [...] leporine.io
Cc:
AdminCc:
Bug Information
Severity: Normal
Broken in: (no value)
Fixed in: (no value)

History Show all quoted text
  Sat Apr 28 09:49:03 2012 ribasushi [...] leporine.io - Ticket created
Subject: Global destruction detection is not thread-safe
Your distribution is using the construct END { $global_destroy = 1 } in [1], which is not thread safe as described here [2]. Please consider switching to https://metacpan.org/release/Devel-GlobalDestruction, which has both an XS and a PurePerl variant, both of which are tested to behave correctly under ithreads. Cheers [1] http://cpansearch.perl.org/src/KARASIK/IO-Lambda-1.23/lib/IO/Lambda/Loop/POE.pm [2] https://metacpan.org/source/DOY/Devel-GlobalDestruction-0.05/lib/Devel/GlobalDestruction.pm#L36
  Sat Apr 28 13:53:36 2012 KARASIK [...] cpan.org - Correspondence added
Hmm I have hard time imagining a realistic situation where this may be a problem ... any real example or this is just a theoretical exercise? I'm skeptic because POE and ithreads don't live well together, AFAIK.
  Sat Apr 28 13:53:37 2012 The RT System itself - Status changed from 'new' to 'open'
  Tue May 15 04:58:13 2012 KARASIK [...] cpan.org - Status changed from 'open' to 'rejected'
  Sun Nov 04 06:16:48 2012 ribasushi [...] leporine.io - Correspondence added
On Sat Apr 28 13:53:36 2012, KARASIK wrote: Show quoted text
> Hmm I have hard time imagining a realistic situation where this may be a > problem ... any real example or this is just a theoretical exercise? I'm > skeptic because POE and ithreads don't live well together, AFAIK.
This is not entirely correct. On win32 fork() == thread->new. And POE does work under Win32 (even if limited). As far as realistic problem scenarios - the test suite of D::GD is rather extensive. Anyhow - I do not have a *specific* problem with this module (in fact I never used it). I am just doing a "public service" based on: http://grep.cpan.me/?q=(%3Fms%3AEND\s*\{.{0%2C200}(%3Fi%3Aglobal).{0%2C10}(%3Fi%3Adestr)) If you still believe this does not deserve attention - re-reject the ticket and I will let it stay closed.
  Sun Nov 04 06:16:49 2012 The RT System itself - Status changed from 'rejected' to 'open'
  Sun Nov 04 11:24:21 2012 KARASIK [...] cpan.org - Correspondence added
Well POE doesn't play nice with win32 forks (i.e. threads) either, and END with forks behaves wildly different on unix and win32 models. Running POE on a forked thread under win32, this is just wrong on so many levels; I just don't want to encourage people doing that, sorry.
  Sun Nov 04 11:24:22 2012 KARASIK [...] cpan.org - Status changed from 'open' to 'rejected'
  Sun Nov 04 14:32:02 2012 ribasushi [...] leporine.io - Reference by ticket #80623 added