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

Re: [PATCH 2/2] Add support for URLs to git-apply

From
Mike Hommey <mh@glandium.org>
Date
Dec 10, 2007, 06:46 UTC
Message-ID
<20071210064659.GA4148@glandium.org>
In-Reply-To
<7vhciro4vx.fsf@gitster.siamese.dyndns.org>
On Sun, Dec 09, 2007 at 02:54:58PM -0800, Junio C Hamano wrote:
Show 14 quoted lines
> Andreas Ericsson <ae@op5.se> writes:
> 
> > Mike Hommey wrote:
> >> Instead of doing several "wget -O - url | git-apply -" in a raw, you now
> >> can just git-apply url1 url2 ...
> >>
> >
> > I seriously like this idea. Combined with gitweb (or cgit), it could be
> > used as a cherry-pick from someone else's repo :)
> 
> FWIW, my initial impression is that I seriously dislike this.  It may be
> good if the patch were to git-am, but when git-apply rejects an
> inapplicable patch, there won't be nothing left for you to recover with
> and you need to re-download the patch anyway.

There are some usecase differences between git-apply and git-am. Probably, this change would be good to have on both.

Mike
Previous: Junio C HamanoNext: Johannes Schindelin
Message 6 of 7 in “Move the file read logic to read_patch_file() in builtin-apply.c”
  1. 1/2 Move the file read logic to read_patch_file() in builtin-apply.cMike Hommey, Dec 9, 2007
  2. 2/2 Add support for URLs to git-applyMike Hommey, Dec 9, 2007
  3. Mike HommeyDec 9, 2007
  4. Andreas EricssonDec 9, 2007
  5. Junio C HamanoDec 9, 2007
  6. Mike HommeyDec 10, 2007
  7. Johannes SchindelinDec 10, 2007

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.