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
Junio C Hamano <gitster@pobox.com>
Date
Dec 9, 2007, 22:54 UTC
Message-ID
<7vhciro4vx.fsf@gitster.siamese.dyndns.org>
In-Reply-To
<475C5869.4080900@op5.se>
Andreas Ericsson <ae@op5.se> writes:
Show 7 quoted lines
> 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.

Note that I said my "initial" impression. I reserve the right to change my mind, as always ;-)

Previous: Andreas EricssonNext: Mike Hommey
Message 5 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.