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

Re: Applying patches from gmane can be dangerous.

From
Junio C Hamano <gitster@pobox.com>
Date
Feb 7, 2008, 08:21 UTC
Message-ID
<7vsl05p46n.fsf@gitster.siamese.dyndns.org>
In-Reply-To
<ve51w5yb.fsf@blue.sea.net>
Jari Aalto <jari.aalto@cante.net> writes:
Show 12 quoted lines
> FYI,
>
> Emacs Gnus + news.gmane.org gives access to raw articles with single
> command. Suppose cursor is at thread start "!"
>
> ! R. [  40: Junio C Hamano         ] Applying patches from gmane can be dangerous.
>   R.     [  19: Nicolas Pitre          ]
>
>
> Pressing "C-u g" will display the unmodified article as seen by mail
> transport. Running git's apply command can be automated pretty easily
> from there.

I do not think Gnus demiming (that C-u g helps us with) is not an issue.

Have you tried to look at the article in question? Inside your Gnus + news.gmane.org, try typing this:

	j 7 2 6 9 9 <Enter> C-u g
The last two keystrokes are your "C-u g".
And look at the second line in the buffer, that says:
    From: Brian Downing <bdowning-oU/tDdhfGLReoWH0uzbU5w@public.gmane.org>
and weep X-<.

Then scroll down to find Signed-off-by: lines that are similarly mangled, and weep more.

Maybe you run a much newer Gnus, and Lars taught "C-u g" to unmangle them. After all, gmane and Gnus are both his creations, so it _is_ possible. But somehow I doubt it.

If there weren't gmane address mangling, a quickest way to apply a gmane patch to commit is:

	C-u g | g i t  a m - 3 - s <Enter>
	For Gnus uninitiated, it reads: show as raw as opposed
	to demimed (C-u g), pipe the article to the shell
	command (|) that is "git am -3 -s".

But I usually work in batches, so I first C-o (write to file) the articles to a separate mbox, review and _edit_ them as needed before running "git am". And C-o does not suffer from Gnus demiming, so C-u g is not useful in my workflow.

Previous: Jari AaltoNext: Mike Hommey
Message 7 of 17 in “compat: Add simplified merge sort implementation from glibc”
  1. compat: Add simplified merge sort implementation from glibcBrian Downing, Feb 5, 2008
  2. Applying patches from gmane can be dangerous.Junio C Hamano, Feb 7, 2008
  3. Nicolas PitreFeb 7, 2008
  4. Junio C HamanoFeb 7, 2008
  5. 'next' will be rewound and rebuilt after feature releases.Junio C Hamano, Feb 10, 2008
  6. Jari AaltoFeb 7, 2008
  7. Junio C HamanoFeb 7, 2008
  8. Mike HommeyFeb 7, 2008
  9. Johannes SchindelinFeb 7, 2008
  10. Brian DowningFeb 7, 2008
  11. Aidan Van DykFeb 7, 2008
  12. Brian DowningFeb 7, 2008
  13. Johannes SchindelinFeb 7, 2008
  14. Johannes SchindelinFeb 11, 2008
  15. Frank LichtenheldFeb 7, 2008
  16. Johannes SchindelinFeb 5, 2008
  17. Brian DowningFeb 6, 2008

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.