diff options
| author | Gavin Mak <gavinmak@google.com> | 2025-08-13 01:10:37 +0000 |
|---|---|---|
| committer | Gavin Mak <gavinmak@google.com> | 2025-08-13 23:16:55 -0700 |
| commit | 380bf9546e3c54d634e5e57ed77aa19746317032 (patch) | |
| tree | 39394c07cea746ed3da68cecd5a4908539146d7b | |
| parent | d9cc0a15265299b6dcfc1d65f192fd14cfb17b02 (diff) | |
| download | git-repo-380bf9546e3c54d634e5e57ed77aa19746317032.tar.gz | |
sync: always show sync result stderr_text on error
_ProcessSyncInterleavedResults currently only shows stderr_text if
verbose. Show it if a sync worker fails, regardless of verbosity.
Bug: 438178765
Change-Id: If24dcb10fb5d6857386782d371e3f9c6844dece9
Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/498061
Reviewed-by: Scott Lee <ddoman@google.com>
Tested-by: Gavin Mak <gavinmak@google.com>
| -rw-r--r-- | subcmds/sync.py | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/subcmds/sync.py b/subcmds/sync.py index 41b8bb03..c9b1b5c8 100644 --- a/subcmds/sync.py +++ b/subcmds/sync.py | |||
| @@ -2398,7 +2398,8 @@ later is required to fix a server side protocol bug. | |||
| 2398 | pm.update() | 2398 | pm.update() |
| 2399 | project = projects[result.project_index] | 2399 | project = projects[result.project_index] |
| 2400 | 2400 | ||
| 2401 | if opt.verbose and result.stderr_text: | 2401 | success = result.fetch_success and result.checkout_success |
| 2402 | if result.stderr_text and (opt.verbose or not success): | ||
| 2402 | pm.display_message(result.stderr_text) | 2403 | pm.display_message(result.stderr_text) |
| 2403 | 2404 | ||
| 2404 | if result.fetch_start: | 2405 | if result.fetch_start: |
