diff options
author | George Burgess IV <gbiv@google.com> | 2023-10-31 12:58:35 -0600 |
---|---|---|
committer | Chromeos LUCI <chromeos-scoped@luci-project-accounts.iam.gserviceaccount.com> | 2023-10-31 22:29:06 +0000 |
commit | acf15930b191a7bd6d03d60d290327dc9e637027 (patch) | |
tree | c36982026ed4453720de361347e90d9a8e66688a /rust_tools/copy_rust_bootstrap.py | |
parent | 8f2c2657e374596ea00ddb71c2df38b1058c2fe4 (diff) | |
download | toolchain-utils-acf15930b191a7bd6d03d60d290327dc9e637027.tar.gz |
copy_rust_bootstrap: add missing `stdin` for gsutil
If `stdin` is left to a tty here, `gsutil` may ask if the user wants to
update `gsutil`. This is not a good UX.
BUG=b:303832189
TEST=Ran the script
Change-Id: I3ae6a438fe5d639f76584d602fb4b9637e0e2549
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/4996045
Commit-Queue: George Burgess <gbiv@chromium.org>
Tested-by: George Burgess <gbiv@chromium.org>
Reviewed-by: Bob Haarman <inglorion@chromium.org>
Diffstat (limited to 'rust_tools/copy_rust_bootstrap.py')
-rwxr-xr-x | rust_tools/copy_rust_bootstrap.py | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/rust_tools/copy_rust_bootstrap.py b/rust_tools/copy_rust_bootstrap.py index c32d1fd9..b10f4ef3 100755 --- a/rust_tools/copy_rust_bootstrap.py +++ b/rust_tools/copy_rust_bootstrap.py @@ -52,6 +52,7 @@ def _download(remote_path: str, local_file: Path): subprocess.run( ["gsutil", "cp", remote_path, str(local_file)], check=True, + stdin=subprocess.DEVNULL, ) |