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 | ||
