Re: [PATCH] send-email: don't cc *-by lines with '-' prefix

2019-04-04 Thread Baruch Siach
Hi Junio, On Thu, Apr 04 2019, Junio C. Hamano wrote: > Baruch Siach writes: > >>> Independently, I think it makes sense to do something like >>> >>> /^([a-z][a-z-]*-by|Cc): (.*)/i >>> >>> to tighten the match to exclude a non-trailer; t

Re: [PATCH] send-email: don't cc *-by lines with '-' prefix

2019-04-04 Thread Baruch Siach
Hi Junio, On Mon, Mar 18 2019, Junio C. Hamano wrote: > Joe Perches writes: > >> My preference would be for correctness. >> I presume something like this isn't too onerous. > > I am guessing that /^---/ is to stop at the three-dash line *OR* > after the initial handful of lines of the first diff

Re: [PATCH] send-email: don't cc *-by lines with '-' prefix

2019-03-16 Thread Baruch Siach
Hi Joe, On Sat, Mar 16 2019, Joe Perches wrote: > On Sat, 2019-03-16 at 21:49 +0200, Baruch Siach wrote: >> On Sat, Mar 16 2019, Joe Perches wrote: >> > On Sat, 2019-03-16 at 21:26 +0200, Baruch Siach wrote: >> > > Since commit ef0cc1df90f6b ("send-email: also p

Re: [PATCH] send-email: don't cc *-by lines with '-' prefix

2019-03-16 Thread Baruch Siach
Hi Joe, On Sat, Mar 16 2019, Joe Perches wrote: > On Sat, 2019-03-16 at 21:26 +0200, Baruch Siach wrote: >> Since commit ef0cc1df90f6b ("send-email: also pick up cc addresses from >> -by trailers") in git version 2.20, git send-email adds to cc list >> addresses

[PATCH] send-email: don't cc *-by lines with '-' prefix

2019-03-16 Thread Baruch Siach
es in cc. Cc: Joe Perches Cc: Rasmus Villemoes Signed-off-by: Baruch Siach --- git-send-email.perl | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/git-send-email.perl b/git-send-email.perl index 8eb63b5a2f8d..5656ba83d9b1 100755 --- a/git-send-email.perl +++ b/git-send-email.