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

Re: [PATCH] gitweb: make remote_heads config setting work.

From
Jeff King <peff@peff.net>
Date
Nov 8, 2012, 16:56 UTC
Message-ID
<20121108165648.GG15560@sigill.intra.peff.net>
In-Reply-To
<20121105235047.GA78156@redoubt.spodhuis.org>
On Mon, Nov 05, 2012 at 06:50:47PM -0500, Phil Pennock wrote:
Show 8 quoted lines
> Git configuration items can not contain underscores in their name; the
> 'remote_heads' feature can not be enabled on a per-repository basis with
> that name.
> 
> This changes the git-config option to be `gitweb.remoteheads` but does
> not change the gitweb.conf option, to avoid backwards compatibility
> issues.  We strip underscores from keys before looking through
> git-config output for them.

Makes sense. Thanks for considering the backwards compatibility angle. Hopefully we can avoid adding names with underscores in the future, but I think the mapping of "remote_head -> remotehead" is obvious enough that we do not need to worry about deprecating and replacing the old name.

-Peff
Previous: Phil PennockNext: Junio C Hamano
Message 2 of 6 in “gitweb: make remote_heads config setting work.”
  1. gitweb: make remote_heads config setting work.Phil Pennock, Nov 5, 2012
  2. Jeff KingNov 8, 2012
  3. Junio C HamanoNov 9, 2012
  4. Jeff KingNov 9, 2012
  5. Re* [PATCH] gitweb: make remote_heads config setting work.Junio C Hamano, Nov 20, 2012
  6. Jeff KingNov 21, 2012

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.