Skip to content

Silently ignore the screen size is bogus error in the Process Explorer#104277

Merged
RMacfarlane merged 1 commit intomasterfrom
adi-screen-size-bogus
Aug 11, 2020
Merged

Silently ignore the screen size is bogus error in the Process Explorer#104277
RMacfarlane merged 1 commit intomasterfrom
adi-screen-size-bogus

Conversation

@9at8
Copy link
Copy Markdown
Contributor

@9at8 9at8 commented Aug 7, 2020

Fixes #98590

What

This pr just ignores the "screen size is bogus" error.

flex tape meme for silently ignoring screen size is bogus

We can try spawn instead of exec, which does not create a subshell.

@9at8 9at8 requested a review from RMacfarlane August 7, 2020 23:15
@9at8
Copy link
Copy Markdown
Contributor Author

9at8 commented Aug 7, 2020

@RMacfarlane I triggered a build, I can paste the link for it here. I'm not sure if this should be a candidate or not. 🤷‍♂️

@9at8 9at8 changed the title Silently ignore the screen size is bogus error Silently ignore the screen size is bogus error in the Process Explorer Aug 7, 2020
Copy link
Copy Markdown
Contributor

@RMacfarlane RMacfarlane left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome! I think this doesn't qualify as a candidate, but will merge it in for next month's release. Thank you!

@RMacfarlane RMacfarlane merged commit f6f8735 into master Aug 11, 2020
@RMacfarlane RMacfarlane deleted the adi-screen-size-bogus branch August 11, 2020 17:38
@RMacfarlane RMacfarlane added this to the August 2020 milestone Aug 11, 2020
@github-actions github-actions bot locked and limited conversation to collaborators Sep 25, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Process Explorer doesn't work on WSL (screen size is bogus)

2 participants