summaryrefslogtreecommitdiffstats
path: root/Tools/Scripts/webkitpy/tool/steps/updatechangelogswithreviewer.py
diff options
context:
space:
mode:
Diffstat (limited to 'Tools/Scripts/webkitpy/tool/steps/updatechangelogswithreviewer.py')
-rw-r--r--Tools/Scripts/webkitpy/tool/steps/updatechangelogswithreviewer.py79
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)