threads / discuss / 6430

git-send-email --suppress-from option doesn't work.

Subject: git-send-email --suppress-from option doesn't work.

## tl;dr

3 messages between Jan 18, 2007 and Jan 25, 2007.

replies: 2people: 2as markdown or json

Timur Tabi· Jan 18, 2007, 21:07 UTC · lore
The --suppress-from option doesn't work for me because the comparison is too strict.
Here's an example usage of git-send-email:

git-send-email --suppress-from --from timur@freescale.com --to timur@tabi.org --smtp-server remotesmtp.freescale.net 0001-Add-support-for-the-MPC8349E-mITX-GP.txt

I need to specify the --from option, because otherwise git-send-email will prompt me for a From address, and I don't want it to prompt me for anything.

The problem is that the patchfile contains this line:
From: Timur Tabi <timur@freescale.com>

That means that the patchfile contains the string "Timur Tabi <timur@freescale.com>", but the --from specifies the string "timur@freescale.com". The code which checks the suppress-from option is here:

                         } elsif (/^(Cc|From):\s+(.*)$/) {
                                 if ($2 eq $from) {
                                         next if ($suppress_from);
                                 }

I don't know Perl, but I'm guess the The "$2 eq $from" is a strict comparison that fails in my case.

Some of you might say at this point, "Why don't you just specify --from "Timur Tabi <timur@freescale.com>"? I tried that, and it still doesn't work.

Basically, what I want is to prevent git-send-email from CC'ing me on my own patches. No matter what I do, I can't prevent it from adding "Cc: Timur Tabi <timur@freescale.com>" to the email header.

-- 
Timur Tabi
Linux Kernel Developer @ Freescale
Andreas Ericsson· Jan 19, 2007, 01:24 UTC · re: Timur Tabi · lore

Re: git-send-email --suppress-from option doesn't work.

Timur Tabi wrote:
Show 23 quoted lines
> The --suppress-from option doesn't work for me because the comparison is
> too strict.
> 
> Here's an example usage of git-send-email:
> 
> git-send-email --suppress-from --from timur@freescale.com --to
> timur@tabi.org --smtp-server remotesmtp.freescale.net
> 0001-Add-support-for-the-MPC8349E-mITX-GP.txt
> 
> I need to specify the --from option, because otherwise git-send-email
> will prompt me for a From address, and I don't want it to prompt me for
> anything.
> 
> The problem is that the patchfile contains this line:
> 
> From: Timur Tabi <timur@freescale.com>
> 
> That means that the patchfile contains the string "Timur Tabi
> <timur@freescale.com>", but the --from specifies the string
> "timur@freescale.com".  The code which checks the suppress-from option
> is here:
> 
>                         } elsif (/^(Cc|From):\s+(.*)$/) {
I believe
	(/^(Cc|From):[^<]+<([^>])+>.*$/)

would do the trick for your case. It would however fail when specifying a proper --from address in the 'git commit --author="Foo Barson <foo@barson.com>"' style.

Are you sure you need to specify --from for those patches though?
Show 11 quoted lines
>                                 if ($2 eq $from) {
>                                         next if ($suppress_from);
>                                 }
> 
> I don't know Perl, but I'm guess the The "$2 eq $from" is a strict
> comparison that fails in my case.
> 
> Some of you might say at this point, "Why don't you  just specify --from
> "Timur Tabi <timur@freescale.com>"?  I tried that, and it still doesn't
> work.
> 

That sounds extremely odd indeed. Could this have to do with character conversion?

-- 
Andreas Ericsson                   andreas.ericsson@op5.se
OP5 AB                             www.op5.se
Tel: +46 8-230225                  Fax: +46 8-230231
Timur Tabi· Jan 25, 2007, 16:47 UTC · re: Andreas Ericsson · lore

Re: git-send-email --suppress-from option doesn't work.

Andreas Ericsson wrote:
Show 7 quoted lines
> 
> I believe
> 
> 	(/^(Cc|From):[^<]+<([^>])+>.*$/)
> 
> would do the trick for your case. It would however fail when specifying a proper
> --from address in the 'git commit --author="Foo Barson <foo@barson.com>"' style.

Even so, it would still be very picky about the layout. --suppress-from should just compare the actual email addresses, not the names or any other characters.

> Are you sure you need to specify --from for those patches though?
Yes.  If I don't specify --from, then git-send-email will prompt me for the 
From: address.  I want git-send-email to be completely non-interactive.
Show 6 quoted lines
>> Some of you might say at this point, "Why don't you  just specify --from
>> "Timur Tabi <timur@freescale.com>"?  I tried that, and it still doesn't
>> work.
> 
> That sounds extremely odd indeed. Could this have to do with character
> conversion?
Actually, I figured out the problem is that I can't do this:

FROM='--from "Foo Barson <foo@barson.com>"' git-send-email $FROM ...

I got all sorts of weird messages about unbalanced > or something. Instead, I need to do this:

FROM="Foo Barson <foo@barson.com>" git-send-email --from $FROM ...

This is probably a shell issue instead of a git-send-email issue, but it is annoying.

-- 
Timur Tabi
Linux Kernel Developer @ Freescale

← back to recent threads