[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, of, until, deadline, verbose=False):
+
+  hash = of

[Michael Achenbach, 2015/04/27 13:09:43]

not liked

[Michael Hablich, 2015/04/27 15:39:00]

Done.

+
+  all_commits_raw = (_git_execute(
+      git_working_dir,
+      ["rev-list", "--reverse", hash + ".." + until], verbose))
+  if verbose:
+    print "All commits between <of> and <until>: " + all_commits_raw
+
+  all_commits = 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, of, until, deadline, verbose=False):
+
+  hash = of

[Michael Achenbach, 2015/04/27 13:09:43]

not liked

[Michael Hablich, 2015/04/27 15:39:00]

Done.

+
+  if deadline:
+
+    commits_between = (_git_execute(
+        git_working_dir,
+        ["rev-list", "--reverse", hash + ".." + deadline],
+        verbose))
+    if commits_between.strip() == "":
+      return []
+
+  #Extract commit position
+  original_message = _git_execute(git_working_dir,
+      ["show", "-s", "--format=%B", 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 = hash + ".." + until
+
+  found_by_hash = (_git_execute(
+      git_working_dir, (

[Michael Achenbach, 2015/04/27 13:09:43]

You don't need additional parentheses in python as long as you are in the scope
of one opening parenthesis.

[Michael Hablich, 2015/04/27 15:39:00]

Done.

+          ["log", "--reverse",
+          search_range,

[Michael Achenbach, 2015/04/27 13:09:43]

nit: indentation, align list items with content in [

+          "--grep=" + 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,(
+          ["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, pre_text, verbose):

[Michael Achenbach, 2015/04/27 13:09:43]

Format and readability - how about:

  output = _git_execute(
      options.git_dir,
      ["show",
       "--quiet",
       "--date=iso",
       "--format=%ad # %H # %s",
       hash],
       verbose,
  )
  print pre_text + output.strip()

[Michael Hablich, 2015/04/27 15:39:00]

Done.

+
+  text_to_print = pre_text + (
+      (_git_execute(
+          options.git_dir,
+          (
+              ["show",
+              "--quiet",
+              "--date=iso",
+              hash,
+              "--format=%ad # %H # %s"]),
+          verbose)).strip())
+  print text_to_print
+
+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)
+  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:
+        _pretty_print_entry(current_key, "+", options.verbose)
+      else:
+        print "+" + current_key
+
+      found_commits = all_related_commits[current_key]
+      for current_commit in found_commits:
+        if options.prettyprint:
+          _pretty_print_entry(current_commit, "| ", options.verbose)
+        else:
+          print "| " + current_commit
+  else:
+    hits = search_related_commits(options.git_dir, options.of[0],
+        options.until[0], options.deadline, options.verbose)
+    if len(hits) > 0:
+      print "\n".join(hits)