All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] doc: documentation for http.uploadarchive config option
@ 2025-02-05 21:20 Piotr Szlazak via GitGitGadget
  2025-02-06 14:04 ` Junio C Hamano
  2025-02-06 17:45 ` [PATCH v2] " Piotr Szlazak via GitGitGadget
  0 siblings, 2 replies; 4+ messages in thread
From: Piotr Szlazak via GitGitGadget @ 2025-02-05 21:20 UTC (permalink / raw)
  To: git; +Cc: Jiang Xin, Jeff King, Piotr Szlazak, Piotr Szlazak

From: Piotr Szlazak <piotr.szlazak@gmail.com>

In Git v2.44.0 support for 'git archive' over HTTP protocol
was added, but it was nowhere documented how it should be
enabled in git-http-backend.
This commit adds needed information.

Signed-off-by: Piotr Szlazak <piotr.szlazak@gmail.com>
---
    doc: documentation for http.uploadarchive config option

Published-As: https://github.com/gitgitgadget/git/releases/tag/pr-git-1885%2Fpszlazak%2Fdocument-http-uploadarchive-v1
Fetch-It-Via: git fetch https://github.com/gitgitgadget/git pr-git-1885/pszlazak/document-http-uploadarchive-v1
Pull-Request: https://github.com/git/git/pull/1885

 Documentation/git-http-backend.txt | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/Documentation/git-http-backend.txt b/Documentation/git-http-backend.txt
index f37ddaded82..1dea4268520 100644
--- a/Documentation/git-http-backend.txt
+++ b/Documentation/git-http-backend.txt
@@ -56,6 +56,10 @@ http.receivepack::
 	disabled by setting this item to `false`, or enabled for all
 	users, including anonymous users, by setting it to `true`.
 
+http.uploadarchive::
+	This serves 'git archive' clients for remote archive over HTTP/HTTPS
+	protocols. It is disabled by default. It only works in protocol v2.
+
 URL TRANSLATION
 ---------------
 To determine the location of the repository on disk, 'git http-backend'

base-commit: bc204b742735ae06f65bb20291c95985c9633b7f
-- 
gitgitgadget

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH] doc: documentation for http.uploadarchive config option
  2025-02-05 21:20 [PATCH] doc: documentation for http.uploadarchive config option Piotr Szlazak via GitGitGadget
@ 2025-02-06 14:04 ` Junio C Hamano
  2025-02-06 17:54   ` Piotr Szlazak
  2025-02-06 17:45 ` [PATCH v2] " Piotr Szlazak via GitGitGadget
  1 sibling, 1 reply; 4+ messages in thread
From: Junio C Hamano @ 2025-02-06 14:04 UTC (permalink / raw)
  To: Piotr Szlazak via GitGitGadget; +Cc: git, Jiang Xin, Jeff King, Piotr Szlazak

"Piotr Szlazak via GitGitGadget" <gitgitgadget@gmail.com> writes:

> From: Piotr Szlazak <piotr.szlazak@gmail.com>
>
> In Git v2.44.0 support for 'git archive' over HTTP protocol
> was added, but it was nowhere documented how it should be
> enabled in git-http-backend.

Good eyes

> This commit adds needed information.

It is more customary to say something like:

    Add missing documentation.

around here.

> diff --git a/Documentation/git-http-backend.txt b/Documentation/git-http-backend.txt
> index f37ddaded82..1dea4268520 100644
> --- a/Documentation/git-http-backend.txt
> +++ b/Documentation/git-http-backend.txt
> @@ -56,6 +56,10 @@ http.receivepack::
>  	disabled by setting this item to `false`, or enabled for all
>  	users, including anonymous users, by setting it to `true`.
>  
> +http.uploadarchive::
> +	This serves 'git archive' clients for remote archive over HTTP/HTTPS
> +	protocols. It is disabled by default. It only works in protocol v2.

The description looks good, from the correctness point-of-view,
and its style is in line with other entries before it.

We may want to later consider moving these four http.* configuration
variables somehow to Documentation/config/. directory so that it is
shared across git-http-backend.1 and git-config.1 pages, but that is
something we should leave outside the current effort.

Thanks.

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [PATCH v2] doc: documentation for http.uploadarchive config option
  2025-02-05 21:20 [PATCH] doc: documentation for http.uploadarchive config option Piotr Szlazak via GitGitGadget
  2025-02-06 14:04 ` Junio C Hamano
