git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: [PATCH 0/2] gitweb: Improve handling of configuration files

From
Junio C Hamano <gitster@pobox.com>
Date
May 25, 2011, 19:34 UTC
Message-ID
<7vr57m377i.fsf@alter.siamese.dyndns.org>
In-Reply-To
<4DDD4CC1.3070103@kernel.org>
"J.H." <warthog9@kernel.org> writes:
> I would argue against this, for a couple of reasons:
>
> 1) Git itself treats /etc/git.conf in this exact manor,...
> Case in point turning on things like updateserverinfo, or automatic packing.

If we read both from the beginning of time it would have been more convenient, and I agree with that 100%. So this is not a valid reason to "argue against this".

> 2) Trying to get people to do includes in their config files, or to
> completely ignore the system wide config isn't a sustainable practice.
I also agree with this.
In any case, I do not think we have to choose between Jakub's two patches.
Previous: J.H.
Message 9 of 9 in “gitweb: Improve handling of configuration files”
  1. 0/2 gitweb: Improve handling of configuration filesJakub Narebski, May 25, 2011
  2. 1/2 gitweb: Refactor reading and parsing config file into read_config_fileJakub Narebski, May 25, 2011
  3. Junio C HamanoMay 25, 2011
  4. 2/2 gitweb: Mention read_config_file in gitweb/READMEJakub Narebski, May 25, 2011
  5. 2/2 gitweb: Use /etc/gitweb.conf even if gitweb_conf.perl existJakub Narebski, May 25, 2011
  6. Junio C HamanoMay 25, 2011
  7. 2/2 gitweb: Introduce common system-wide settings for convenienceJakub Narebski, May 26, 2011
  8. J.H.May 25, 2011
  9. Junio C HamanoMay 25, 2011

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.