From: Avery Pennarun <apenwarr@gmail.com>
Date: Fri, 24 Apr 2009 21:53:10 +0000 (-0400)
Subject: Skip over empty commits.
X-Git-Tag: v1.7.11-rc0~176^2~85
X-Git-Url: https://git.lorimer.id.au/gitweb.git/diff_plain/768d6d10051cbf2d0261b9cf671b1f969fb77a61

Skip over empty commits.

But we still need to get rid of unnecessary merge commits somehow...
---

diff --git a/git-subtree.sh b/git-subtree.sh
index 1e1237f520..7ae71886f4 100755
--- a/git-subtree.sh
+++ b/git-subtree.sh
@@ -167,6 +167,32 @@ merge_msg()
 	EOF
 }
 
+tree_for_commit()
+{
+	git ls-tree "$1" -- "$dir" |
+	while read mode type tree name; do
+		assert [ "$name" = "$dir" ]
+		echo $tree
+		break
+	done
+}
+
+tree_changed()
+{
+	tree=$1
+	shift
+	if [ $# -ne 1 ]; then
+		return 0   # weird parents, consider it changed
+	else
+		ptree=$(tree_for_commit $1)
+		if [ "$ptree" != "$tree" ]; then
+			return 0   # changed
+		else
+			return 1   # not changed
+		fi
+	fi
+}
+
 cmd_split()
 {
 	debug "Splitting $dir..."
@@ -199,21 +225,24 @@ cmd_split()
 		newparents=$(cache_get $parents)
 		debug "  newparents: $newparents"
 		
-		git ls-tree $rev -- "$dir" |
-		while read mode type tree name; do
-			assert [ "$name" = "$dir" ]
-			debug "  tree is: $tree"
-			p=""
-			for parent in $newparents; do
-				p="$p -p $parent"
-			done
+		tree=$(tree_for_commit $rev)
+		debug "  tree is: $tree"
+		[ -z $tree ] && continue
+
+		p=""
+		for parent in $newparents; do
+			p="$p -p $parent"
+		done
 			
+		if tree_changed $tree $parents; then
 			newrev=$(copy_commit $rev $tree "$p") || exit $?
-			debug "  newrev is: $newrev"
-			cache_set $rev $newrev
-			cache_set latest_new $newrev
-			cache_set latest_old $rev
-		done || exit $?
+		else
+			newrev="$newparents"
+		fi
+		debug "  newrev is: $newrev"
+		cache_set $rev $newrev
+		cache_set latest_new $newrev
+		cache_set latest_old $rev
 	done || exit $?
 	latest_new=$(cache_get latest_new)
 	if [ -z "$latest_new" ]; then