Index: recipe_modules/bot_update/resources/bot_update.py |
diff --git a/recipe_modules/bot_update/resources/bot_update.py b/recipe_modules/bot_update/resources/bot_update.py |
index 2323f4c4b14b57c88eb9f075a09acf31896a9e3d..ab71dd4a9a5bf7feff021eb0b125643d9bb2aaa0 100755 |
--- a/recipe_modules/bot_update/resources/bot_update.py |
+++ b/recipe_modules/bot_update/resources/bot_update.py |
@@ -555,8 +555,7 @@ |
def apply_rietveld_issue(issue, patchset, root, server, _rev_map, _revision, |
- email_file, key_file, oauth2_file, |
- whitelist=None, blacklist=None): |
+ email_file, key_file, whitelist=None, blacklist=None): |
apply_issue_bin = ('apply_issue.bat' if sys.platform.startswith('win') |
else 'apply_issue') |
cmd = [apply_issue_bin, |
@@ -571,10 +570,8 @@ |
# Don't run gclient sync when it sees a DEPS change. |
'--ignore_deps', |
] |
- # Use an oauth key or json file if specified. |
- if oauth2_file: |
- cmd.extend(['--auth-refresh-token-json', oauth2_file]) |
- elif email_file and key_file: |
+ # Use an oauth key file if specified. |
+ if email_file and key_file: |
cmd.extend(['--email-file', email_file, '--private-key-file', key_file]) |
else: |
cmd.append('--no-auth') |
@@ -717,9 +714,8 @@ |
def ensure_checkout(solutions, revisions, first_sln, target_os, target_os_only, |
patch_root, issue, patchset, rietveld_server, gerrit_repo, |
gerrit_ref, gerrit_rebase_patch_ref, revision_mapping, |
- apply_issue_email_file, apply_issue_key_file, |
- apply_issue_oauth2_file, shallow, refs, git_cache_dir, |
- gerrit_reset): |
+ apply_issue_email_file, apply_issue_key_file, shallow, refs, |
+ git_cache_dir, gerrit_reset): |
# Get a checkout of each solution, without DEPS or hooks. |
# Calling git directly because there is no way to run Gclient without |
# invoking DEPS. |
@@ -742,8 +738,7 @@ |
if issue: |
apply_rietveld_issue(issue, patchset, patch_root, rietveld_server, |
revision_mapping, git_ref, apply_issue_email_file, |
- apply_issue_key_file, apply_issue_oauth2_file, |
- whitelist=[target]) |
+ apply_issue_key_file, whitelist=[target]) |
already_patched.append(target) |
elif gerrit_ref: |
apply_gerrit_ref(gerrit_repo, gerrit_ref, patch_root, gerrit_reset, |
@@ -770,9 +765,8 @@ |
# Apply the rest of the patch here (sans DEPS) |
if issue: |
apply_rietveld_issue(issue, patchset, patch_root, rietveld_server, |
- revision_mapping, apply_issue_oauth2_file, git_ref, |
- apply_issue_email_file, apply_issue_key_file, |
- blacklist=already_patched) |
+ revision_mapping, git_ref, apply_issue_email_file, |
+ apply_issue_key_file, blacklist=already_patched) |
elif gerrit_ref and not applied_gerrit_patch: |
# If gerrit_ref was for solution's main repository, it has already been |
# applied above. This chunk is executed only for patches to DEPS-ed in |
@@ -840,9 +834,6 @@ |
help='--email-file option passthrough for apply_patch.py.') |
parse.add_option('--apply_issue_key_file', |
help='--private-key-file option passthrough for ' |
- 'apply_patch.py.') |
- parse.add_option('--apply_issue_oauth2_file', |
- help='--auth-refresh-token-json option passthrough for ' |
'apply_patch.py.') |
parse.add_option('--root', dest='patch_root', |
help='DEPRECATED: Use --patch_root.') |
@@ -981,7 +972,6 @@ |
revision_mapping=options.revision_mapping, |
apply_issue_email_file=options.apply_issue_email_file, |
apply_issue_key_file=options.apply_issue_key_file, |
- apply_issue_oauth2_file=options.apply_issue_oauth2_file, |
# Finally, extra configurations such as shallowness of the clone. |
shallow=shallow, |