| Index: recipe_modules/rietveld/api.py
|
| diff --git a/recipe_modules/rietveld/api.py b/recipe_modules/rietveld/api.py
|
| deleted file mode 100644
|
| index aa7374fc3f7735ac410bcd1b42f0c794b05091a2..0000000000000000000000000000000000000000
|
| --- a/recipe_modules/rietveld/api.py
|
| +++ /dev/null
|
| @@ -1,89 +0,0 @@
|
| -# Copyright 2013 The Chromium Authors. All rights reserved.
|
| -# Use of this source code is governed by a BSD-style license that can be
|
| -# found in the LICENSE file.
|
| -
|
| -import urlparse
|
| -
|
| -from recipe_engine import recipe_api
|
| -
|
| -
|
| -class RietveldApi(recipe_api.RecipeApi):
|
| - def calculate_issue_root(self, extra_patch_project_roots=None):
|
| - """Returns path where a patch should be applied to based on "patch_project".
|
| -
|
| - Maps Rietveld's "patch_project" to a path of directories relative to
|
| - api.gclient.c.solutions[0].name which describe where to place the patch.
|
| -
|
| - Args:
|
| - extra_patch_project_roots: Dict mapping project names to relative roots.
|
| -
|
| - Returns:
|
| - Relative path or empty string if patch_project is not set or path for a
|
| - given is unknown.
|
| - """
|
| - # Property 'patch_project' is set by Rietveld, 'project' is set by git-try
|
| - # when TRYSERVER_PROJECT is present in codereview.settings.
|
| - patch_project = (self.m.properties.get('patch_project') or
|
| - self.m.properties.get('project'))
|
| -
|
| - # Please avoid adding projects into this hard-coded list unless your project
|
| - # CLs are being run by multiple recipes. Instead pass patch_project_roots to
|
| - # ensure_checkout.
|
| - patch_project_roots = {
|
| - 'angle/angle': ['third_party', 'angle'],
|
| - 'blink': ['third_party', 'WebKit'],
|
| - 'v8': ['v8'],
|
| - 'luci-py': ['luci'],
|
| - 'recipes-py': ['recipes-py'],
|
| - }
|
| -
|
| - # Make sure to update common projects (above) with extra projects (and not
|
| - # vice versa, so that recipes can override default values if needed.
|
| - if extra_patch_project_roots:
|
| - patch_project_roots.update(extra_patch_project_roots)
|
| -
|
| - path_parts = patch_project_roots.get(patch_project)
|
| - return self.m.path.join(*path_parts) if path_parts else ''
|
| -
|
| - def apply_issue(self, *root_pieces, **kwargs):
|
| - """Call apply_issue from depot_tools.
|
| -
|
| - Args:
|
| - root_pieces (strings): location where to run apply_issue, relative to the
|
| - checkout root.
|
| - authentication (string or None): authentication scheme to use. Can be None
|
| - or 'oauth2'. See also apply_issue.py --help (-E and --no-auth options.)
|
| - """
|
| - # TODO(pgervais): replace *root_pieces by a single Path object.
|
| - authentication = kwargs.get('authentication', None)
|
| - rietveld_url = self.m.properties['rietveld']
|
| - issue_number = self.m.properties['issue']
|
| -
|
| - if authentication == 'oauth2':
|
| - step_result = self.m.python(
|
| - 'apply_issue',
|
| - self.m.path['depot_tools'].join('apply_issue.py'), [
|
| - '-r', self.m.path['checkout'].join(*root_pieces),
|
| - '-i', issue_number,
|
| - '-p', self.m.properties['patchset'],
|
| - '-s', rietveld_url,
|
| - '-E', self.m.path['build'].join('site_config',
|
| - '.rietveld_client_email'),
|
| - '-k', self.m.path['build'].join('site_config',
|
| - '.rietveld_secret_key')
|
| - ],
|
| - )
|
| -
|
| - else:
|
| - step_result = self.m.python(
|
| - 'apply_issue',
|
| - self.m.path['depot_tools'].join('apply_issue.py'), [
|
| - '-r', self.m.path['checkout'].join(*root_pieces),
|
| - '-i', issue_number,
|
| - '-p', self.m.properties['patchset'],
|
| - '-s', rietveld_url,
|
| - '--no-auth'],
|
| - )
|
| - step_result.presentation.links['Applied issue %s' % issue_number] = (
|
| - urlparse.urljoin(rietveld_url, str(issue_number)))
|
| -
|
|
|