Commit eee67fe5 authored by Andrew Grieve's avatar Andrew Grieve Committed by LUCI CQ

Fix git-crrev-parse to work with both master & main branches

Was finding only "master" commits.

Change-Id: I434a0cb8d2112d6173e638814ad10f401c2fb9b3
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/3226313Reviewed-by: 's avatarJosip Sokcevic <sokcevic@google.com>
Reviewed-by: 's avatarDirk Pranke <dpranke@google.com>
Commit-Queue: Josip Sokcevic <sokcevic@google.com>
Auto-Submit: Andrew Grieve <agrieve@chromium.org>
parent 650f853c
...@@ -7,17 +7,17 @@ ...@@ -7,17 +7,17 @@
# This git extension converts a chromium commit number to its git commit hash. # This git extension converts a chromium commit number to its git commit hash.
# It accepts the following input formats: # It accepts the following input formats:
# #
# $ git crrev-parse Cr-Commit-Position: refs/heads/master@{#311769} # $ git crrev-parse Cr-Commit-Position: refs/heads/main@{#311769}
# $ git crrev-parse ' Cr-Commit-Position: refs/heads/master@{#311769}' # $ git crrev-parse ' Cr-Commit-Position: refs/heads/main@{#311769}'
# $ git crrev-parse 'Cr-Commit-Position: refs/heads/master@{#311769}' # $ git crrev-parse 'Cr-Commit-Position: refs/heads/main@{#311769}'
# $ git crrev-parse refs/heads/master@{#311769} # $ git crrev-parse refs/heads/main@{#311769}
# #
# It also works for branches (assuming you have branches in your local # It also works for branches (assuming you have branches in your local
# checkout): # checkout):
# #
# $ git crrev-parse refs/branch-heads/2278@{#2} # $ git crrev-parse refs/branch-heads/2278@{#2}
# #
# If you don't specify a branch, refs/heads/master is assumed: # If you don't specify a branch, refs/heads/main is assumed:
# #
# $ git crrev-parse @{#311769} # $ git crrev-parse @{#311769}
# $ git crrev-parse 311769 # $ git crrev-parse 311769
...@@ -35,7 +35,7 @@ while [ -n "$1" ]; do ...@@ -35,7 +35,7 @@ while [ -n "$1" ]; do
fi fi
ref="${commit_pos%@\{#*\}}" ref="${commit_pos%@\{#*\}}"
if [ "$ref" = "$commit_pos" -o -z "$ref" ]; then if [ "$ref" = "$commit_pos" -o -z "$ref" ]; then
ref="refs/heads/master" ref="refs/heads/main"
fi fi
remote_ref="${ref/refs\/heads/refs\/remotes\/origin}" remote_ref="${ref/refs\/heads/refs\/remotes\/origin}"
remote_ref="${remote_ref/refs\/branch-heads/refs\/remotes\/branch-heads}" remote_ref="${remote_ref/refs\/branch-heads/refs\/remotes\/branch-heads}"
...@@ -45,8 +45,14 @@ while [ -n "$1" ]; do ...@@ -45,8 +45,14 @@ while [ -n "$1" ]; do
if [ -z "$ref" -o -z "$num" ]; then if [ -z "$ref" -o -z "$num" ]; then
git rev-parse "$1" git rev-parse "$1"
else else
grep_str="^Cr-Commit-Position: $ref@{#$num}" # When the ref is not specified on the command-line, accept either
git rev-list -n 1 --grep="$grep_str" "$remote_ref" # Cr-refs/heads/master@{#$NUM} or refs/heads/main@{#$NUM}.
if [[ "$ref" = "refs/heads/main" ]]; then
grep_str="^Cr-Commit-Position: refs/heads/(master|main)@\\{#$num\\}"
else
grep_str="^Cr-Commit-Position: $ref@\\{#$num\\}"
fi
git rev-list -n 1 --extended-regexp --grep="$grep_str" "$remote_ref"
fi fi
shift shift
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment