git/list[1] front-page[2] threads[3] people[4] search[5] about
 

[PATCH v5 08/10] fast-import: add support to delete refs

From
Felipe Contreras <felipe.contreras@gmail.com>
Date
Oct 31, 2013, 09:36 UTC
Message-ID
<1383212197-14259-4-git-send-email-felipe.contreras@gmail.com>
In-Reply-To
<1383212197-14259-1-git-send-email-felipe.contreras@gmail.com>
Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>
---
 Documentation/git-fast-import.txt |  3 +++
 fast-import.c                     | 13 ++++++++++---
 t/t9300-fast-import.sh            | 18 ++++++++++++++++++
 3 files changed, 31 insertions(+), 3 deletions(-)
diff --git a/Documentation/git-fast-import.txt b/Documentation/git-fast-import.txt
index 73f9806..c49ede4 100644
--- a/Documentation/git-fast-import.txt
+++ b/Documentation/git-fast-import.txt
@@ -483,6 +483,9 @@ Marks must be declared (via `mark`) before they can be used.
 * Any valid Git SHA-1 expression that resolves to a commit.  See
   ``SPECIFYING REVISIONS'' in linkgit:gitrevisions[7] for details.
 
+* The special null SHA-1 (40 zeros) specifices that the branch is to be
+  removed.
+
 The special case of restarting an incremental import from the
 current branch value should be written as:
 ----
diff --git a/fast-import.c b/fast-import.c
index f4d9969..fdce0b7 100644
--- a/fast-import.c
+++ b/fast-import.c
@@ -248,6 +248,7 @@ struct branch {
 	uintmax_t last_commit;
 	uintmax_t num_notes;
 	unsigned active : 1;
+	unsigned delete : 1;
 	unsigned pack_id : PACK_ID_BITS;
 	unsigned char sha1[20];
 };
@@ -1690,10 +1691,13 @@ static int update_branch(struct branch *b)
 	struct ref_lock *lock;
 	unsigned char old_sha1[20];
 
-	if (is_null_sha1(b->sha1))
-		return 0;
 	if (read_ref(b->name, old_sha1))
 		hashclr(old_sha1);
+	if (is_null_sha1(b->sha1)) {
+		if (b->delete)
+			delete_ref(b->name, old_sha1, 0);
+		return 0;
+	}
 	lock = lock_any_ref_for_update(b->name, old_sha1, 0, NULL);
 	if (!lock)
 		return error("Unable to lock %s", b->name);
@@ -2620,8 +2624,11 @@ static int parse_from(struct branch *b)
 			free(buf);
 		} else
 			parse_from_existing(b);
-	} else if (!get_sha1(from, b->sha1))
+	} else if (!get_sha1(from, b->sha1)) {
 		parse_from_existing(b);
+		if (is_null_sha1(b->sha1))
+			b->delete = 1;
+	}
 	else
 		die("Invalid ref name or SHA1 expression: %s", from);
 
diff --git a/t/t9300-fast-import.sh b/t/t9300-fast-import.sh
index 88fc407..f453388 100755
--- a/t/t9300-fast-import.sh
+++ b/t/t9300-fast-import.sh
@@ -2999,4 +2999,22 @@ test_expect_success 'T: ls root tree' '
 	test_cmp expect actual
 '
 
+test_expect_success 'T: delete branch' '
+	git branch to-delete &&
+	git fast-import <<-EOF &&
+	reset refs/heads/to-delete
+	from 0000000000000000000000000000000000000000
+	EOF
+	test_must_fail git rev-parse --verify refs/heads/to-delete
+'
+
+test_expect_success 'T: empty reset doesnt delete branch' '
+	git branch not-to-delete &&
+	git fast-import <<-EOF &&
+	reset refs/heads/not-to-delete
+	EOF
+	git show-ref &&
+	git rev-parse --verify refs/heads/not-to-delete
+'
+
 test_done
-- 
1.8.4.2+fc1
Previous: Junio C HamanoNext: Max Horn
Message 15 of 45 in “transport-helper: add support to delete branches”
  1. 10/10 transport-helper: add support to delete branchesFelipe Contreras, Oct 31, 2013
  2. 00/10 transport-helper: updatesFelipe Contreras, Oct 31, 2013
  3. Max HornOct 31, 2013
  4. Junio C HamanoOct 31, 2013
  5. Felipe ContrerasOct 31, 2013
  6. Junio C HamanoOct 31, 2013
  7. Max HornOct 31, 2013
  8. 11/10 test-hg.sh: tests are now expected to passRichard Hansen, Nov 11, 2013
  9. 12/10 remote-bzr: support the new 'force' optionRichard Hansen, Nov 11, 2013
  10. Felipe ContrerasNov 11, 2013
  11. Richard HansenNov 11, 2013
  12. Felipe ContrerasNov 11, 2013
  13. 01/10 transport-helper: fix extra linesFelipe Contreras, Oct 31, 2013
  14. Junio C HamanoOct 31, 2013
  15. 08/10 fast-import: add support to delete refsFelipe Contreras, Oct 31, 2013
  16. Max HornOct 31, 2013
  17. 05/10 fast-export: improve argument parsingFelipe Contreras, Oct 31, 2013
  18. 06/10 fast-export: add new --refspec optionFelipe Contreras, Oct 31, 2013
  19. Junio C HamanoOct 31, 2013
  20. Felipe ContrerasOct 31, 2013
  21. Junio C HamanoNov 6, 2013
  22. Jeff KingNov 6, 2013
  23. Junio C HamanoNov 6, 2013
  24. Junio C HamanoNov 6, 2013
  25. Junio C HamanoNov 8, 2013
  26. Jeff KingNov 9, 2013
  27. Felipe ContrerasNov 7, 2013
  28. 03/10 transport-helper: add 'force' to 'export' helpersFelipe Contreras, Oct 31, 2013
  29. Junio C HamanoOct 31, 2013
  30. Felipe ContrerasOct 31, 2013
  31. Junio C HamanoOct 31, 2013
  32. Junio C HamanoNov 1, 2013
  33. Felipe ContrerasNov 1, 2013
  34. Junio C HamanoNov 1, 2013
  35. Eric SunshineNov 1, 2013
  36. fixup! transport-helper: add 'force' to 'export' helpersRichard Hansen, Nov 11, 2013
  37. 09/10 fast-export: add support to delete refsFelipe Contreras, Oct 31, 2013
  38. Max HornOct 31, 2013
  39. Felipe ContrerasOct 31, 2013
  40. Max HornOct 31, 2013
  41. Felipe ContrerasOct 31, 2013
  42. Max HornOct 31, 2013
  43. 07/10 transport-helper: add support for old:new refspecFelipe Contreras, Oct 31, 2013
  44. 04/10 transport-helper: check for 'forced update' messageFelipe Contreras, Oct 31, 2013
  45. 02/10 transport-helper: don't update refs in dry-runFelipe Contreras, Oct 31, 2013

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.