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

Re: [PATCH] Brown paper bag fix to previous send-email change

From
Junio C Hamano <gitster@pobox.com>
Date
Dec 11, 2007, 08:02 UTC
Message-ID
<7vsl29fym6.fsf@gitster.siamese.dyndns.org>
In-Reply-To
<20071211074427.GA12959@glandium.org>
Mike Hommey <mh@glandium.org> writes:
Show 21 quoted lines
> On Mon, Dec 10, 2007 at 11:04:36PM -0800, Junio C Hamano wrote:
>> Mike Hommey <mh@glandium.org> writes:
>> 
>> > My previous change led to the In-Reply-To header being <> when the given
>> > value was empty. This fixes it.
>> >
>> > Signed-off-by: Mike Hommey <mh@glandium.org>
>> > ---
>> >
>> >  I think I have an antispam problem somewhere with this patch, because even
>> >  keeping sending it, it never reached the list. And Junio didn't answer to my
>> >  private message about it, so I guess that was some server being smart with
>> >  the subject line. *sigh*
>> >
>> >  PS: sorry Junio if you get this patch multiple time.
>> 
>> No, I happened to have noticed the same and have an almost identical fix
>> (I said "defined $initial_reply_to") already queued in my tree.
>
> Testing "defined $initial_reply_to" might not be enough, because $initial_reply_to
> can be defined and empty.

And if empty what does your regexp do? Produce "<>"? How's that useful?

Previous: Mike HommeyNext: Junio C Hamano
Message 4 of 5 in “Brown paper bag fix to previous send-email change”
  1. Brown paper bag fix to previous send-email changeMike Hommey, Dec 11, 2007
  2. Junio C HamanoDec 11, 2007
  3. Mike HommeyDec 11, 2007
  4. Junio C HamanoDec 11, 2007
  5. Junio C HamanoDec 11, 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.