Re: [PATCH v2 2/2] send-email: Support separate Reply-To address

2018-01-18 Thread Martin Ågren
On 17 January 2018 at 19:08, Christian Ludwig wrote: > In some projects contributions from groups are only accepted from a > common group email address. But every individual may want to recieve > replies to her own personal address. That's what we have 'Reply-To' >

Re: [PATCH v2 2/2] send-email: Support separate Reply-To address

2018-01-17 Thread Ævar Arnfjörð Bjarmason
On Wed, Jan 17 2018, Christian Ludwig jotted: > +if (defined $reply_to) { > + $reply_to =~ s/^\s+|\s+$//g; > + ($reply_to) = expand_aliases($reply_to); > + $reply_to = sanitize_address($reply_to); > +} Just a note to other reviewers: I found it odd to call a function with one

Re: [PATCH v2 2/2] send-email: Support separate Reply-To address

2018-01-17 Thread Junio C Hamano
Christian Ludwig writes: > In some projects contributions from groups are only accepted from a > common group email address. But every individual may want to recieve > replies to her own personal address. That's what we have 'Reply-To' > headers for in SMTP. > >

[PATCH v2 2/2] send-email: Support separate Reply-To address

2018-01-17 Thread Christian Ludwig
In some projects contributions from groups are only accepted from a common group email address. But every individual may want to recieve replies to her own personal address. That's what we have 'Reply-To' headers for in SMTP. Introduce an optional '--reply-to' command line option. Unfortunately