diff mbox series

[v5,05/14] packfile-uri.txt: support for excluding commits and trees

Message ID 8e5bf4010cb0d7e8b4f6c7285fe1f5365af955e8.1629805396.git.dyroneteng@gmail.com (mailing list archive)
State New, archived
Headers show
Series packfile-uris: commits, trees and tags exclusion | expand

Commit Message

Teng Long Aug. 25, 2021, 2:21 a.m. UTC
Signed-off-by: Teng Long <dyroneteng@gmail.com>
---
 Documentation/technical/packfile-uri.txt | 32 ++++++++++++++++--------
 1 file changed, 21 insertions(+), 11 deletions(-)

Comments

Ævar Arnfjörð Bjarmason Aug. 25, 2021, 11:52 p.m. UTC | #1
On Wed, Aug 25 2021, Teng Long wrote:

>  Client design
>  -------------
> @@ -65,9 +78,6 @@ The protocol design allows some evolution of the server and client without any
>  need for protocol changes, so only a small-scoped design is included here to
>  form the MVP. For example, the following can be done:
>  
> - * On the server, more sophisticated means of excluding objects (e.g. by
> -   specifying a commit to represent that commit and all objects that it
> -   references).
>   * On the client, resumption of clone. If a clone is interrupted, information
>     could be recorded in the repository's config and a "clone-resume" command
>     can resume the clone in progress. (Resumption of subsequent fetches is more
> @@ -78,4 +88,4 @@ There are some possible features that will require a change in protocol:
>  
>   * Additional HTTP headers (e.g. authentication)
>   * Byte range support
> - * Different file formats referenced by URIs (e.g. raw object)
> + * Different file formats referenced by URIs (e.g. raw object)
> \ No newline at end of file

Newline churn?
Teng Long Sept. 2, 2021, 11:23 a.m. UTC | #2
> Newline churn?

Agree.

I have no impression why the NEWLINE was deleted, will be fixed
in the next patch.

Thank you.
diff mbox series

Patch

diff --git a/Documentation/technical/packfile-uri.txt b/Documentation/technical/packfile-uri.txt
index f7eabc6c76..c3e4873956 100644
--- a/Documentation/technical/packfile-uri.txt
+++ b/Documentation/technical/packfile-uri.txt
@@ -35,13 +35,26 @@  include some sort of non-trivial implementation in the Minimum Viable Product,
 at least so that we can test the client.
 
 This is the implementation: a feature, marked experimental, that allows the
-server to be configured by one or more `uploadpack.blobPackfileUri=<sha1>
-<uri>` entries. Whenever the list of objects to be sent is assembled, all such
-blobs are excluded, replaced with URIs. As noted in "Future work" below, the
-server can evolve in the future to support excluding other objects (or other
-implementations of servers could be made that support excluding other objects)
-without needing a protocol change, so clients should not expect that packfiles
-downloaded in this way only contain single blobs.
+server to be configured by one or more entries with the format:
+
+    uploadpack.excludeobject=<object-hash> <pack-hash> <uri>
+
+Value <object-hash> is the key of entry, and the object type can be a blob,
+tree, or commit. The exclusion of tree and commit is recursive by default,
+which means that when a tree or commit object is excluded, the object itself
+and all reachable objects of the object will be excluded recursively. Whenever
+the list of objects to be sent is assembled, all such objects are excluded,
+replaced with URIs.
+
+Configuration compatibility
+-------------
+
+The old configuration of packfile-uri:
+
+	`uploadpack.blobPackfileUri=<object-hash> <pack-hash> <uri>`
+
+For the old configuration is compatible with the new one, but it only
+supports the exclusion of blob objects.
 
 Client design
 -------------
@@ -65,9 +78,6 @@  The protocol design allows some evolution of the server and client without any
 need for protocol changes, so only a small-scoped design is included here to
 form the MVP. For example, the following can be done:
 
- * On the server, more sophisticated means of excluding objects (e.g. by
-   specifying a commit to represent that commit and all objects that it
-   references).
  * On the client, resumption of clone. If a clone is interrupted, information
    could be recorded in the repository's config and a "clone-resume" command
    can resume the clone in progress. (Resumption of subsequent fetches is more
@@ -78,4 +88,4 @@  There are some possible features that will require a change in protocol:
 
  * Additional HTTP headers (e.g. authentication)
  * Byte range support
- * Different file formats referenced by URIs (e.g. raw object)
+ * Different file formats referenced by URIs (e.g. raw object)
\ No newline at end of file