diff options
| author | Bob Haarman <inglorion@chromium.org> | 2024-01-31 13:35:38 +0000 |
|---|---|---|
| committer | Chromeos LUCI <chromeos-scoped@luci-project-accounts.iam.gserviceaccount.com> | 2024-02-01 18:41:16 +0000 |
| commit | bf1e26ec64ed7904322aaa32e7b80458c3a5d99f (patch) | |
| tree | 4610b9dbddf314d001f20cabdb0e1e6c823cb639 /rust_tools/rust_uprev.py | |
| parent | 7d31a04df666d131619cdafc42c30f8c4a6f63b6 (diff) | |
| download | toolchain-utils-bf1e26ec64ed7904322aaa32e7b80458c3a5d99f.tar.gz | |
rust_uprev: use stdout=PIPE instead of capture_output
When running equery fails, we currently don't get an error message,
because it is swallowed by subprocess.run. This change makes it so
that the error message is shown on standard error instead.
BUG=None
TEST=made it fail, saw error message
Change-Id: Ib057f3215505d5824593d387c22df5f95b163921
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/5259297
Commit-Queue: Bob Haarman <inglorion@chromium.org>
Tested-by: Bob Haarman <inglorion@chromium.org>
Reviewed-by: George Burgess <gbiv@chromium.org>
Diffstat (limited to 'rust_tools/rust_uprev.py')
| -rwxr-xr-x | rust_tools/rust_uprev.py | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/rust_tools/rust_uprev.py b/rust_tools/rust_uprev.py index 4ff8a4c9..ce88f9a5 100755 --- a/rust_tools/rust_uprev.py +++ b/rust_tools/rust_uprev.py @@ -203,7 +203,8 @@ def compute_rust_bootstrap_prebuilt_name(version: RustVersion) -> str: def find_ebuild_for_package(name: str) -> str: """Returns the path to the ebuild for the named package.""" return run_in_chroot( - [EQUERY, "w", name], capture_output=True + [EQUERY, "w", name], + stdout=subprocess.PIPE, ).stdout.strip() |