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

Re: [PATCH] Set OLD_ICONV on Cygwin.

From
Andreas Ericsson <ae@op5.se>
Date
Dec 4, 2007, 13:43 UTC
Message-ID
<4755597B.9050709@op5.se>
In-Reply-To
<21977191.137171196775591804.JavaMail.www@wwinf1621>
Pascal Obry wrote:
Show 6 quoted lines
> Andreas Ericsson wrote:
>> Only one, insofar as I can see; You didn't CC Junio. He probably missed it.
> 
> Hum, I don't think this is a requirement. At least nothing is said about
> CCing Junio on the Git Web site.
> 

It's not a requirement, but he won't act on things he doesn't see, and CC'ing him directly is the recommended way of getting his attention. Most people do so for patches that aren't directly related to a sub-part of git primarily maintained by someone else (Shawn O. Pearce for git-gui, Eric Wong for git-svn, etc., etc.)

> Anyway, now Junio is CCed :)
> 

Nice. If my suspicions are correct and he missed your patch the first time around, you should probably resend it though.

-- 
Andreas Ericsson                   andreas.ericsson@op5.se
OP5 AB                             www.op5.se
Tel: +46 8-230225                  Fax: +46 8-230231
Previous: Pascal ObryNext: Junio C Hamano
Message 2 of 5 in “Re: [PATCH] Set OLD_ICONV on Cygwin.”
  1. Pascal ObryDec 4, 2007
  2. Andreas EricssonDec 4, 2007
  3. Junio C HamanoDec 4, 2007
  4. Junio C HamanoDec 4, 2007
  5. Pascal ObryDec 4, 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.