diff options
Diffstat (limited to 'Tools/Scripts/webkitpy/tool/steps/updatechangelogswithreviewer.py')
-rw-r--r-- | Tools/Scripts/webkitpy/tool/steps/updatechangelogswithreviewer.py | 79 |
1 files changed, 0 insertions, 79 deletions
diff --git a/Tools/Scripts/webkitpy/tool/steps/updatechangelogswithreviewer.py b/Tools/Scripts/webkitpy/tool/steps/updatechangelogswithreviewer.py deleted file mode 100644 index ef210a02f..000000000 --- a/Tools/Scripts/webkitpy/tool/steps/updatechangelogswithreviewer.py +++ /dev/null @@ -1,79 +0,0 @@ -# Copyright (C) 2010 Google Inc. All rights reserved. -# -# Redistribution and use in source and binary forms, with or without -# modification, are permitted provided that the following conditions are -# met: -# -# * Redistributions of source code must retain the above copyright -# notice, this list of conditions and the following disclaimer. -# * Redistributions in binary form must reproduce the above -# copyright notice, this list of conditions and the following disclaimer -# in the documentation and/or other materials provided with the -# distribution. -# * Neither the name of Google Inc. nor the names of its -# contributors may be used to endorse or promote products derived from -# this software without specific prior written permission. -# -# THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS -# "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT -# LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR -# A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT -# OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, -# SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT -# LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE, -# DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY -# THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT -# (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE -# OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE. - -import logging - -from webkitpy.common.checkout.changelog import ChangeLog -from webkitpy.tool.grammar import pluralize -from webkitpy.tool.steps.abstractstep import AbstractStep -from webkitpy.tool.steps.options import Options - -_log = logging.getLogger(__name__) - - -class UpdateChangeLogsWithReviewer(AbstractStep): - @classmethod - def options(cls): - return AbstractStep.options() + [ - Options.git_commit, - Options.reviewer, - ] - - def _guess_reviewer_from_bug(self, bug_id): - # FIXME: It's unclear if it would be safe to use self.cached_lookup(state, 'bug') - # here as we don't currently have a way to invalidate a bug after making changes (like ObsoletePatches does). - patches = self._tool.bugs.fetch_bug(bug_id).reviewed_patches() - if not patches: - _log.info("%s on bug %s, cannot infer reviewer." % ("No reviewed patches", bug_id)) - return None - patch = patches[-1] - _log.info("Guessing \"%s\" as reviewer from attachment %s on bug %s." % (patch.reviewer().full_name, patch.id(), bug_id)) - return patch.reviewer().full_name - - def run(self, state): - bug_id = state.get("bug_id") - if not bug_id and state.get("patch"): - bug_id = state.get("patch").bug_id() - - reviewer = self._options.reviewer - if not reviewer: - if not bug_id: - _log.info("No bug id provided and --reviewer= not provided. Not updating ChangeLogs with reviewer.") - return - reviewer = self._guess_reviewer_from_bug(bug_id) - - if not reviewer: - _log.info("Failed to guess reviewer from bug %s and --reviewer= not provided. Not updating ChangeLogs with reviewer." % bug_id) - return - - # cached_lookup("changelogs") is always absolute paths. - for changelog_path in self.cached_lookup(state, "changelogs"): - ChangeLog(changelog_path).set_reviewer(reviewer) - - # Tell the world that we just changed something on disk so that the cached diff is invalidated. - self.did_modify_checkout(state) |