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

[PATCH 0/2] Skip ls-refs if possible for HTTP

From
JTJonathan Tan <jonathantanmy@google.com>
Date
Aug 21, 2019, 22:20 UTC
Message-ID
<cover.1566425828.git.jonathantanmy@google.com>

This was discovered by one of my colleagues when using a partial clone. I thought I had resolved the problem with the commits mentioned in patch 1 (e70a3030e7 and ancestors), but apparently that is not the case (that only worked for native protocols). So here is a fix for HTTP.

I'm not sure of the value of the test in patch 2, but that test does fail if I don't update fetch_refs_from_bundle() to first call get_refs_from_bundle() if it hasn't already been called.

Jonathan Tan (2):
  transport-helper: skip ls-refs if unnecessary
  transport: teach all vtables to allow fetch first
 t/t5607-clone-bundle.sh | 11 +++++++++++
 t/t5702-protocol-v2.sh  | 13 +++++++++++++
 transport-helper.c      | 38 ++++++++++++++++++++++++++++++++------
 transport-internal.h    |  6 ------
 transport.c             | 18 ++++++------------
 5 files changed, 62 insertions(+), 24 deletions(-)
-- 
2.23.0.187.g17f5b7556c-goog
Next: Jonathan Tan
Message 1 of 5 in “Skip ls-refs if possible for HTTP”
  1. 0/2 Skip ls-refs if possible for HTTPJonathan Tan, Aug 21, 2019
  2. 1/2 transport-helper: skip ls-refs if unnecessaryJonathan Tan, Aug 21, 2019
  3. 2/2 transport: teach all vtables to allow fetch firstJonathan Tan, Aug 21, 2019
  4. Junio C HamanoAug 22, 2019
  5. Jonathan TanAug 22, 2019

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.