Felipe Contreras [off-list ref] writes:
There are no tests for 'envelope-sender', so I don't think it should
be a requirement for this patch to do so....
The fact that the lack of test was pointed out as a problem makes it a
requirement. Others' earlier mistakes are not an excuse for you to do a
poor job.
I do use --envelope-sender when sending patches out via msmtp. What
wonderful things this patch adds am I missing in my current setup?
On Sun, Nov 22, 2009 at 4:58 AM, Junio C Hamano [off-list ref] wrote:
Felipe Contreras [off-list ref] writes:
quoted
There are no tests for 'envelope-sender', so I don't think it should
be a requirement for this patch to do so....
The fact that the lack of test was pointed out as a problem makes it a
requirement. Others' earlier mistakes are not an excuse for you to do a
poor job.
Maybe I should wait until somebody adds the tests to --envelope-sender
before pushing this patch.
I do use --envelope-sender when sending patches out via msmtp. What
wonderful things this patch adds am I missing in my current setup?
You need to specify your mail address when setting the variable,
right? I use multiple email addresses, so that when I change it
($EMAIL or user.email) the right one will be used in the envelope. If
I use 'sendemail.envelopesender' I would
need to change it as well.
Another option is to do something like 'sendemail.envelopesender=auto'.
--
Felipe Contreras