Re:Re: [PATCH v5] send-email: --batch-size to work around some SMTP server limit

2017-05-23 Thread 赵小强
At 2017-05-22 17:26:41, "Ævar Arnfjörð Bjarmason" wrote: >On Sun, May 21, 2017 at 2:59 PM, xiaoqiang zhao wrote: >> Some email servers (e.g. smtp.163.com) limit the number emails to be >> sent per session(connection) and this will lead to a faliure when >> sending many messages. > >This OK to m

Re: [PATCH v5] send-email: --batch-size to work around some SMTP server limit

2017-05-23 Thread Jan Viktorin
On Tue, 23 May 2017 16:46:27 +0900 Junio C Hamano wrote: > Ævar Arnfjörð Bjarmason writes: > > > Looking at this the Nth time now though I wonder about this approach > > in general. In all your E-Mails I don't think you ever said /what/ > > sort of error you had from the SMTP server, you just s

Re: [PATCH v5] send-email: --batch-size to work around some SMTP server limit

2017-05-23 Thread Junio C Hamano
Ævar Arnfjörð Bjarmason writes: > Looking at this the Nth time now though I wonder about this approach > in general. In all your E-Mails I don't think you ever said /what/ > sort of error you had from the SMTP server, you just said you had a > failure or an error, I assume you hit one of the die'

Re: [PATCH v5] send-email: --batch-size to work around some SMTP server limit

2017-05-22 Thread Ævar Arnfjörð Bjarmason
On Sun, May 21, 2017 at 2:59 PM, xiaoqiang zhao wrote: > Some email servers (e.g. smtp.163.com) limit the number emails to be > sent per session(connection) and this will lead to a faliure when > sending many messages. This OK to me, the nits I had are addressed by Junio's reply. Looking at this

Re: [PATCH v5] send-email: --batch-size to work around some SMTP server limit

2017-05-21 Thread Zhaoxiangqiang
Junio C Hamano 于 2017 年 5 月 22 日 星期一 写道: > I think this is almost perfect. > > I'd propose squashing the patch below to > > - Add cross reference between config and option > > - Spell configuration variables in camelCase to mimic other >sendemail.* variables > > - Spell SMTP in all cap

Re: [PATCH v5] send-email: --batch-size to work around some SMTP server limit

2017-05-21 Thread Junio C Hamano
I think this is almost perfect. I'd propose squashing the patch below to - Add cross reference between config and option - Spell configuration variables in camelCase to mimic other sendemail.* variables - Spell SMTP in all caps to mimic other parts of the manual - Suggest use of credent

[PATCH v5] send-email: --batch-size to work around some SMTP server limit

2017-05-21 Thread xiaoqiang zhao
Some email servers (e.g. smtp.163.com) limit the number emails to be sent per session(connection) and this will lead to a faliure when sending many messages. Teach send-email to disconnect after sending a number of messages (configurable via the --batch-size= option), wait for a few seconds (confi