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

[PATCH 00/17] http*: refactor (v4)

From
Tay Ray Chuan <rctay89@gmail.com>
Date
May 29, 2009, 17:58 UTC
Message-ID
<20090530015828.2de63b0a.rctay89@gmail.com>
Junio: this replaces rc/http-push~10 in pu. I've included your
suggestions and changes.
Mike: I've only taken some of your patches from your WIP patch series
dated Jan 18 (http://kerneltrap.org/mailarchive/git/2009/1/18/4757804).
Here's a list of changes from Mike's patches:
- the contentious http_cleanup has been removed in patch #6 (cleanup).
- a new http_request return state, HTTP_START_FAILED, has been added.
- http_request prints an error message ("Unable to start request") when
  start_active_slot fails.
- a new http API method, http_error, has been added.
- in patch #11 (http_fetch_ref), the error ("Unable to start request")
  is still removed, like Mike did, since http_request already handles 
  it.
- in patch #12 (remote_exists), the patch has been re-titled from
  update_remote_info_refs.
Mike Hommey (7):
  Don't expect verify_pack() callers to set pack_size
  transport.c::get_refs_via_curl(): do not leak refs_url
  http.c: new functions for the http API
  transport.c::get_refs_via_curl(): use the new http API
  http.c::http_fetch_ref(): use the new http API
  http-push.c::remote_exists(): use the new http API
  http-push.c::fetch_symref(): use the new http API
Tay Ray Chuan (10):
  http-push, http-walker: style fixes
  http-walker: verify remote packs
  http*: copy string returned by sha1_to_hex
  http-push: do not SEGV after fetching a bad pack idx file
  http*: move common variables and macros to http.[ch]
  http: create function end_url_with_slash
  http*: add http_get_info_packs
  http: use new http API in fetch_index()
  http*: add helper methods for fetching packs
  http*: add helper methods for fetching objects (loose)
 http-push.c   |  536 ++++++---------------------------------------------
 http-walker.c |  530 ++++++++------------------------------------------
 http.c        |  598 ++++++++++++++++++++++++++++++++++++++++++++++++++++++---
 http.h        |   85 ++++++++
 pack-check.c  |    8 +-
 transport.c   |   33 +--
 6 files changed, 820 insertions(+), 970 deletions(-)
-- 
Cheers,
Ray Chuan
Message 1 of 1 in “http*: refactor (v4)”
  1. 00/17 http*: refactor (v4)Tay Ray Chuan, May 29, 2009

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.