Re: [PATCH] Docs: web server must setenv GIT_PROTOCOL for v2

Philippe Blain [off-list ref] writes:

I agree it's nice to make it work out of the box, without the web server
admin having to configure anything. But, I'm not sure we should completely
drop the documentation patches: your patch will only affect future versions
of git-http-backend, and users of previous versions will be left without
any documentation as to how to configure it for protocol v2. So I would think we should
keep the documentation patches, maybe with a mention "this should not be necessary
in Git 2.34 and later versions" or something like that (since your
commit message mentions that it "generally" should work like that depending
on the web servers).

Thanks, exactly my thought on the need for docs.

On Thu, Sep 09, 2021 at 10:39:29PM -0700, Junio C Hamano wrote:

Philippe Blain [off-list ref] writes:

quoted

I agree it's nice to make it work out of the box, without the web server
admin having to configure anything. But, I'm not sure we should completely
drop the documentation patches: your patch will only affect future versions
of git-http-backend, and users of previous versions will be left without
any documentation as to how to configure it for protocol v2. So I would think we should
keep the documentation patches, maybe with a mention "this should not be necessary
in Git 2.34 and later versions" or something like that (since your
commit message mentions that it "generally" should work like that depending
on the web servers).

Thanks, exactly my thought on the need for docs.

OK. I'll try to cook something up here.

-Peff