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

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

[Mike Hommey]

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.

 git-send-email.perl |    6 ++++--
 1 files changed, 4 insertions(+), 2 deletions(-)

diff --git a/git-send-email.perl b/git-send-email.perl
index c0e1dd3..0e8c457 100755
--- a/git-send-email.perl
+++ b/git-send-email.perl
@@ -369,8 +369,10 @@ if ($thread && !defined $initial_reply_to && $prompting) {
 	$initial_reply_to = $_;
 }
 
-$initial_reply_to =~ s/^\s*<?/</;
-$initial_reply_to =~ s/>?\s*$/>/;
+if ($initial_reply_to) {
+	$initial_reply_to =~ s/^\s*<?/</;
+	$initial_reply_to =~ s/>?\s*$/>/;
+}
 
 if (!defined $smtp_server) {
 	foreach (qw( /usr/sbin/sendmail /usr/lib/sendmail )) {
-- 
1.5.3.7.1161.g14b8

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

[Junio C Hamano]

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.

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

[Mike Hommey]

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.

Mike

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

[Junio C Hamano]

Mike Hommey <mh@glandium.org> writes:

> 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?

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

[Junio C Hamano]

Mike Hommey <mh@glandium.org> writes:

>> 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.

Ah, I see what you mean.  Thanks.