diff --git a/.vpython b/.vpython index e3ec2487e..398aab95f 100644 --- a/.vpython +++ b/.vpython @@ -1,6 +1 @@ python_version: "2.7" - -wheel: < - name: "infra/python/wheels/httplib2-py2_py3" - version: "version:0.10.3" -> diff --git a/auth.py b/auth.py index fecb1452d..f717be15d 100644 --- a/auth.py +++ b/auth.py @@ -9,7 +9,6 @@ from __future__ import print_function import collections import datetime import functools -import httplib2 import json import logging import optparse @@ -21,6 +20,7 @@ import urlparse import subprocess2 +from third_party import httplib2 from third_party.oauth2client import client diff --git a/recipes/README.recipes.md b/recipes/README.recipes.md index 38ae95d4a..f8b829378 100644 --- a/recipes/README.recipes.md +++ b/recipes/README.recipes.md @@ -344,16 +344,16 @@ Module for interact with gerrit endpoints Wrapper for easy calling of gerrit_utils steps. -— **def [abandon\_change](/recipes/recipe_modules/gerrit/api.py#160)(self, host, change, message=None, name=None, step_test_data=None):** +— **def [abandon\_change](/recipes/recipe_modules/gerrit/api.py#159)(self, host, change, message=None, name=None, step_test_data=None):** -— **def [create\_gerrit\_branch](/recipes/recipe_modules/gerrit/api.py#32)(self, host, project, branch, commit, \*\*kwargs):** +— **def [create\_gerrit\_branch](/recipes/recipe_modules/gerrit/api.py#31)(self, host, project, branch, commit, \*\*kwargs):** Create a new branch from given project and commit Returns: the ref of the branch created -— **def [get\_change\_description](/recipes/recipe_modules/gerrit/api.py#73)(self, host, change, patchset):** +— **def [get\_change\_description](/recipes/recipe_modules/gerrit/api.py#72)(self, host, change, patchset):** Get the description for a given CL and patchset. @@ -365,7 +365,7 @@ Args: Returns: The description corresponding to given CL and patchset. -— **def [get\_changes](/recipes/recipe_modules/gerrit/api.py#118)(self, host, query_params, start=None, limit=None, o_params=None, step_test_data=None, \*\*kwargs):** +— **def [get\_changes](/recipes/recipe_modules/gerrit/api.py#117)(self, host, query_params, start=None, limit=None, o_params=None, step_test_data=None, \*\*kwargs):** Query changes for the given host. @@ -383,14 +383,14 @@ Returns: A list of change dicts as documented here: https://gerrit-review.googlesource.com/Documentation/rest-api-changes.html#list-changes -— **def [get\_gerrit\_branch](/recipes/recipe_modules/gerrit/api.py#54)(self, host, project, branch, \*\*kwargs):** +— **def [get\_gerrit\_branch](/recipes/recipe_modules/gerrit/api.py#53)(self, host, project, branch, \*\*kwargs):** Get a branch from given project and commit Returns: the revision of the branch -— **def [get\_revision\_info](/recipes/recipe_modules/gerrit/api.py#88)(self, host, change, patchset):** +— **def [get\_revision\_info](/recipes/recipe_modules/gerrit/api.py#87)(self, host, change, patchset):** Returns the info for a given patchset of a given change. diff --git a/recipes/recipe_modules/bot_update/examples/full.expected/deprecated_got_revision_mapping.json b/recipes/recipe_modules/bot_update/examples/full.expected/deprecated_got_revision_mapping.json index 5a7faf4cd..c86183a51 100644 --- a/recipes/recipe_modules/bot_update/examples/full.expected/deprecated_got_revision_mapping.json +++ b/recipes/recipe_modules/bot_update/examples/full.expected/deprecated_got_revision_mapping.json @@ -1,7 +1,7 @@ [ { "cmd": [ - "vpython", + "python", "-u", "RECIPE_REPO[depot_tools]/gerrit_client.py", "changes", diff --git a/recipes/recipe_modules/bot_update/examples/full.expected/no_apply_patch_on_gclient.json b/recipes/recipe_modules/bot_update/examples/full.expected/no_apply_patch_on_gclient.json index 1c496d352..eb1ea8b0a 100644 --- a/recipes/recipe_modules/bot_update/examples/full.expected/no_apply_patch_on_gclient.json +++ b/recipes/recipe_modules/bot_update/examples/full.expected/no_apply_patch_on_gclient.json @@ -1,7 +1,7 @@ [ { "cmd": [ - "vpython", + "python", "-u", "RECIPE_REPO[depot_tools]/gerrit_client.py", "changes", diff --git a/recipes/recipe_modules/bot_update/examples/full.expected/tryjob_fail.json b/recipes/recipe_modules/bot_update/examples/full.expected/tryjob_fail.json index cf0341d08..ae558fb78 100644 --- a/recipes/recipe_modules/bot_update/examples/full.expected/tryjob_fail.json +++ b/recipes/recipe_modules/bot_update/examples/full.expected/tryjob_fail.json @@ -1,7 +1,7 @@ [ { "cmd": [ - "vpython", + "python", "-u", "RECIPE_REPO[depot_tools]/gerrit_client.py", "changes", diff --git a/recipes/recipe_modules/bot_update/examples/full.expected/tryjob_fail_patch.json b/recipes/recipe_modules/bot_update/examples/full.expected/tryjob_fail_patch.json index d35179c9b..71b9132df 100644 --- a/recipes/recipe_modules/bot_update/examples/full.expected/tryjob_fail_patch.json +++ b/recipes/recipe_modules/bot_update/examples/full.expected/tryjob_fail_patch.json @@ -1,7 +1,7 @@ [ { "cmd": [ - "vpython", + "python", "-u", "RECIPE_REPO[depot_tools]/gerrit_client.py", "changes", diff --git a/recipes/recipe_modules/bot_update/examples/full.expected/tryjob_fail_patch_download.json b/recipes/recipe_modules/bot_update/examples/full.expected/tryjob_fail_patch_download.json index a42112fc9..cfde7a9ce 100644 --- a/recipes/recipe_modules/bot_update/examples/full.expected/tryjob_fail_patch_download.json +++ b/recipes/recipe_modules/bot_update/examples/full.expected/tryjob_fail_patch_download.json @@ -1,7 +1,7 @@ [ { "cmd": [ - "vpython", + "python", "-u", "RECIPE_REPO[depot_tools]/gerrit_client.py", "changes", diff --git a/recipes/recipe_modules/bot_update/examples/full.expected/tryjob_gerrit_angle.json b/recipes/recipe_modules/bot_update/examples/full.expected/tryjob_gerrit_angle.json index 1c496d352..eb1ea8b0a 100644 --- a/recipes/recipe_modules/bot_update/examples/full.expected/tryjob_gerrit_angle.json +++ b/recipes/recipe_modules/bot_update/examples/full.expected/tryjob_gerrit_angle.json @@ -1,7 +1,7 @@ [ { "cmd": [ - "vpython", + "python", "-u", "RECIPE_REPO[depot_tools]/gerrit_client.py", "changes", diff --git a/recipes/recipe_modules/bot_update/examples/full.expected/tryjob_gerrit_branch_heads.json b/recipes/recipe_modules/bot_update/examples/full.expected/tryjob_gerrit_branch_heads.json index 621ef0fca..5641ce512 100644 --- a/recipes/recipe_modules/bot_update/examples/full.expected/tryjob_gerrit_branch_heads.json +++ b/recipes/recipe_modules/bot_update/examples/full.expected/tryjob_gerrit_branch_heads.json @@ -1,7 +1,7 @@ [ { "cmd": [ - "vpython", + "python", "-u", "RECIPE_REPO[depot_tools]/gerrit_client.py", "changes", diff --git a/recipes/recipe_modules/bot_update/examples/full.expected/tryjob_gerrit_feature_branch.json b/recipes/recipe_modules/bot_update/examples/full.expected/tryjob_gerrit_feature_branch.json index beed9a511..139147bbf 100644 --- a/recipes/recipe_modules/bot_update/examples/full.expected/tryjob_gerrit_feature_branch.json +++ b/recipes/recipe_modules/bot_update/examples/full.expected/tryjob_gerrit_feature_branch.json @@ -1,7 +1,7 @@ [ { "cmd": [ - "vpython", + "python", "-u", "RECIPE_REPO[depot_tools]/gerrit_client.py", "changes", diff --git a/recipes/recipe_modules/bot_update/examples/full.expected/tryjob_gerrit_v8_feature_branch.json b/recipes/recipe_modules/bot_update/examples/full.expected/tryjob_gerrit_v8_feature_branch.json index 8bd5a4f31..89fadef83 100644 --- a/recipes/recipe_modules/bot_update/examples/full.expected/tryjob_gerrit_v8_feature_branch.json +++ b/recipes/recipe_modules/bot_update/examples/full.expected/tryjob_gerrit_v8_feature_branch.json @@ -1,7 +1,7 @@ [ { "cmd": [ - "vpython", + "python", "-u", "RECIPE_REPO[depot_tools]/gerrit_client.py", "changes", diff --git a/recipes/recipe_modules/bot_update/examples/full.expected/tryjob_gerrit_webrtc.json b/recipes/recipe_modules/bot_update/examples/full.expected/tryjob_gerrit_webrtc.json index a43fd3b2a..d11a80d4a 100644 --- a/recipes/recipe_modules/bot_update/examples/full.expected/tryjob_gerrit_webrtc.json +++ b/recipes/recipe_modules/bot_update/examples/full.expected/tryjob_gerrit_webrtc.json @@ -1,7 +1,7 @@ [ { "cmd": [ - "vpython", + "python", "-u", "RECIPE_REPO[depot_tools]/gerrit_client.py", "changes", diff --git a/recipes/recipe_modules/bot_update/examples/full.expected/tryjob_v8.json b/recipes/recipe_modules/bot_update/examples/full.expected/tryjob_v8.json index a0f6e80fe..d616b344f 100644 --- a/recipes/recipe_modules/bot_update/examples/full.expected/tryjob_v8.json +++ b/recipes/recipe_modules/bot_update/examples/full.expected/tryjob_v8.json @@ -1,7 +1,7 @@ [ { "cmd": [ - "vpython", + "python", "-u", "RECIPE_REPO[depot_tools]/gerrit_client.py", "changes", diff --git a/recipes/recipe_modules/bot_update/examples/full.expected/tryjob_v8_head_by_default.json b/recipes/recipe_modules/bot_update/examples/full.expected/tryjob_v8_head_by_default.json index bba65b517..bb2c38ad6 100644 --- a/recipes/recipe_modules/bot_update/examples/full.expected/tryjob_v8_head_by_default.json +++ b/recipes/recipe_modules/bot_update/examples/full.expected/tryjob_v8_head_by_default.json @@ -1,7 +1,7 @@ [ { "cmd": [ - "vpython", + "python", "-u", "RECIPE_REPO[depot_tools]/gerrit_client.py", "changes", diff --git a/recipes/recipe_modules/gerrit/api.py b/recipes/recipe_modules/gerrit/api.py index 27a3e60b8..94a0e0a3f 100644 --- a/recipes/recipe_modules/gerrit/api.py +++ b/recipes/recipe_modules/gerrit/api.py @@ -26,7 +26,6 @@ class GerritApi(recipe_api.RecipeApi): self.repo_resource('gerrit_client.py'), cmd, infra_step=infra_step, - venv=True, **kwargs) def create_gerrit_branch(self, host, project, branch, commit, **kwargs): diff --git a/recipes/recipe_modules/gerrit/examples/full.expected/basic.json b/recipes/recipe_modules/gerrit/examples/full.expected/basic.json index 02ad3eb53..5cf761854 100644 --- a/recipes/recipe_modules/gerrit/examples/full.expected/basic.json +++ b/recipes/recipe_modules/gerrit/examples/full.expected/basic.json @@ -1,7 +1,7 @@ [ { "cmd": [ - "vpython", + "python", "-u", "RECIPE_REPO[depot_tools]/gerrit_client.py", "branch", @@ -32,7 +32,7 @@ }, { "cmd": [ - "vpython", + "python", "-u", "RECIPE_REPO[depot_tools]/gerrit_client.py", "branchinfo", @@ -60,7 +60,7 @@ }, { "cmd": [ - "vpython", + "python", "-u", "RECIPE_REPO[depot_tools]/gerrit_client.py", "changes", @@ -110,7 +110,7 @@ }, { "cmd": [ - "vpython", + "python", "-u", "RECIPE_REPO[depot_tools]/gerrit_client.py", "changes", @@ -137,7 +137,7 @@ }, { "cmd": [ - "vpython", + "python", "-u", "RECIPE_REPO[depot_tools]/gerrit_client.py", "changes", @@ -185,7 +185,7 @@ }, { "cmd": [ - "vpython", + "python", "-u", "RECIPE_REPO[depot_tools]/gerrit_client.py", "abandon", @@ -229,7 +229,7 @@ }, { "cmd": [ - "vpython", + "python", "-u", "RECIPE_REPO[depot_tools]/gerrit_client.py", "changes", diff --git a/recipes/recipe_modules/git_cl/api.py b/recipes/recipe_modules/git_cl/api.py index c4e1e349d..732aa1be8 100644 --- a/recipes/recipe_modules/git_cl/api.py +++ b/recipes/recipe_modules/git_cl/api.py @@ -18,7 +18,7 @@ class GitClApi(recipe_api.RecipeApi): with self.m.context(cwd=self.m.context.cwd or my_loc): return self.m.python( name, self.repo_resource('git_cl.py'), [subcmd] + args, - venv=True, **kwargs) + **kwargs) def get_description(self, patch_url=None, codereview=None, **kwargs): """DEPRECATED. Consider using gerrit.get_change_description instead.""" diff --git a/recipes/recipe_modules/git_cl/examples/full.expected/basic.json b/recipes/recipe_modules/git_cl/examples/full.expected/basic.json index 1bb2db34c..6f0b33085 100644 --- a/recipes/recipe_modules/git_cl/examples/full.expected/basic.json +++ b/recipes/recipe_modules/git_cl/examples/full.expected/basic.json @@ -1,7 +1,7 @@ [ { "cmd": [ - "vpython", + "python", "-u", "RECIPE_REPO[depot_tools]/git_cl.py", "upload", @@ -12,7 +12,7 @@ }, { "cmd": [ - "vpython", + "python", "-u", "RECIPE_REPO[depot_tools]/git_cl.py", "issue" @@ -21,7 +21,7 @@ }, { "cmd": [ - "vpython", + "python", "-u", "RECIPE_REPO[depot_tools]/git_cl.py", "description", @@ -33,7 +33,7 @@ }, { "cmd": [ - "vpython", + "python", "-u", "RECIPE_REPO[depot_tools]/git_cl.py", "description", @@ -54,7 +54,7 @@ }, { "cmd": [ - "vpython", + "python", "-u", "RECIPE_REPO[depot_tools]/git_cl.py", "description", @@ -72,7 +72,7 @@ }, { "cmd": [ - "vpython", + "python", "-u", "RECIPE_REPO[depot_tools]/git_cl.py", "description", @@ -85,7 +85,7 @@ }, { "cmd": [ - "vpython", + "python", "-u", "RECIPE_REPO[depot_tools]/git_cl.py", "description", diff --git a/recipes/recipe_modules/presubmit/api.py b/recipes/recipe_modules/presubmit/api.py index 3fa7b9d4f..c6690dee1 100644 --- a/recipes/recipe_modules/presubmit/api.py +++ b/recipes/recipe_modules/presubmit/api.py @@ -106,7 +106,7 @@ class PresubmitApi(recipe_api.RecipeApi): '--upstream', upstream, # '' if not in bot_update mode. ]) - venv = True + venv = None # TODO(iannucci): verify that presubmit_support.py correctly finds and # uses .vpython files, then remove this configuration. if self._vpython_spec_path: diff --git a/recipes/recipe_modules/tryserver/examples/full.expected/with_gerrit_patch.json b/recipes/recipe_modules/tryserver/examples/full.expected/with_gerrit_patch.json index 3c04236c3..d3adadfb4 100644 --- a/recipes/recipe_modules/tryserver/examples/full.expected/with_gerrit_patch.json +++ b/recipes/recipe_modules/tryserver/examples/full.expected/with_gerrit_patch.json @@ -1,7 +1,7 @@ [ { "cmd": [ - "vpython", + "python", "-u", "RECIPE_REPO[depot_tools]/gerrit_client.py", "changes", @@ -41,7 +41,7 @@ }, { "cmd": [ - "vpython", + "python", "-u", "RECIPE_REPO[depot_tools]/gerrit_client.py", "changes", diff --git a/recipes/recipe_modules/tryserver/examples/full.expected/with_gerrit_patch_and_target_ref.json b/recipes/recipe_modules/tryserver/examples/full.expected/with_gerrit_patch_and_target_ref.json index 288bcdbbd..0a7368bbc 100644 --- a/recipes/recipe_modules/tryserver/examples/full.expected/with_gerrit_patch_and_target_ref.json +++ b/recipes/recipe_modules/tryserver/examples/full.expected/with_gerrit_patch_and_target_ref.json @@ -1,7 +1,7 @@ [ { "cmd": [ - "vpython", + "python", "-u", "RECIPE_REPO[depot_tools]/gerrit_client.py", "changes", @@ -41,7 +41,7 @@ }, { "cmd": [ - "vpython", + "python", "-u", "RECIPE_REPO[depot_tools]/gerrit_client.py", "changes", diff --git a/tests/auth_test.py b/tests/auth_test.py index c540ed8a1..dded7a1d0 100755 --- a/tests/auth_test.py +++ b/tests/auth_test.py @@ -7,7 +7,6 @@ import __builtin__ import datetime -import httplib2 import json import logging import os @@ -19,6 +18,7 @@ sys.path.insert(0, os.path.dirname(os.path.dirname(os.path.abspath(__file__)))) from testing_support import auto_stub +from third_party import httplib2 from third_party import mock import auth