Page MenuHomePhabricator

Avoid leaving stdin in nonblocking mode after a modern prompt
ClosedPublic

Authored by epriestley on May 30 2021, 8:31 AM.
Tags
None
Referenced Files
F13270030: D21666.diff
Wed, May 29, 8:42 AM
F13269606: D21666.id.diff
Wed, May 29, 6:30 AM
F13265198: D21666.id51561.diff
Tue, May 28, 2:03 AM
F13208357: D21666.id51562.diff
Thu, May 16, 2:33 PM
F13188485: D21666.diff
Sat, May 11, 5:17 AM
F13183936: D21666.id.diff
Fri, May 10, 1:22 PM
Unknown Object (File)
Tue, May 7, 8:31 AM
Unknown Object (File)
Mon, May 6, 7:41 AM
Subscribers
None

Details

Summary

Ref T13649. Currently, "arc" may leave stdin nonblocking after showing a prompt. This can cause various odd behaviors down the line.

I can't immediately reproduce this behavior on macOS in "zsh" or "bash" (I'm unable to get stdin to remain nonblocking beyond the process lifespan), and also don't have pcntl locally so there's a fair amount of handwaving here.

Test Plan

This is somewhat speculative since I can't immediately reproduce the behavior. I tested the locally-reachable paths (no pcntl) but they're not interesting.

Diff Detail

Repository
rARC Arcanist
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This revision was not accepted when it landed; it landed in state Needs Review.May 30 2021, 8:31 AM
epriestley requested review of this revision.
This revision was automatically updated to reflect the committed changes.