-
Notifications
You must be signed in to change notification settings - Fork 669
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
"Sending on the channel failed. Did the context die?" when using threads #11249
Comments
Hey @BafS, can you reproduce the issue on https://psalm.dev? These will be used as phpunit tests when implementing the feature or fixing this bug. |
What is the full output? |
No we don't use ev, event or uv
Full output (from gitlab CI):
|
What version of PHP are you running? Please try running scans specifically on php 8.4.3, this may be a PHP bug. |
I won't be able to try with PHP 8.4.3 right now because we don't use PHP 8.4 yet on the CI and I cannot reproduce it locally (but I'll try again) |
Try using 8.3.16, and also try using 8.4.3 if possible. |
I tried with 8.3.16 and it didn't change anything, I have the exact same errors unfortunately |
And on 8.4.3? |
Will add some better tracking for the exit code of subprocesses to try and identify the issue, but I'd like to see the result of the scan on 8.4 (which should not change anything in the actual output as long as the target PHP version is set to the usual one) |
Okay thanks. I'll tell you when we have 8.4.x available on our CI but it won't be soon. I'll try to reproduce it locally tomorrow. |
When trying to use the
--threads
option I hit this issue "Sending on the channel failed. Did the context die?
". I tried combination of various flags and using more or less number of threads but errors remain. It does work without using the--threads
option.Psalm version: 6.2
Trace:
When using
--threads=2
I have an other trace:The text was updated successfully, but these errors were encountered: