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

Re: [PATCH 3/5] git-instaweb: store parameters in config

From
Johannes Schindelin <johannes.schindelin@gmx.de>
Date
Jul 26, 2006, 21:13 UTC
Message-ID
<Pine.LNX.4.63.0607262311560.29667@wbgn013.biozentrum.uni-wuerzburg.de>
In-Reply-To
<Pine.LNX.4.63.0607262147280.29667@wbgn013.biozentrum.uni-wuerzburg.de>
Hi,
On Wed, 26 Jul 2006, Johannes Schindelin wrote:
Show 12 quoted lines
> On Wed, 26 Jul 2006, Junio C Hamano wrote:
> 
> > Johannes Schindelin <Johannes.Schindelin@gmx.de> writes:
> > 
> > > It already reads the config for defaults, and now it also stores
> > > the last passed parameters in the config.
> > 
> > This might need to be conditional -- usually people override the
> > default from the command line as a one-shot thing, and storing
> > them in the configuration file would upset them.
> 
> I agree that this should be conditional, but the other way round, no?

Being bitten by my own words. It is plain dumb to store parameters for which instaweb fails. And my patch did that. Oh, well.

So, I'd say: just forget about patches [3-5]/5.

Ciao, Dscho

Previous: Johannes Schindelin
Message 4 of 4 in “git-instaweb: store parameters in config”
  1. 3/5 git-instaweb: store parameters in configJohannes Schindelin, Jul 26, 2006
  2. Junio C HamanoJul 26, 2006
  3. Johannes SchindelinJul 26, 2006
  4. Johannes SchindelinJul 26, 2006

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.