[release-tools] Tool to find related commits

tools/release/search_related_commits.py

+#!/usr/bin/env python
+# Copyright 2015 the V8 project 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 argparse
+import os
+import sys
+import re
+import operator
+from subprocess import Popen, PIPE
+
+def search_all_related_commits(
+    git_working_dir, start_hash, until, deadline, verbose=False):
+
+  all_commits_raw = (_git_execute(
+      git_working_dir,
+      ["rev-list", "--reverse", start_hash + ".." + until], verbose))
+  if verbose:
+    print "All commits between <of> and <until>: " + all_commits_raw
+
+  #Adding start hash too

[Michael Achenbach, 2015/04/28 21:23:51]

nit: Space after # - end comments with periods.

[Michael Hablich, 2015/04/29 09:22:42]

Done.

+  all_commits = [start_hash]
+  all_commits.extend(all_commits_raw.splitlines())
+  all_related_commits = {}
+  already_treated_commits = []
+  for commit in all_commits:
+    if commit in already_treated_commits:
+      continue
+
+    related_commits = (search_related_commits(
+        git_working_dir, commit, until, deadline, verbose))
+    if len(related_commits) > 0:
+      all_related_commits[commit] = related_commits
+      already_treated_commits.extend(related_commits)
+
+    already_treated_commits.append(commit)
+
+  return all_related_commits
+
+def search_related_commits(
+    git_working_dir, start_hash, until, deadline, verbose=False):
+
+  if deadline:
+    commits_between = (_git_execute(
+        git_working_dir,
+        ["rev-list", "--reverse", start_hash + ".." + deadline],
+        verbose))
+    if commits_between.strip() == "":
+      return []
+
+  #Extract commit position
+  original_message = _git_execute(git_working_dir,
+      ["show", "-s", "--format=%B", start_hash], verbose)
+  title = original_message.splitlines()[0]
+
+  matches = re.search("(\{#)([0-9]*)(\})", original_message)
+  commit_position = matches.group(2)
+  if verbose:
+    print "1.) Commit position to look for: " + commit_position
+
+  search_range = start_hash + ".." + until
+
+  found_by_hash = _git_execute(
+      git_working_dir, (
+          ["log",
+           "--reverse",
+           search_range,
+           "--grep=" + start_hash,
+           "--format=%H"]),
+      verbose)
+  found_by_hash = found_by_hash.strip()
+
+  if verbose:
+    print "2.) Found by hash: " + found_by_hash
+
+  found_by_commit_pos = (_git_execute(
+      git_working_dir,(

[Michael Achenbach, 2015/04/28 21:23:51]

nit: space after comma

[Michael Hablich, 2015/04/29 09:22:42]

Done.

+          ["log",
+           "--reverse",
+           search_range,
+           "--grep=" + commit_position,
+           "--format=%H"]),
+      verbose))
+
+  found_by_commit_pos = found_by_commit_pos.strip()
+
+  if verbose:
+    print "3.) Found by commit position: " + found_by_commit_pos
+
+  #Replace brackets or else they are wrongly interpreted by --grep
+  title = title.replace("[", "\\[")
+  title = title.replace("]", "\\]")
+
+  found_by_title = (_git_execute(
+      git_working_dir,(
+          ["log", "--reverse",
+           search_range,
+           '--grep=' + title,
+           "--format=%H"]),
+      verbose))
+
+  found_by_title = found_by_title.strip()
+
+  if verbose:
+    print "4.) Found by title: " + found_by_title
+
+  hits = (
+      _convert_to_array(found_by_hash) +
+      _convert_to_array(found_by_commit_pos) +
+      _convert_to_array(found_by_title))
+  hits = _remove_duplicates(hits)
+
+  return hits
+
+def _convert_to_array(string_of_hashes):
+  if len(string_of_hashes) == 0:
+    return []
+  return string_of_hashes.splitlines()
+
+def _remove_duplicates(array):
+   no_duplicates = []
+   for current in array:
+    if not current in no_duplicates:
+      no_duplicates.append(current)
+   return no_duplicates
+
+def _git_execute(working_dir, commands, verbose=False):
+
+  fullCommand = ["git", "-C", working_dir] + commands
+  if verbose:
+    print "Git working dir: " + working_dir
+    print "Executing git command:" + str(fullCommand)
+  p = Popen(args=fullCommand, stdin=PIPE,
+      stdout=PIPE, stderr=PIPE)
+  output, err = p.communicate()
+  rc = p.returncode
+  if rc != 0:
+    raise Exception(err)
+  if verbose:
+    print "Git return value: " + output
+  return output
+
+def _pretty_print_entry(hash, git_dir, pre_text, verbose):
+
+  text_to_print = _git_execute(
+      git_dir,
+      ["show",
+       "--quiet",
+       "--date=iso",
+       hash,
+       "--format=%ad # %H # %s"],
+      verbose)
+  return pre_text + text_to_print.strip()
+
+def main(options):
+
+  output = []
+  if options.all:
+    all_related_commits = search_all_related_commits(
+        options.git_dir,
+        options.of[0],
+        options.until[0],
+        options.deadline,
+        options.verbose)
+
+    high_level_commits = sorted(all_related_commits.keys(), key = lambda x: (
+        (_git_execute(options.git_dir,
+                      ["show", "--quiet", "--date=iso", x, "--format=%ad"],
+                      options.verbose)).strip()))
+
+    for current_key in high_level_commits:
+      if options.prettyprint:
+        output.append(_pretty_print_entry(
+            current_key, options.git_dir, "+", options.verbose))
+      else:
+        output.append("+" + current_key)
+
+      found_commits = all_related_commits[current_key]
+      for current_commit in found_commits:
+        if options.prettyprint:
+          output.append(_pretty_print_entry(
+              current_commit, options.git_dir, "| ", options.verbose))
+        else:
+          output.append("| " + current_commit)
+  else:
+    hits = search_related_commits(options.git_dir, options.of[0],
+        options.until[0], options.deadline, options.verbose)
+    if len(hits) > 0:
+      output.append("\n".join(hits))
+
+  return output
+
+if __name__ == "__main__":  # pragma: no cover
+  parser = argparse.ArgumentParser(
+      ("This tool searches the git repository for "
+       "commits which are related to the commit <of>."))
+  parser.add_argument("-g", "--git-dir", required=False, default=".",
+                        help="The path to your git working directory.")
+  parser.add_argument("--verbose", action="store_true",
+      help="Enables verbose output")
+  parser.add_argument("of", nargs=1,
+      help="Hash of the commit to be searched.")
+  parser.add_argument("until", nargs=1,
+      help="Commit when searching should stop")
+  parser.add_argument("--all", action="store_true",
+      help=("Searches for related commits in all "
+            "commits between <of> and <until>"))
+  parser.add_argument("--deadline", required=False,
+      help=("The script will only list related commits "
+            "which are separated by hash <--deadline>."))
+  parser.add_argument("--prettyprint", action="store_true",
+      help=("Pretty prints the output"))
+
+  args = sys.argv[1:]
+  options = parser.parse_args(args)
+  for current_line in main(options):
+    print current_line