{"thread":{"id":"9156","subject":"[PATCH 0/4] Modularize http, commit-walker, http-fetch","startedAt":"2007-07-22T22:09:16Z","lastAt":"2007-07-22T22:14:26Z","messageCount":2,"participants":["Daniel Barkalow"],"isPatch":true,"patchVersion":1,"patchTotal":4},"messages":[{"id":"48209","messageId":"Pine.LNX.4.64.0707221745440.29679@iabervon.org","threadId":"9156","inReplyTo":null,"subject":"[PATCH 0/4] Modularize http, commit-walker, http-fetch","fromName":"Daniel Barkalow","fromEmail":"barkalow@iabervon.org","sentAt":"2007-07-22T22:09:16Z","receivedAt":"2007-07-22T22:09:16Z","isPatch":true,"sender":{"key":"barkalow@iabervon.org","avatar":"https://avatars.githubusercontent.com/u/55364219?v=4"},"body":"This series cleans up a whole lot of flow control and makes it possible to \nextend commit-walkers and http (each individually) in more useful \ndirections.\n\nDue to the limitations on what I can test, it could use a bunch more \ntesting before it goes in. In particular, http-push is completely \nuntested, and I haven't really beaten on USE_CURL_MULTI, although it \ncompiles each way.\n\nThe first patch just makes the http library code easier to work with.\n\nThe second patch turns the caller-provided global function for making \ndelayed requests into a callback in a chain.\n\nThe third patch (posted previously) removes obsolete commit-walkers.\n\nThe fourth patch (posted previously) reworks the commit-walker framework, \nand makes http-fetch into a commit-walker implementation which can be used \nfrom library code. It also turns http-fetch into a (trivial) builtin which \ncalls the walker code with the library walker.\n\n Makefile                      |   37 ++----\n builtin-http-fetch.c          |   77 +++++++++++++\n builtin.h                     |    1 +\n fetch.h                       |   54 ---------\n git.c                         |    3 +\n http-push.c                   |   25 +---\n http-fetch.c => http-walker.c |  245 +++++++++++++++++----------------------\n http.c                        |   50 ++++++++\n http.h                        |    7 +-\n local-fetch.c                 |  254 -----------------------------------------\n rsh.c                         |   83 -------------\n rsh.h                         |    7 -\n ssh-fetch.c                   |  166 ---------------------------\n ssh-pull.c                    |    4 -\n ssh-push.c                    |    4 -\n ssh-upload.c                  |  143 -----------------------\n fetch.c => walker.c           |   83 +++++++-------\n walker.h                      |   37 ++++++\n 18 files changed, 339 insertions(+), 941 deletions(-)\n\n\t-Daniel\n*This .sig left intentionally blank*\n"},{"id":"48218","messageId":"Pine.LNX.4.64.0707221811090.29679@iabervon.org","threadId":"9156","inReplyTo":"Pine.LNX.4.64.0707221745440.29679@iabervon.org","subject":"Re: [PATCH 0/4] Modularize http, commit-walker, http-fetch","fromName":"Daniel Barkalow","fromEmail":"barkalow@iabervon.org","sentAt":"2007-07-22T22:14:26Z","receivedAt":"2007-07-22T22:14:26Z","isPatch":true,"sender":{"key":"barkalow@iabervon.org","avatar":"https://avatars.githubusercontent.com/u/55364219?v=4"},"body":"I just realized I forgot to use -C, so some of these patches are probably \nless than trivial to review as changes. If anybody wants to be able to \nactually read them all without applying them, I have -C versions; if \nnobody does, I'd just as soon not spam the list further.\n\n\t-Daniel\n*This .sig left intentionally blank*\n"}]}