On 03/18, Jeff King wrote: > On Mon, Mar 18, 2019 at 11:52:54AM +0900, Junio C Hamano wrote: > > > Jeff King <peff@xxxxxxxx> writes: > > > > > Hmm. I guess it is still an issue in GGG. This thread has identical > > > timestamps on patches 1 and 2 (and my server received them out of order > > > by 2 seconds, so mutt orders them wrong): > > > > > > https://public-inbox.org/git/pull.163.git.gitgitgadget@xxxxxxxxx/ > > > > > > I do still think GGG has a more feasible path forward on this particular > > > bug, though. > > > > If the MSA is rewriting the timestamp (but why? Is the original > > date "Wed, 13 Mar 2019 19:20:12 GMT" malformed or perhaps in the > > future or something?), then there isn't much the sending program > > can---'git send-email' would suffer from the same symptom. > > I think this statement from me is mid-way through my discovery of the > actual issue. Yes, if the mail server is rewriting, the best we can do > is put in an artificial sleep. > > It looks like GitGitGadget just uses normal SMTP to submit the messages. > I wonder if normal people using gmail as their SMTP server for > send-email also suffer from this. I've not ever noticed it, but I > don't know how common that setup is. I am using gmail as my SMTP server with 'git send-email', and it doesn't look like gmail is rewriting anything there, see [*1*] for example. The date header looks like this: Date: Mon, 25 Feb 2019 23:16:04 +0000 Note the +0000 there, compared to the GMT that GitGitGadget uses. Looking at RFC2822, that's the new version of specifying the timezone, while GMT is only defined in the obsolete time and date section. I guess gmail might just not like that anymore and rewrite it. So fixing this might not be that hard, and might not involve sleeping while sending the patch series at all. Changing how the date is calculated in [*2*] might be all that's needed. *1*: https://public-inbox.org/git/20190225231631.30507-1-t.gummerer@xxxxxxxxx/raw *2*: https://github.com/gitgitgadget/gitgitgadget/blob/c37d58cc1581b479892a1f7d29bd16e261676c7d/lib/patch-series.ts#L427
