When "arc" receives SIGWINCH or other signals during display of a prompt, recover
ClosedPublic
Actions

Authored by epriestley on Jun 5 2020, 11:07 PM.

Tags

None

Referenced Files

	F13240764: D21317.id50777.diff
	Wed, May 22, 4:22 PM

	F13238644: D21317.diff
	Tue, May 21, 8:34 PM

	F13236575: D21317.id.diff
	Tue, May 21, 10:11 AM

	F13229443: D21317.diff
	Mon, May 20, 4:32 PM

	F13198741: D21317.id.diff
	Mon, May 13, 9:27 AM

	F13198678: D21317.id50777.diff
	Mon, May 13, 8:45 AM

	Unknown Object (File)
	Tue, May 7, 10:25 AM

	Unknown Object (File)
	Tue, May 7, 7:26 AM

View All 76 Files

Subscribers

None

Details

Reviewers: None
Maniphest Tasks: T13546: Modernize the "arc land" workflow
Commits: rARC3ed81d35a23a: When "arc" receives SIGWINCH or other signals during display of a prompt…

Summary

Ref T13546. Resizing the terminal window to send SIGWINCH, or other signals, may interrupt "stream_select()" with an error which upgrades to a RuntimeException.

When "stream_select()" fails, continue and test the stream itself.

Test Plan

Waited for a prompt, resized the window. Before patch: SIGWINCH interruption. After patch: undisturbed prompt.

Diff Detail

Repository

rARC Arcanist

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Jun 5 2020, 11:07 PM

Harbormaster completed remote builds in B24567: Diff 50745.Jun 5 2020, 11:07 PM

epriestley requested review of this revision.Jun 5 2020, 11:07 PM

This revision was not accepted when it landed; it landed in state Needs Review.Jun 8 2020, 10:57 PM

Closed by commit rARC3ed81d35a23a: When "arc" receives SIGWINCH or other signals during display of a prompt… (authored by epriestley). · Explain Why

This revision was automatically updated to reflect the committed changes.

epriestley added a commit: rARC3ed81d35a23a: When "arc" receives SIGWINCH or other signals during display of a prompt….

Revision Contents
Changeset List

Path

Size

src/

toolset/

ArcanistPrompt.php

26 lines

Diff 50777

View Options

src/toolset/ArcanistPrompt.php

When "arc" receives SIGWINCH or other signals during display of a prompt, recoverClosedPublicActions