Skip to content

Fix run win32 console reader in bare thread #15724

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

Conversation

ysbaddaden
Copy link
Contributor

@ysbaddaden ysbaddaden commented Apr 28, 2025

Simpler alternative to #15714 that focuses on dropping the isolated context that depends on the default context (not yet initialized at this point) and we don't need the isolated context bubble wrap anyway.

Closes #15714

@ysbaddaden ysbaddaden self-assigned this Apr 28, 2025
@straight-shoota straight-shoota added kind:bug A bug in the code. Does not apply to documentation, specs, etc. platform:windows Windows support based on the MSVC toolchain / Win32 API topic:stdlib:runtime labels Apr 28, 2025
@straight-shoota straight-shoota added this to the 1.16.2 milestone Apr 28, 2025
@straight-shoota straight-shoota merged commit fc41c28 into crystal-lang:master Apr 28, 2025
38 checks passed
@crysbot
Copy link
Collaborator

crysbot commented Apr 28, 2025

Server Error

created backport PR for release/1.16:

straight-shoota pushed a commit that referenced this pull request Apr 28, 2025
@ysbaddaden ysbaddaden deleted the fix/start-win32-console-reader-loop-in-bare-thread branch April 29, 2025 09:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport release/1.16 kind:bug A bug in the code. Does not apply to documentation, specs, etc. platform:windows Windows support based on the MSVC toolchain / Win32 API topic:stdlib:runtime
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants