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