Leon Timmermans

Results 377 comments of Leon Timmermans

> Right now I don't see a direct way for us to get the main thread ID/handle. That's something I'd need to add to intrpvar.h to use pthread_kill(). That could...

I see you've already made a PR for the MakeMaker part of change. This PR should be marked as not mergeable until that has been applied, released and synced back...

I can see the use of this, but I'm not sure if POSIX is the right place for it.

The module is using a function that is not part of the API. [0351a62](https://github.com/Perl/perl5/commit/0351a629e71de127cbfd1b142e9eaa6069deabf5) just enforced that but isn't the problem. The real question is «Should that function become API?»....

I have several distributions that I know are broken by the smartmatch change that haven't been detected by the usual mechanisms yet. I'm worried the problem is larger than we...

> Can you name them? (I've been keeping an off-list log and would like to add them if needed.) [Smart::Match](http://matrix.cpantesters.org/?dist=Smart-Match+0.008), [Syntax-Infix-Smartmatch](http://matrix.cpantesters.org/?dist=Syntax-Infix-Smartmatch+0.009) (working on a fix, but that depends on #22612),...

> [Smart::Match](http://matrix.cpantesters.org/?dist=Smart-Match+0.008), [Syntax-Infix-Smartmatch](http://matrix.cpantesters.org/?dist=Syntax-Infix-Smartmatch+0.009) (working on a fix, but that depends on https://github.com/Perl/perl5/pull/22612), [threads-lite](http://matrix.cpantesters.org/?dist=threads-lite+0.034), [SmartMatch-Sugar](http://matrix.cpantesters.org/?dist=SmartMatch-Sugar+0.05) Syntax-Infix-Smartmatch has been fixed.

Are you using [perl-cross](https://github.com/arsv/perl-cross)? Perl's own support for cross-compilation is sadly fairly limited.

The problem here is that thread handlers are process wide, but the handler perl installs assumes that it will run in a Perl thread. Evidently it doesn't here. The easiest...

I'm not sure "with SigAction objects" really adds clarity here, but the "unless you set their C flag explicitly" is spot on.