-
Notifications
You must be signed in to change notification settings - Fork 28.8k
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
Integrated ZSH terminal - job table full or recursion limit exceeded #163843
Comments
Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.72.2. Please try upgrading to the latest version and checking whether this issue remains. Happy Coding! |
Worth nothing that I've just updated to Also worth noting, this issue does not happen if I:
However does happen when I:
|
Can you reproduce this in insiders? Believe this has been fixed |
@meganrogge Same problem in insiders. Same error pointing to the same file (albeit in the |
This would happen if your What is the output of |
In my env, Full envNote, this is the full
|
I see the issue - we should not be setting the zdotdir if the user_zdotdir is our temp directory |
Is there any quick workaround for this? |
@KekOnTheWorld you should be able to try the fix in insider's tomorrow |
OMG this issue is so annoying! Even disabling the shell integration does not seem to help. |
I actually found it working when i just restarted vscode using my launcher. When starting vscode from a terminal the issue still occurs |
This bug has been fixed in the latest release of VS Code Insiders! @kb-ig, you can help us out by commenting If things still don't seem right, please ensure you're on version 9908b3d of Insiders (today's or later - you can use Happy Coding! |
This does not appear to have been resolved in insiders. While I no longer see the error message mentioned in the description, my integrated terminal is still broken, in that I cannot use any of the zsh functionality. As an example, when I load the IDE via launcher and open a folder via the File > Open Folder, my integrated zsh terminal works as expected, displaying something like:
However when opening the editor via the terminal, my integrated zsh terminal displays something like:
And in the latter case, none of the zsh functionality works. Version info
After re-reading my original bug description, I realise I never mentioned that the zsh functionality wasnt working, so that's my bad. Perhaps these two issues are unrelated (recursion limit exceeded and zsh terminal becoming a "basic terminal"), but they appeared at the same time. |
@kb-ig thanks for the info - that sounds like a separate issue. am I correct in my understanding that you see the issue when you open via the cli IE |
@meganrogge Correct, but for both So to confirm, opening VSCode (insiders and normal) via CLI causes this issue. However opening VSCode (insiders and normal) via app launcher is OK. |
Considering this verified as the fix was meant to prevent the recursion from happening, which worked based on the above. Will deal with the other issue separately |
Type: Bug
Since updating to
1.72.11.72.2, I have been constantly getting the following error in the integrated terminal:This path points to a file with the following contents
This only happens when using an integrated terminal in VSCode. My external ZSH terminal works completely fine. Worth mentioning that I'm on ZSH 5.8.
Anyone know why I'm getting this error, and more importantly, how I can go about resolving it? Thanks!
VS Code version: Code 1.72.1 (129500e, 2022-10-10T17:23:00.760Z)
OS version: Linux x64 5.15.0-50-generic
Modes:
Sandboxed: No
System Info
canvas_oop_rasterization: disabled_off
direct_rendering_display_compositor: disabled_off_ok
gpu_compositing: enabled
multiple_raster_threads: enabled_on
opengl: enabled_on
rasterization: enabled
raw_draw: disabled_off_ok
skia_renderer: enabled_on
video_decode: disabled_software
video_encode: disabled_software
vulkan: disabled_off
webgl: enabled
webgl2: enabled
webgpu: disabled_off
Extensions (28)
(18 theme extensions excluded)
A/B Experiments
The text was updated successfully, but these errors were encountered: