From 380bf9546e3c54d634e5e57ed77aa19746317032 Mon Sep 17 00:00:00 2001 From: Gavin Mak Date: Wed, 13 Aug 2025 01:10:37 +0000 Subject: [PATCH] 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 Tested-by: Gavin Mak --- subcmds/sync.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/subcmds/sync.py b/subcmds/sync.py index 41b8bb038..c9b1b5c8c 100644 --- a/subcmds/sync.py +++ b/subcmds/sync.py @@ -2398,7 +2398,8 @@ later is required to fix a server side protocol bug. pm.update() project = projects[result.project_index] - if opt.verbose and result.stderr_text: + success = result.fetch_success and result.checkout_success + if result.stderr_text and (opt.verbose or not success): pm.display_message(result.stderr_text) if result.fetch_start: