diff options
author | Mike Frysinger <vapier@google.com> | 2020-03-07 01:53:53 -0500 |
---|---|---|
committer | David Pursehouse <dpursehouse@collab.net> | 2020-03-07 13:10:14 +0000 |
commit | e20da3eeedf03dcffbb882ca10b159719f8e334d (patch) | |
tree | 31e3eca7a95fb1dbf9baaad5aaa9bc3a7c17d20b /subcmds/sync.py | |
parent | 910dfe84978b6ed3400e367c6a9f3cb3f17a2384 (diff) | |
download | git-repo-e20da3eeedf03dcffbb882ca10b159719f8e334d.tar.gz |
sync: fix os.environ logic errors
This is a dict to index, not a function to call.
Change-Id: I0117eeaaa8b2ef4762ab6f0d22f9ffdaee961f52
Reviewed-on: https://gerrit-review.googlesource.com/c/git-repo/+/258132
Tested-by: Mike Frysinger <vapier@google.com>
Reviewed-by: David Pursehouse <dpursehouse@collab.net>
Diffstat (limited to 'subcmds/sync.py')
-rw-r--r-- | subcmds/sync.py | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/subcmds/sync.py b/subcmds/sync.py index e2bc1d17..ae29f085 100644 --- a/subcmds/sync.py +++ b/subcmds/sync.py | |||
@@ -729,12 +729,12 @@ later is required to fix a server side protocol bug. | |||
729 | branch = branch[len(R_HEADS):] | 729 | branch = branch[len(R_HEADS):] |
730 | 730 | ||
731 | if 'SYNC_TARGET' in os.environ: | 731 | if 'SYNC_TARGET' in os.environ: |
732 | target = os.environ('SYNC_TARGET') | 732 | target = os.environ['SYNC_TARGET'] |
733 | [success, manifest_str] = server.GetApprovedManifest(branch, target) | 733 | [success, manifest_str] = server.GetApprovedManifest(branch, target) |
734 | elif ('TARGET_PRODUCT' in os.environ and | 734 | elif ('TARGET_PRODUCT' in os.environ and |
735 | 'TARGET_BUILD_VARIANT' in os.environ): | 735 | 'TARGET_BUILD_VARIANT' in os.environ): |
736 | target = '%s-%s' % (os.environ('TARGET_PRODUCT'), | 736 | target = '%s-%s' % (os.environ['TARGET_PRODUCT'], |
737 | os.environ('TARGET_BUILD_VARIANT')) | 737 | os.environ['TARGET_BUILD_VARIANT']) |
738 | [success, manifest_str] = server.GetApprovedManifest(branch, target) | 738 | [success, manifest_str] = server.GetApprovedManifest(branch, target) |
739 | else: | 739 | else: |
740 | [success, manifest_str] = server.GetApprovedManifest(branch) | 740 | [success, manifest_str] = server.GetApprovedManifest(branch) |