From: Stefan Beller <sbeller@google.com>
To: dturner@twopensource.com
Cc: git@vger.kernel.org, Stefan Beller <sbeller@google.com>
Subject: [WIP PATCH 00/14] Protocol v2 patches
Date: Fri, 29 Apr 2016 16:34:33 -0700 [thread overview]
Message-ID: <1461972887-22100-1-git-send-email-sbeller@google.com> (raw)
Hi David,
here are my patches for a protocol v2.
("Negotiate capabilities before doing anything else", or as code:
static void upload_pack_version_2(void)
{
send_capabilities_version_2();
receive_capabilities_version_2();
/* The rest of the protocol stays the same, capabilities advertising
is disabled though. */
advertise_capabilities = 0;
upload_pack();
}
)
They are rough and unfinished as you can see by the tailing WIPs.
However the plumbing (upload-pack and fetch-pack) works and we'd need to
integrate that into user porcelains, i.e. fetch, clone, push.
Also we need to add tests for all the options again, so we'd need to be smart
about testing that.
I am not sure if it makes sense to integrate that with the http series, though.
Thanks,
Stefan
Nguyễn Thái Ngọc Duy (1):
upload-pack: make client capability parsing code a separate function
Stefan Beller (13):
upload-pack.c: Refactor capability advertising
upload-pack-2: Implement the version 2 of upload-pack
connect: rewrite feature parsing to work on string_list
transport: add infrastructure to support a protocol version number
remote.h: add get_remote_capabilities, request_capabilities
fetch-pack: move capability selection out of do_fetch_pack
fetch-pack: factor out get_selected_capabilities_list
fetch-pack: Add negotiate_capabilities
do_fetch_pack: select capabilities for transport version 1 only
builtin/fetch-pack: add argument for transport version
Add test for fetch-pack
WIP add test for git pull
WIP test git fetch
.gitignore | 1 +
Makefile | 4 ++
builtin/fetch-pack.c | 20 ++++++-
builtin/receive-pack.c | 15 +++--
connect.c | 141 +++++++++++++++++++++++++++++--------------
connect.h | 2 +-
fetch-pack.c | 102 ++++++++++++++++++++++++-------
fetch-pack.h | 7 +++
remote.c | 2 +
remote.h | 5 ++
t/t5500-fetch-pack.sh | 21 +++++++
t/t5510-fetch.sh | 5 ++
t/t5520-pull.sh | 6 ++
transport-helper.c | 1 +
transport.c | 20 ++++++-
transport.h | 8 +++
upload-pack-2.c | 1 +
upload-pack.c | 159 ++++++++++++++++++++++++++++++++++++-------------
18 files changed, 403 insertions(+), 117 deletions(-)
create mode 120000 upload-pack-2.c
--
2.8.0.32.g71f8beb.dirty
next reply other threads:[~2016-04-29 23:35 UTC|newest]
Thread overview: 40+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-04-29 23:34 Stefan Beller [this message]
2016-04-29 23:34 ` [PATCH 01/14] upload-pack: make client capability parsing code a separate function Stefan Beller
2016-04-29 23:34 ` [PATCH 02/14] upload-pack.c: Refactor capability advertising Stefan Beller
2016-04-30 1:04 ` David Turner
2016-05-04 20:05 ` Junio C Hamano
2016-04-29 23:34 ` [PATCH 03/14] upload-pack-2: Implement the version 2 of upload-pack Stefan Beller
2016-05-02 17:43 ` David Turner
2016-05-02 17:51 ` Stefan Beller
2016-05-02 18:56 ` David Turner
2016-05-03 0:31 ` Duy Nguyen
2016-05-04 20:11 ` Junio C Hamano
2016-04-29 23:34 ` [PATCH 04/14] connect: rewrite feature parsing to work on string_list Stefan Beller
2016-05-02 18:18 ` David Turner
2016-05-02 18:46 ` Stefan Beller
2016-05-04 20:13 ` Junio C Hamano
2016-05-17 22:23 ` David Turner
2016-04-29 23:34 ` [PATCH 05/14] transport: add infrastructure to support a protocol version number Stefan Beller
2016-04-29 23:34 ` [PATCH 06/14] remote.h: add get_remote_capabilities, request_capabilities Stefan Beller
2016-05-02 18:57 ` David Turner
2016-05-03 5:33 ` Jeff King
2016-05-03 21:21 ` David Turner
2016-05-04 16:44 ` Stefan Beller
2016-04-29 23:34 ` [PATCH 07/14] fetch-pack: move capability selection out of do_fetch_pack Stefan Beller
2016-04-29 23:34 ` [PATCH 08/14] fetch-pack: factor out get_selected_capabilities_list Stefan Beller
2016-05-02 19:09 ` David Turner
2016-04-29 23:34 ` [PATCH 09/14] fetch-pack: Add negotiate_capabilities Stefan Beller
2016-04-29 23:34 ` [PATCH 10/14] do_fetch_pack: select capabilities for transport version 1 only Stefan Beller
2016-04-29 23:34 ` [PATCH 11/14] builtin/fetch-pack: add argument for transport version Stefan Beller
2016-04-29 23:34 ` [PATCH 12/14] Add test for fetch-pack Stefan Beller
2016-05-02 19:45 ` David Turner
2016-04-29 23:34 ` [PATCH 13/14] WIP add test for git pull Stefan Beller
2016-04-29 23:34 ` [PATCH 14/14] WIP test git fetch Stefan Beller
2016-05-02 20:41 ` [WIP PATCH 00/14] Protocol v2 patches David Turner
2016-05-02 20:43 ` Stefan Beller
2016-05-24 22:46 ` David Turner
2016-05-24 23:03 ` Duy Nguyen
2016-05-25 16:45 ` David Turner
2016-05-25 16:23 ` Junio C Hamano
2016-05-25 19:31 ` David Turner
2016-05-25 21:29 ` Jeff King
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: http://vger.kernel.org/majordomo-info.html
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1461972887-22100-1-git-send-email-sbeller@google.com \
--to=sbeller@google.com \
--cc=dturner@twopensource.com \
--cc=git@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://80x24.org/mirrors/git.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).