Re: [PATCH 2/2] send-email: supply a --send-delay=1 by default

2018-03-27 Thread Eric Wong
Ævar Arnfjörð Bjarmason wrote: > Good point. I also see that (via git log --author=Ævar --grep='^\[PATCH > ') that this series itself arrived out of order (0 -> 2 -> 1), but I > don't know to what extent public-inbox itself might be batching things. public-inbox doesn't batch,

Re: [PATCH 2/2] send-email: supply a --send-delay=1 by default

2018-03-27 Thread Eric Wong
Ævar Arnfjörð Bjarmason wrote: > Good point. I also see that (via git log --author=Ævar --grep='^\[PATCH > ') that this series itself arrived out of order (0 -> 2 -> 1), but I > don't know to what extent public-inbox itself might be batching things. public-inbox doesn't batch, aside from when

Re: [PATCH 2/2] send-email: supply a --send-delay=1 by default

2018-03-25 Thread Junio C Hamano
Eric Sunshine writes: > A minor point: Are you sure that it's git-format-patch that's being > careful about arranging Date: to display in the desired order, and not > git-send-email? Looking at old patches I still have hanging around > which were created with

Re: [PATCH 2/2] send-email: supply a --send-delay=1 by default

2018-03-25 Thread Junio C Hamano
Eric Sunshine writes: > A minor point: Are you sure that it's git-format-patch that's being > careful about arranging Date: to display in the desired order, and not > git-send-email? Looking at old patches I still have hanging around > which were created with git-format-patch, I see the Date:

Re: [PATCH 2/2] send-email: supply a --send-delay=1 by default

2018-03-25 Thread Junio C Hamano
"brian m. carlson" writes: > I'm not sure that this is going to have the effect you want it to have. > Let me give an example to demonstrate why. > ... > In short, I don't think this is going to be especially helpful because > it won't change the status quo for a

Re: [PATCH 2/2] send-email: supply a --send-delay=1 by default

2018-03-25 Thread Junio C Hamano
"brian m. carlson" writes: > I'm not sure that this is going to have the effect you want it to have. > Let me give an example to demonstrate why. > ... > In short, I don't think this is going to be especially helpful because > it won't change the status quo for a lot of senders. You'd have to >

Re: [PATCH 2/2] send-email: supply a --send-delay=1 by default

2018-03-25 Thread Eric Sunshine
On Sun, Mar 25, 2018 at 2:28 PM, Ævar Arnfjörð Bjarmason wrote: > The earlier change to add this option described the problem this > option is trying to solve. > > This turns it on by default with a value of 1 second, which'll > hopefully solve it, and if not user reports as

Re: [PATCH 2/2] send-email: supply a --send-delay=1 by default

2018-03-25 Thread Eric Sunshine
On Sun, Mar 25, 2018 at 2:28 PM, Ævar Arnfjörð Bjarmason wrote: > The earlier change to add this option described the problem this > option is trying to solve. > > This turns it on by default with a value of 1 second, which'll > hopefully solve it, and if not user reports as well as the >

Re: [PATCH 2/2] send-email: supply a --send-delay=1 by default

2018-03-25 Thread Ævar Arnfjörð Bjarmason
On Sun, Mar 25 2018, brian m. carlson wrote: > On Sun, Mar 25, 2018 at 06:28:03PM +, Ævar Arnfjörð Bjarmason wrote: >> The earlier change to add this option described the problem this >> option is trying to solve. >> >> This turns it on by default with a value of 1 second, which'll >>

Re: [PATCH 2/2] send-email: supply a --send-delay=1 by default

2018-03-25 Thread Ævar Arnfjörð Bjarmason
On Sun, Mar 25 2018, brian m. carlson wrote: > On Sun, Mar 25, 2018 at 06:28:03PM +, Ævar Arnfjörð Bjarmason wrote: >> The earlier change to add this option described the problem this >> option is trying to solve. >> >> This turns it on by default with a value of 1 second, which'll >>

Re: [PATCH 2/2] send-email: supply a --send-delay=1 by default

2018-03-25 Thread brian m. carlson
On Sun, Mar 25, 2018 at 06:28:03PM +, Ævar Arnfjörð Bjarmason wrote: > The earlier change to add this option described the problem this > option is trying to solve. > > This turns it on by default with a value of 1 second, which'll > hopefully solve it, and if not user reports as well as the

Re: [PATCH 2/2] send-email: supply a --send-delay=1 by default

2018-03-25 Thread brian m. carlson
On Sun, Mar 25, 2018 at 06:28:03PM +, Ævar Arnfjörð Bjarmason wrote: > The earlier change to add this option described the problem this > option is trying to solve. > > This turns it on by default with a value of 1 second, which'll > hopefully solve it, and if not user reports as well as the

[PATCH 2/2] send-email: supply a --send-delay=1 by default

2018-03-25 Thread Ævar Arnfjörð Bjarmason
The earlier change to add this option described the problem this option is trying to solve. This turns it on by default with a value of 1 second, which'll hopefully solve it, and if not user reports as well as the X-Mailer-Send-Delay header should help debug it. I think the trade-off of slowing

[PATCH 2/2] send-email: supply a --send-delay=1 by default

2018-03-25 Thread Ævar Arnfjörð Bjarmason
The earlier change to add this option described the problem this option is trying to solve. This turns it on by default with a value of 1 second, which'll hopefully solve it, and if not user reports as well as the X-Mailer-Send-Delay header should help debug it. I think the trade-off of slowing