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

[PATCH v4 07/22] send-pack: always send capabilities

From
Junio C Hamano <gitster@pobox.com>
Date
Sep 5, 2014, 20:54 UTC
Message-ID
<1409950510-10209-8-git-send-email-gitster@pobox.com>
In-Reply-To
<1409950510-10209-1-git-send-email-gitster@pobox.com>

We tried to avoid sending one extra byte, NUL and nothing behind it to signal there is no protocol capabilities being sent, on the first command packet on the wire, but it just made the code look ugly.

Signed-off-by: Junio C Hamano <gitster@pobox.com>
---
 send-pack.c | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/send-pack.c b/send-pack.c
index 43e98fa..e81f741 100644
--- a/send-pack.c
+++ b/send-pack.c
@@ -281,8 +281,7 @@ int send_pack(struct send_pack_args *args,
 			char *new_hex = sha1_to_hex(ref->new_sha1);
 			int quiet = quiet_supported && (args->quiet || !args->progress);
 
-			if (!cmds_sent && (status_report || use_sideband ||
-					   quiet || agent_supported)) {
+			if (!cmds_sent)
 				packet_buf_write(&req_buf,
 						 "%s %s %s%c%s%s%s%s%s",
 						 old_hex, new_hex, ref->name, 0,
@@ -292,7 +291,6 @@ int send_pack(struct send_pack_args *args,
 						 agent_supported ? " agent=" : "",
 						 agent_supported ? git_user_agent_sanitized() : ""
 						);
-			}
 			else
 				packet_buf_write(&req_buf, "%s %s %s",
 						 old_hex, new_hex, ref->name);
-- 
2.1.0-399-g2df620b
Previous: Junio C HamanoNext: Junio C Hamano
Message 8 of 23 in “Signed push”
  1. 00/22 Signed pushJunio C Hamano, Sep 5, 2014
  2. 01/22 receive-pack: do not overallocate command structureJunio C Hamano, Sep 5, 2014
  3. 02/22 receive-pack: parse feature request a bit earlierJunio C Hamano, Sep 5, 2014
  4. 03/22 receive-pack: do not reuse old_sha1[] for other thingsJunio C Hamano, Sep 5, 2014
  5. 04/22 receive-pack: factor out queueing of commandJunio C Hamano, Sep 5, 2014
  6. 05/22 send-pack: move REF_STATUS_REJECT_NODELETE logic a bit higherJunio C Hamano, Sep 5, 2014
  7. 06/22 send-pack: refactor decision to send update per refJunio C Hamano, Sep 5, 2014
  8. 07/22 send-pack: always send capabilitiesJunio C Hamano, Sep 5, 2014
  9. 08/22 send-pack: factor out capability string generationJunio C Hamano, Sep 5, 2014
  10. 09/22 receive-pack: factor out capability string generationJunio C Hamano, Sep 5, 2014
  11. 10/22 send-pack: rename "new_refs" to "need_pack_data"Junio C Hamano, Sep 5, 2014
  12. 11/22 send-pack: refactor inspecting and resetting status and sending commandsJunio C Hamano, Sep 5, 2014
  13. 12/22 send-pack: clarify that cmds_sent is a booleanJunio C Hamano, Sep 5, 2014
  14. 13/22 gpg-interface: move parse_gpg_output() to where it should beJunio C Hamano, Sep 5, 2014
  15. 14/22 gpg-interface: move parse_signature() to where it should beJunio C Hamano, Sep 5, 2014
  16. 15/22 pack-protocol doc: typofix for PKT-LINEJunio C Hamano, Sep 5, 2014
  17. 16/22 push: the beginning of "git push --signed"Junio C Hamano, Sep 5, 2014
  18. 17/22 receive-pack: GPG-validate push certificatesJunio C Hamano, Sep 5, 2014
  19. 18/22 send-pack: send feature request on push-cert packetJunio C Hamano, Sep 5, 2014
  20. 19/22 signed push: remove duplicated protocol infoJunio C Hamano, Sep 5, 2014
  21. 20/22 signed push: add "pushee" header to push certificateJunio C Hamano, Sep 5, 2014
  22. 21/22 signed push: fortify against replay attacksJunio C Hamano, Sep 5, 2014
  23. 22/22 signed push: allow stale nonce in stateless modeJunio C Hamano, Sep 5, 2014

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.