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

[PATCH v5 00/10] transport-helper: updates

From
Felipe Contreras <felipe.contreras@gmail.com>
Date
Oct 31, 2013, 09:36 UTC
Message-ID
<1383212197-14259-2-git-send-email-felipe.contreras@gmail.com>
In-Reply-To
<1383212197-14259-1-git-send-email-felipe.contreras@gmail.com>
Hi,

Here are the patches that allow transport helpers to be completely transparent; renaming branches, deleting them, custom refspecs, --force, --dry-run, reporting forced update, everything works.

Some of these were were sent before and rejected without a reason, but here they are again in case anybody is interested.

Diff from v3:
diff --git a/Documentation/gitremote-helpers.txt b/Documentation/gitremote-helpers.txt
index f1f4ca9..e75699c 100644
--- a/Documentation/gitremote-helpers.txt
+++ b/Documentation/gitremote-helpers.txt
@@ -437,6 +437,10 @@ set by Git if the remote helper has the 'option' capability.
 'option check-connectivity' \{'true'|'false'\}::
 	Request the helper to check connectivity of a clone.
 
+'option force' \{'true'|'false'\}::
+	Request the helper to perform a force update.  Defaults to
+	'false'.
+
 SEE ALSO
 --------
 linkgit:git-remote[1]
diff --git a/git-remote-testgit.sh b/git-remote-testgit.sh
index 6d2f282..716aa4c 100755
--- a/git-remote-testgit.sh
+++ b/git-remote-testgit.sh
@@ -39,6 +39,7 @@ do
 		fi
 		test -n "$GIT_REMOTE_TESTGIT_SIGNED_TAGS" && echo "signed-tags"
 		test -n "$GIT_REMOTE_TESTGIT_NO_PRIVATE_UPDATE" && echo "no-private-update"
+		echo 'option'
 		echo
 		;;
 	list)
@@ -93,6 +94,7 @@ do
 		before=$(git for-each-ref --format=' %(refname) %(objectname) ')
 
 		git fast-import \
+			${force:+--force} \
 			${testgitmarks:+"--import-marks=$testgitmarks"} \
 			${testgitmarks:+"--export-marks=$testgitmarks"} \
 			--quiet
@@ -115,6 +117,20 @@ do
 
 		echo
 		;;
+	option\ *)
+		read cmd opt val <<-EOF
+		$line
+		EOF
+		case $opt in
+		force)
+			test $val = "true" && force="true" || force=
+			echo "ok"
+			;;
+		*)
+			echo "unsupported"
+			;;
+		esac
+		;;
 	'')
 		exit
 		;;
diff --git a/t/t5801-remote-helpers.sh b/t/t5801-remote-helpers.sh
index be543c0..c667965 100755
--- a/t/t5801-remote-helpers.sh
+++ b/t/t5801-remote-helpers.sh
@@ -102,6 +102,19 @@ test_expect_success 'push delete branch' '
 	 rev-parse --verify refs/heads/new-name
 '
 
+test_expect_success 'forced push' '
+	(cd local &&
+	git checkout -b force-test &&
+	echo content >> file &&
+	git commit -a -m eight &&
+	git push origin force-test &&
+	echo content >> file &&
+	git commit -a --amend -m eight-modified &&
+	git push --force origin force-test
+	) &&
+	compare_refs local refs/heads/force-test server refs/heads/force-test
+'
+
 test_expect_success 'cloning without refspec' '
 	GIT_REMOTE_TESTGIT_REFSPEC="" \
 	git clone "testgit::${PWD}/server" local2 2>error &&

Felipe Contreras (10):
  transport-helper: fix extra lines
  transport-helper: don't update refs in dry-run
  transport-helper: add 'force' to 'export' helpers
  transport-helper: check for 'forced update' message
  fast-export: improve argument parsing
  fast-export: add new --refspec option
  transport-helper: add support for old:new refspec
  fast-import: add support to delete refs
  fast-export: add support to delete refs
  transport-helper: add support to delete branches

 Documentation/git-fast-export.txt   |  4 ++++
 Documentation/git-fast-import.txt   |  3 +++
 Documentation/gitremote-helpers.txt |  4 ++++
 builtin/fast-export.c               | 47 ++++++++++++++++++++++++++++++++++++-
 fast-import.c                       | 13 +++++++---
 git-remote-testgit.sh               | 16 +++++++++++++
 t/t5801-remote-helpers.sh           | 23 +++++++++++++++++-
 t/t9300-fast-import.sh              | 18 ++++++++++++++
 t/t9350-fast-export.sh              | 18 ++++++++++++++
 transport-helper.c                  | 47 +++++++++++++++++++++++++------------
 10 files changed, 173 insertions(+), 20 deletions(-)
-- 
1.8.4.2+fc1
Previous: Felipe ContrerasNext: Max Horn
Message 2 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.