Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add initial support for Git protocol v2 #1244

Merged
merged 12 commits into from
Jun 28, 2024
Merged

add initial support for Git protocol v2 #1244

Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
2b975ec
add initial support for Git protocol v2
stspdotname Jun 25, 2024
e543e98
keep dulwich pull command line similar to the Git CLI; suggested by J…
stspdotname Jun 25, 2024
26344f8
use None to represent unspecified optional parameters; suggested by J…
stspdotname Jun 25, 2024
325de7d
make the Git protocol version configurable from the command line
stspdotname Jun 25, 2024
2400933
make Git protocol v2 work over SSH connections
stspdotname Jun 25, 2024
2647c58
use dict() instead of deepcopy()
stspdotname Jun 25, 2024
65220bb
emulate Git's behaviour when a server does not support object filtering
stspdotname Jun 25, 2024
343ed68
set the default git protocol version via global constants
stspdotname Jun 25, 2024
5c1a5ec
don't hard-code protocol v2 and tweak version comparisons to be futur…
stspdotname Jun 25, 2024
5b2f74f
skip test_fetch_pack_no_side_band_64k if git protocol v2 is used
stspdotname Jun 25, 2024
0e18ede
run client compat tests with git protocol version 0, too
stspdotname Jun 25, 2024
f270f0c
blind attempt at fixing the unexpected-success "failures" on windows CI
stspdotname Jun 25, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
use dict() instead of deepcopy() 
suggested by Jelmer
  • Loading branch information
@stspdotname
stspdotname committed Jun 25, 2024
commit 2647c582be7258251ad5c0e7442c48023939930f
2 changes: 1 addition & 1 deletion tests/compat/test_client.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -434,7 +434,7 @@ def run_command(
cmd, path = command.split(" ")
cmd = cmd.split("-", 1)
path = path.replace("'", "")
env = copy.deepcopy(os.environ)
env = dict(os.environ)
if protocol_version is None or protocol_version == 2:
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we do the inverse here, i.e.

If protocol version is not set, set it to 2.

If protocol version != 0 then set this header.

That prepares us a bit better to future versions.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, we can do that. I don't anticipate a new protocol version in the foreseeable future, but it doesn't hurt to handle things that way.

env["GIT_PROTOCOL"] = "version=2"

Expand Down