From: Phillip Wood <phillip.wood123@gmail.com>
Date: Tue, 29 Apr 2025 09:00:13 GMT
Subject: Re: [PATCH v7 0/2] bundle-uri: copy all bundle references ino the refs/bundle space
Message-ID: <d71b3fc1-d2fc-436d-876c-0bd1475a88ce@gmail.com>
In-Reply-To: <pull.1897.v7.git.git.1745609589.gitgitgadget@gmail.com>

Hi Scott

Thank you for updating the documentation. This version looks good to me

Thanks

Phillip

On 25/04/2025 20:33, Scott Chacon via GitGitGadget wrote:
> Sorry everyone for the noise. There was a whitespace issue the tests
> complained about, hopefully this fixes it.
> 
>> bundle-uri: copy all bundle references ino the refs/bundle space
>> bundle-uri: update bundle clone tests with new refspec path
> 
> Scott Chacon (2):
>    bundle-uri: copy all bundle references ino the refs/bundle space
>    bundle-uri: add test for bundle-uri clones with tags
> 
>   Documentation/technical/bundle-uri.adoc |  14 +-
>   bundle-uri.c                            |   2 +-
>   t/t5558-clone-bundle-uri.sh             | 202 ++++++++++++++----------
>   3 files changed, 124 insertions(+), 94 deletions(-)
> 
> 
> base-commit: f65182a99e545d2f2bc22e6c1c2da192133b16a3
> Published-As: https://github.com/gitgitgadget/git/releases/tag/pr-git-1897%2Fschacon%2Fsc-more-bundle-refs-v7
> Fetch-It-Via: git fetch https://github.com/gitgitgadget/git pr-git-1897/schacon/sc-more-bundle-refs-v7
> Pull-Request: https://github.com/git/git/pull/1897
> 
> Range-diff vs v6:
> 
>   1:  d6ec5c87b43 ! 1:  1cbf0787d49 bundle-uri: copy all bundle references ino the refs/bundle space
>       @@ Documentation/technical/bundle-uri.adoc: will interact with bundle URIs accordin
>        -   introduce a numbered namespace (such as `refs/bundles/<i>/*`) such that
>        -   stale bundle refs can be deleted.
>        +   client unbundles that data using a refspec. The refspec used is
>       -+   `+refs/*:refs/bundles/*`. These refs are stored so that later
>       ++   `+refs/*:refs/bundles/*`. These refs are stored so that later
>        +   `git fetch` negotiations can communicate each bundled ref as a `have`,
>        +   reducing the size of the fetch over the Git protocol. To allow pruning
>        +   refs from this ref namespace, Git may introduce a numbered namespace
>   2:  825d2b01eae = 2:  7f36484e267 bundle-uri: add test for bundle-uri clones with tags
>