From 29e377af6f259fd87067ad189f254621ba2e6703 Mon Sep 17 00:00:00 2001
From: Stephen Haberman <stephen@exigencecorp.com>
Date: Mon, 29 Sep 2008 22:47:38 -0500
Subject: [PATCH] Adding a functions and redoing the tattoo to use the r/X
 tags.

---
 scripts/functions            | 11 +++++
 scripts/gc-push              | 10 ++--
 scripts/gc-tattoo            | 90 +++++++-----------------------------
 tests/t5500-script-tattoo.sh |  7 ++-
 4 files changed, 38 insertions(+), 80 deletions(-)
 create mode 100644 scripts/functions

diff --git a/scripts/functions b/scripts/functions
new file mode 100644
index 0000000..b665bcb
--- /dev/null
+++ b/scripts/functions
@@ -0,0 +1,11 @@
+#!/bin/sh
+
+function set_branch_name() {
+	branch_name=$(git symbolic-ref --quiet HEAD)
+	if [[ $? -ne 0 ]] ; then
+		branch_name=""
+	else
+		branch_name=${branch_name/refs\/heads\//}
+	fi
+}
+
diff --git a/scripts/gc-push b/scripts/gc-push
index 4e72fe5..26ffa7b 100644
--- a/scripts/gc-push
+++ b/scripts/gc-push
@@ -1,13 +1,13 @@
 #!/bin/sh
 
-branch_name=$(git symbolic-ref --quiet HEAD)
-if [[ $? -ne 0 ]] ; then
-	echo "not on a branch"
+. $(dirname $0)/functions
+
+set_branch_name
+if [ "$branch_name" == "" ] ; then
+	echo "Not on a branch"
 	exit 1
 fi
 
-branch_name=${branch_name/refs\/heads\//}
-
 git push origin $branch_name
 if [[ $? -ne 0 ]] ; then
     exit $?
diff --git a/scripts/gc-tattoo b/scripts/gc-tattoo
index d7139ba..f1bd3a1 100644
--- a/scripts/gc-tattoo
+++ b/scripts/gc-tattoo
@@ -1,83 +1,27 @@
 #!/bin/sh
 
-#
-# Some docs
-#
-# A --- B --- C    stable
-# |\         /
-# | D ----- E      topic1
-#  \       /
-#   F --- G        topic2
-#
-# topic1 stable result
-#  D      A      contains, merge base=A
-#  D      C      contains, merge base=A
-#  E      A      contains, merge base=E
-#  E      C      contains, merge base=A
+. $(dirname $0)/functions
 
-#
-# * --- * --------- *   stable
-# |\     \
-# | A --- B ---- C      topic1
-# \             /
-#  D --- E --- F        topic2
-#
-# F: contains=topic1,topic2
-# C: contains=topic1
+git fetch
 
-head=$(git rev-parse HEAD)
+set_branch_name
 
-# Watch out for the very first commit in the repo because we use head^
-# Hopefully we can optimize this away later.
-git rev-parse --verify --quiet "$head^" >/dev/null
-if [ $? -ne 0 ] ; then
-	echo "0"
-	exit 0
-fi
+echo "branch_name='$branch_name'"
 
-contains=($(git branch -r --contains $head))
-if [ ${#contains[@]} -eq 0 ] ; then
-	echo "$head has not been pushed"
+if [ "$branch_name" == "" ] ; then
+	echo "Not on a branch (for $branch_name)"
 	exit 1
 fi
 
-potential=" "
-for branch in ${contains[@]} ; do
-	branch=${branch##origin/}
-	#echo "branch=$branch"
-
-	# Walk back until we hit a baserev that is not the branch_tip itself (because it was merged)
-	branch_tip=$(git rev-parse origin/$branch)
-	stable_rev=$(git rev-parse origin/stable)
-	stable_base=$(git merge-base "$branch_tip" "$stable_rev")
-
-	# echo "stable_base=$stable_base"
-	while [ "$stable_base" == "$branch_tip" ] ; do
-		stable_rev=$(git rev-parse "${stable_rev}^")
-		stable_base=$(git merge-base "$branch_tip" "$stable_rev")
-	done
-	# echo "stable_base=$stable_base"
-
-	git rev-list --first-parent $stable_base..$branch_tip | grep --quiet "$head"
-	if [ $? -eq 0 ] ; then
-		if [ "$branch" == "stable" ] ; then
-			describe=$(git describe $head 2>/dev/null)
-			if [ $? -eq 0 ] ; then
-				potential="$potential $describe"
-			else
-				potential="$potential stable-$head"
-			fi
-		else
-			number=$(git rev-list --first-parent "$stable_base..$head" | wc -l)
-			potential="$potential $branch-$number"
-		fi
-	fi
-done
-
-potential=($potential)
-if [ ${#potential[@]} -eq 1 ] ; then
-	echo "${potential[0]}"
-else
-	echo "unknown"
-fi
+describe=$(git describe --tags --always HEAD)
+case "$describe" in
+	r/*)
+		describe=${describe##r\/}
+		echo "$branch_name-$describe"
+		;;
+	*)
+		echo "No commit number tag found"
+		exit 1
+		;;
+esac
 
diff --git a/tests/t5500-script-tattoo.sh b/tests/t5500-script-tattoo.sh
index bab6dd2..561b4bf 100644
--- a/tests/t5500-script-tattoo.sh
+++ b/tests/t5500-script-tattoo.sh
@@ -19,11 +19,14 @@ test_expect_success 'setup' '
 
 # tattoo makes assumptions based on the update stable hook sanity checks
 install_update_hook 'update-stable'
+install_post_receive_hook 'post-receive-assign-commit-numbers'
 
-test_expect_success 'tattoo of first commit' '
-	gc-tattoo | grep 0
+test_expect_success 'tattoo of unpublished commit fails' '
+	gc-tattoo | grep "No commit number tag found"
 '
 
+test_done
+
 test_expect_success 'tattoo 1' '
 	git checkout -b topic1 &&
 	echo "$test_name" >a.topic1 &&
-- 
2.39.5