diff options
author | LaMont Jones <lamontjones@google.com> | 2022-09-01 15:15:04 +0000 |
---|---|---|
committer | LaMont Jones <lamontjones@google.com> | 2022-09-19 22:03:18 +0000 |
commit | 1eddca847625c50d985d9310e2bee2901c909925 (patch) | |
tree | 9c4383b0c729bf7cb9573264c9d0973eea52331d /subcmds/selfupdate.py | |
parent | aefa4d3a29f35b00ce7e94870435b0a63c1e1612 (diff) | |
download | git-repo-1eddca847625c50d985d9310e2bee2901c909925.tar.gz |
sync: use namedtuples for internal return values
Replace tuple returns with namedtuples, to simplify adding new fields.
Extend the Sync_NetworkHalf return value to:
- success: True if successful (the former return value)
- remote_fetched: True if we called `git fetch`
Change-Id: If63c24c2f849523f77fa19c05bbf23a5e9a20ba9
Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/344534
Reviewed-by: Mike Frysinger <vapier@google.com>
Tested-by: LaMont Jones <lamontjones@google.com>
Diffstat (limited to 'subcmds/selfupdate.py')
-rw-r--r-- | subcmds/selfupdate.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/subcmds/selfupdate.py b/subcmds/selfupdate.py index 282f518e..898bc3f2 100644 --- a/subcmds/selfupdate.py +++ b/subcmds/selfupdate.py | |||
@@ -51,7 +51,7 @@ need to be performed by an end-user. | |||
51 | _PostRepoUpgrade(self.manifest) | 51 | _PostRepoUpgrade(self.manifest) |
52 | 52 | ||
53 | else: | 53 | else: |
54 | if not rp.Sync_NetworkHalf(): | 54 | if not rp.Sync_NetworkHalf().success: |
55 | print("error: can't update repo", file=sys.stderr) | 55 | print("error: can't update repo", file=sys.stderr) |
56 | sys.exit(1) | 56 | sys.exit(1) |
57 | 57 | ||