diff options
author | Mike Frysinger <vapier@google.com> | 2021-05-04 15:31:51 -0400 |
---|---|---|
committer | Mike Frysinger <vapier@google.com> | 2021-05-04 20:00:20 +0000 |
commit | 14208f4c938ade527db32f5ea2a5793a36eafd8f (patch) | |
tree | 5b62cc73b1565b1fce7db877f2c479b932348c43 /subcmds/sync.py | |
parent | 2ee0a62db077c6e96ae2002a6f9fc1196146710a (diff) | |
download | git-repo-14208f4c938ade527db32f5ea2a5793a36eafd8f.tar.gz |
sync: fix logic error with linkfile errors
Make sure err_update_linkfiles is always initalized.
Bug: https://crbug.com/gerrit/11008
Change-Id: I7bdd91f82507608ef967daf0fa0f9c859454e19f
Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/305146
Reviewed-by: Raman Tenneti <rtenneti@google.com>
Tested-by: Mike Frysinger <vapier@google.com>
Diffstat (limited to 'subcmds/sync.py')
-rw-r--r-- | subcmds/sync.py | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/subcmds/sync.py b/subcmds/sync.py index 31760e5c..9a09be65 100644 --- a/subcmds/sync.py +++ b/subcmds/sync.py | |||
@@ -986,9 +986,9 @@ later is required to fix a server side protocol bug. | |||
986 | print('\nerror: Local checkouts *not* updated.', file=sys.stderr) | 986 | print('\nerror: Local checkouts *not* updated.', file=sys.stderr) |
987 | sys.exit(1) | 987 | sys.exit(1) |
988 | 988 | ||
989 | if not self.UpdateCopyLinkfileList(): | 989 | err_update_linkfiles = not self.UpdateCopyLinkfileList() |
990 | if err_update_linkfiles: | ||
990 | err_event.set() | 991 | err_event.set() |
991 | err_update_linkfiles = True | ||
992 | if opt.fail_fast: | 992 | if opt.fail_fast: |
993 | print('\nerror: Local update copyfile or linkfile failed.', file=sys.stderr) | 993 | print('\nerror: Local update copyfile or linkfile failed.', file=sys.stderr) |
994 | sys.exit(1) | 994 | sys.exit(1) |