X-Git-Url: https://git.verplant.org/?a=blobdiff_plain;f=commit-tree.c;h=0d504e7e4be10083da6e677bbcb95bc186fc6277;hb=180926636e47ecfe28d03cec493af75899994f0f;hp=e9069971a8bbe413ceb71f63fc48513a475a4d6f;hpb=c4bd907d959673a9113e32e53f65a279b9786f25;p=git.git

diff --git a/commit-tree.c b/commit-tree.c
index e9069971..0d504e7e 100644
--- a/commit-tree.c
+++ b/commit-tree.c
@@ -95,15 +95,28 @@ static void check_valid(unsigned char *sha1, const char *expect)
  * how multi-way merges are represented.
  */
 #define MAXPARENT (16)
+static unsigned char parent_sha1[MAXPARENT][20];
 
 static char *commit_tree_usage = "git-commit-tree <sha1> [-p <sha1>]* < changelog";
 
+static int new_parent(int idx)
+{
+	int i;
+	unsigned char *sha1 = parent_sha1[idx];
+	for (i = 0; i < idx; i++) {
+		if (!memcmp(parent_sha1[i], sha1, 20)) {
+			error("duplicate parent %s ignored", sha1_to_hex(sha1));
+			return 0;
+		}
+	}
+	return 1;
+}
+
 int main(int argc, char **argv)
 {
 	int i, len;
 	int parents = 0;
 	unsigned char tree_sha1[20];
-	unsigned char parent_sha1[MAXPARENT][20];
 	unsigned char commit_sha1[20];
 	char *gecos, *realgecos, *commitgecos;
 	char *email, *commitemail, realemail[1000];
@@ -124,7 +137,8 @@ int main(int argc, char **argv)
 		if (!b || strcmp(a, "-p") || get_sha1(b, parent_sha1[parents]))
 			usage(commit_tree_usage);
 		check_valid(parent_sha1[parents], "commit");
-		parents++;
+		if (new_parent(parents))
+			parents++;
 	}
 	if (!parents)
 		fprintf(stderr, "Committing initial tree %s\n", argv[1]);
@@ -153,7 +167,7 @@ int main(int argc, char **argv)
 		parse_date(audate, date, sizeof(date));
 	cmdate = gitenv("GIT_COMMITTER_DATE");
 	if (cmdate)
-		parse_date(audate, realdate, sizeof(realdate));
+		parse_date(cmdate, realdate, sizeof(realdate));
 
 	remove_special(gecos); remove_special(realgecos); remove_special(commitgecos);
 	remove_special(email); remove_special(realemail); remove_special(commitemail);