Skip to content

Bundle URIs 4.5: fixups from part IV

Derrick Stolee via GitGitGadget git 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest3 patches 13 repliesIn review

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/3bundle-uri: drop unused 'uri' parameterbuiltin/clone.c bundle-uri.c bundle-uri.h+5 -66 replies
2/3bundle-uri: advertise based on repo configbundle-uri.c+1 -12 replies
3/3bundle-uri: remove GIT_TEST_BUNDLE_URI env variablet/lib-bundle-uri-protocol.sh t/t5601-clone.sh transport.c+14 -113 replies

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.

Read the v1 thread

Cover letter [PATCH 0/3] Bundle URIs 4.5: fixups from part IV

These patches represent small fixups that came in review from the last version of 'ds/bundle-uri-4'. Since it was merged to 'next', these patches are forward-fixes on that branch.

Note: I did not include any changes that could be solved by adding an UNUSED macro, saving that for Peff and his already-prepared patches in that area.

Thanks, -Stolee

Derrick Stolee (3):
bundle-uri: drop unused 'uri' parameter
bundle-uri: advertise based on repo config
bundle-uri: remove GIT_TEST_BUNDLE_URI env variable

 builtin/clone.c              |  1 -
 bundle-uri.c                 |  4 ++--
 bundle-uri.h                 |  8 ++++----
 t/lib-bundle-uri-protocol.sh | 12 ++++++++----
 t/t5601-clone.sh             |  8 ++++----
 transport.c                  |  5 ++---
 6 files changed, 20 insertions(+), 18 deletions(-)

base-commit: cec58f9965c845be74753aac6f49b4f177faa064
Published-As: https://github.com/gitgitgadget/git/releases/tag/pr-1443%2Fderrickstolee%2Fbundle-redo%2Fadvertise-fix-v1
Fetch-It-Via: git fetch https://github.com/gitgitgadget/git pr-1443/derrickstolee/bundle-redo/advertise-fix-v1
Pull-Request: https://github.com/gitgitgadget/git/pull/1443

Signature · 1 line
gitgitgadget

Read the cover letter in its thread