[PATCH 01/10] cmd: Use CSTR_* instead of hardcoded values as result of CompareStringW
Dan Kegel
dank at kegel.com
Sat Aug 20 10:54:55 CDT 2011
2011/8/20 Frédéric Delanoy <frederic.delanoy at gmail.com>:
>>> I thought the Date: field was set by the client, so server
>>> races shouldn't matter.
>> Sorry, missed that you use that and not the order you receive it.
>
> That's not always reliable: say you commit locally patches [1-2/3] on
> day D and patch [3/3] on day D+1
> P3 - baz - D+1
> P2 - bar - D
> P1 - foo - D
>
> For whatever reason, before submission, you decide patch [3/3] should
> be the first one, so you use "git rebase" do to that:
> P3 - bar - D
> P2 - foo - D
> P1 - baz - D+1
>
> and you git-send the mails.
git-send's messages' timestamps use current time, not commit time:
http://git.kernel.org/?p=git/git.git;a=blob;f=git-send-email.perl#l98
And it is careful to increment the time after each message!
So I think my usage is compatible with git-send.
> So, you should always use the numbering specified by the author IMHO
I wish it were so easy. It is very difficult to recognize patch series
without relying on them being sent in order.
I've tried:
take all unprocessed messages
divide them by sender
further divide them into groups based on length of patch series in
subject line
sort each group by patch number from subject line
but that breaks down if the developer sends two patch series with
the same length, which happens very often, e.g. when somebody
retries a patch series. Maybe one could further subdivide messages
by retry number, but that sounds hard and fragile.
- Dan
More information about the wine-devel
mailing list