@ 2025-02-06 17:45 ` Piotr Szlazak via GitGitGadget
  1 sibling, 0 replies; 4+ messages in thread
From: Piotr Szlazak via GitGitGadget @ 2025-02-06 17:45 UTC (permalink / raw)
  To: git; +Cc: Jiang Xin, Jeff King, Piotr Szlazak, Piotr Szlazak

From: Piotr Szlazak <piotr.szlazak@gmail.com>

In Git v2.44.0 support for 'git archive' over HTTP protocol
was added, but it was nowhere documented how it should be
enabled in git-http-backend.

Add missing documentation.

Signed-off-by: Piotr Szlazak <piotr.szlazak@gmail.com>
---
    doc: documentation for http.uploadarchive config option

Published-As: https://github.com/gitgitgadget/git/releases/tag/pr-git-1885%2Fpszlazak%2Fdocument-http-uploadarchive-v2
Fetch-It-Via: git fetch https://github.com/gitgitgadget/git pr-git-1885/pszlazak/document-http-uploadarchive-v2
Pull-Request: https://github.com/git/git/pull/1885

Range-diff vs v1:

 1:  4a5465d0329 ! 1:  ace5e71729c doc: documentation for http.uploadarchive config option
     @@ Commit message
          In Git v2.44.0 support for 'git archive' over HTTP protocol
          was added, but it was nowhere documented how it should be
          enabled in git-http-backend.
     -    This commit adds needed information.
     +
     +    Add missing documentation.
      
          Signed-off-by: Piotr Szlazak <piotr.szlazak@gmail.com>
      


 Documentation/git-http-backend.txt | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/Documentation/git-http-backend.txt b/Documentation/git-http-backend.txt
index f37ddaded82..1dea4268520 100644
--- a/Documentation/git-http-backend.txt
+++ b/Documentation/git-http-backend.txt
@@ -56,6 +56,10 @@ http.receivepack::
 	disabled by setting this item to `false`, or enabled for all
 	users, including anonymous users, by setting it to `true`.
 
+http.uploadarchive::
+	This serves 'git archive' clients for remote archive over HTTP/HTTPS
+	protocols. It is disabled by default. It only works in protocol v2.
+
 URL TRANSLATION
 ---------------
 To determine the location of the repository on disk, 'git http-backend'

base-commit: bc204b742735ae06f65bb20291c95985c9633b7f
-- 
gitgitgadget

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH] doc: documentation for http.uploadarchive config option
  2025-02-06 14:04 ` Junio C Hamano
@ 2025-02-06 17:54   ` Piotr Szlazak
  0 siblings, 0 replies; 4+ messages in thread
From: Piotr Szlazak @ 2025-02-06 17:54 UTC (permalink / raw)
  To: Junio C Hamano, Piotr Szlazak via GitGitGadget; +Cc: git, Jiang Xin, Jeff King

On 06.02.2025 15:04, Junio C Hamano wrote:
> "Piotr Szlazak via GitGitGadget" <gitgitgadget@gmail.com> writes:
>
>> From: Piotr Szlazak <piotr.szlazak@gmail.com>
>>
>> In Git v2.44.0 support for 'git archive' over HTTP protocol
>> was added, but it was nowhere documented how it should be
>> enabled in git-http-backend.
> Good eyes
>
>> This commit adds needed information.
> It is more customary to say something like:
>
>      Add missing documentation.
>
> around here.

Commit message corrected in [PATCH v2].

>
>> diff --git a/Documentation/git-http-backend.txt b/Documentation/git-http-backend.txt
>> index f37ddaded82..1dea4268520 100644
>> --- a/Documentation/git-http-backend.txt
>> +++ b/Documentation/git-http-backend.txt
>> @@ -56,6 +56,10 @@ http.receivepack::
>>   	disabled by setting this item to `false`, or enabled for all
>>   	users, including anonymous users, by setting it to `true`.
>>   
>> +http.uploadarchive::
>> +	This serves 'git archive' clients for remote archive over HTTP/HTTPS
>> +	protocols. It is disabled by default. It only works in protocol v2.
> The description looks good, from the correctness point-of-view,
> and its style is in line with other entries before it.
>
> We may want to later consider moving these four http.* configuration
> variables somehow to Documentation/config/. directory so that it is
> shared across git-http-backend.1 and git-config.1 pages, but that is
> something we should leave outside the current effort.
>
> Thanks.

Regards!


^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2025-02-06 17:54 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-02-05 21:20 [PATCH] doc: documentation for http.uploadarchive config option Piotr Szlazak via GitGitGadget
2025-02-06 14:04 ` Junio C Hamano
2025-02-06 17:54   ` Piotr Szlazak
2025-02-06 17:45 ` [PATCH v2] " Piotr Szlazak via GitGitGadget

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.