From 519c4b7298516c849b4a510ad2fd691d662de7d6 Mon Sep 17 00:00:00 2001 From: Patrick Georgi Date: Thu, 22 Sep 2016 12:46:45 +0200 Subject: [PATCH] util/scripts: add gerrit-rebase script gerrit-rebase is a gerrit-context aware rebase script. Given a source and a target branch (that need to have a common ancestor), it prepares a rebase todo list that applies all commits from source that aren't already found on target. It matches commits using Reviewed-on lines in the commit message that are added by gerrit when submitting commits using the "cherry-pick" strategy. This has been shown to be the best preserved meta data to work from in existing data (Change-Id was mangled in all kinds of ways). Change-Id: I9618c1b66ebc1fb7ed006efbc1665fb08386e1a5 Signed-off-by: Patrick Georgi Reviewed-on: https://review.coreboot.org/16695 Tested-by: build bot (Jenkins) Reviewed-by: Paul Menzel Reviewed-by: Martin Roth --- util/scripts/gerrit-rebase | 61 ++++++++++++++++++++++++++++++++++++++ 1 file changed, 61 insertions(+) create mode 100755 util/scripts/gerrit-rebase diff --git a/util/scripts/gerrit-rebase b/util/scripts/gerrit-rebase new file mode 100755 index 0000000000..002c499f8c --- /dev/null +++ b/util/scripts/gerrit-rebase @@ -0,0 +1,61 @@ +#!/bin/bash +# $0 from-branch to-branch +# applies all commits that from-branch has over to-branch, +# based on a common ancestor and gerrit meta-data +from=$1 +to=$2 + +# match string: this is the git commit line that is used to +# identify commits that were already copied over. +# +# Must not contain spaces except for leading and trailing. +# +# The first pick was Change-Id, but it was lost too often, +# so go for Reviewed-on instead. It's also unique because it +# contains the gerrit instance's host name and the change's number +# on that system. +match_string='^ [-A-Za-z]*[Rr]eviewed-on: ' + +# fetch common ancestor +common_base=$(git merge-base ${from} ${to} 2>/dev/null) + +if [ -z "${common_base}" ]; then + echo \"${from}\" or \"${to}\" is not a valid branch name. + exit 1 +fi + +# collect matches that are present on the target side +to_matches="$(git log ${common_base}..${to} | \ + grep "${match_string}" | \ + cut -d: -f2-)" + +# start rebase process, but fail immediately by enforcing an invalid todo +GIT_SEQUENCE_EDITOR="echo foo >" \ + git rebase -i --onto ${to} ${from} ${to} 2>/dev/null + +# write new rebase todo +# the appended "commit" line triggers handling of the last log entry +commit="" +(git log --reverse ${common_base}..${from} | \ + grep -E "(^commit [0-9a-f]{40}\$|${match_string})"; \ + echo "commit") | \ +while read key value; do + if [ "${key}" = "commit" ]; then + if [ -n "${commit}" ]; then + git log -n 1 --pretty="pick %h %s" ${commit} + fi + commit="${value}" + else + # if value was already found on the "to" side, skip this + # commit + if [[ ${to_matches} == *"${value}"* ]]; then + commit="" + fi + fi +done | GIT_SEQUENCE_EDITOR="cat >" git rebase --edit-todo + +# allow user to edit todo +git rebase --edit-todo + +# start processing todo to mimick git rebase -i behavior +git rebase --continue