tree a671e76b13c61b364a88cdd52f2196e650edc9da
parent 4855987e331fc887d80639073adbfc5efc09638d
author Eli Lipsitz <elipsitz@google.com> 1674835261 +0000
committer CQ Bot Account <pigweed-scoped@luci-project-accounts.iam.gserviceaccount.com> 1674835261 +0000

pw_build: Detect EOF on macOS in pw-wrap-ninja

Previously, we relied on Python throwing an OSError when we attempted to
read from the Ninja subprocess's stdout to know if it closed its side of
the TTY.

This doesn't work on macOS -- instead, `read` returns an empty string.
This commit checks for both, preventing the issue where pw-wrap-ninja
would hang indefinitely when running it on macOS.

Change-Id: Ib8fca3e75197848c8aa2df733357010031c96959
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/126921
Pigweed-Auto-Submit: Eli Lipsitz <elipsitz@google.com>
Commit-Queue: Auto-Submit <auto-submit@pigweed.google.com.iam.gserviceaccount.com>
Reviewed-by: Rob Mohr <mohrr@google.com>