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

Re: [PATCH] http-push: refactor request url creation

From
Johannes Schindelin <johannes.schindelin@gmx.de>
Date
Jan 28, 2009, 23:06 UTC
Message-ID
<alpine.DEB.1.00.0901290005200.3586@pacific.mpi-cbg.de>
In-Reply-To
<7vvdrzjb80.fsf@gitster.siamese.dyndns.org>
Hi,
On Wed, 28 Jan 2009, Junio C Hamano wrote:
> I thought at least you did not have these overlong lines in your earlier 
> attempts, and Dscho may have acked one of those, but I doubt he would 
> give his Ack to this one.  I certainly wouldn't Ack it myself.

My understanding is that the ACK is sent to the mailing list by the Acker and in response to the particular patch she is Acking.

> The refactoring looked sane otherwise, but I think we would want to opt 
> for safety by keeping an extra string around.
Yep.  Better safe 'n sorry.

Ciao, Dscho

Previous: Junio C HamanoNext: Tay Ray Chuan
Message 3 of 4 in “http-push: refactor request url creation”
  1. http-push: refactor request url creationTay Ray Chuan, Jan 28, 2009
  2. Junio C HamanoJan 28, 2009
  3. Johannes SchindelinJan 28, 2009
  4. Tay Ray ChuanJan 29, 2009

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.