Skip to content

Prevent deadlock when Windows Forms are loaded in the PowerShell process#3

Merged
CarloToso merged 1 commit intoCarloToso:encodinghelperfrom
stevenebutler:encodinghelper-patch
Jan 7, 2024
Merged

Prevent deadlock when Windows Forms are loaded in the PowerShell process#3
CarloToso merged 1 commit intoCarloToso:encodinghelperfrom
stevenebutler:encodinghelper-patch

Conversation

@stevenebutler
Copy link
Copy Markdown

PR Summary

Must use ConfigureAwait(false) on any awaited methods to prevent use of windows forms in the PowerShell process
from deadlocking the process. This was the cause of the CI failure - test was added specifically for this issue that causes the CI test to deadlock and time out the build stage.

This is needed to stop web methods from deadlocking when windows forms are loaded from
within the PowerShell process.
@stevenebutler
Copy link
Copy Markdown
Author

@CarloToso - hopefully this patch fixes the build failures. I wasn't aware this was an issue but found a patch by seeminglyscience that must have been found post-release of pwsh 7.4. The test included in that patch is what was failing/stalling the build process.

@stevenebutler stevenebutler changed the base branch from master to encodinghelper January 7, 2024 00:40
@CarloToso CarloToso merged commit 4abf713 into CarloToso:encodinghelper Jan 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants