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

Re: [PATCH 0/3] Reduced re-roll of http-proxy-more branch

From
Jeff King <peff@peff.net>
Date
May 11, 2012, 17:00 UTC
Message-ID
<20120511170013.GA26916@sigill.intra.peff.net>
In-Reply-To
<4FAD1075.8020309@seap.minhap.es>
On Fri, May 11, 2012 at 03:13:25PM +0200, Nelson Benitez Leon wrote:
Show 5 quoted lines
> Hi, based on last feedback by Junio and Jeff I've created this
> reduced version of http-proxy-more branch with all the bits
> about "reading proxy from environment" removed. When cURL adds
> the new api we will only need to expand the code introduced by
> commit 1/3 .

Thanks. I know it is frustrating to work on a series only to end up discarding a big chunk of it. But I think the result here makes a lot more sense in the long run.

Patches 2 and 3 look OK, but I have a few comments on the first one (I'll reply separately).

-Peff
Previous: Nelson Benitez Leon
Message 2 of 2 in “Reduced re-roll of http-proxy-more branch”
  1. 0/3 Reduced re-roll of http-proxy-more branchNelson Benitez Leon, May 11, 2012
  2. Jeff KingMay 11, 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.