On 04/13/2010 12:46 AM, Greg Geldorp wrote:
From: Ge van Geldorp
From: Roderick Colenbrander
I'm not sure how the bot works but it looks like it concatenated some of my patches together and then applied to them. I have no idea to what Wine version it was since it applies fine on Wine git here. It could be that I'm doing something wrong but I don't see where.
No, this is the bot messing up. For some reason I don't understand yet it included one of your earlier patches ("[PATCH 08/13] d3d9: Add an initial ColorFill regression test.") as the first of this series. That threw everything off.
I'll disable emailing to wine-devel until I've sorted this out.
Ok, I understand what happened now... This is the sequence of patches you submitted, in the order they arrived in my mail queue (oldest first):
- [PATCH 4/7] wined3d: Move D15S1 over to the formats table.
- [PATCH 1/7] wined3d: Move L6V5U5 conversion to the formats table.
- [PATCH 5/7] wined3d: Move R32G32F convertion to the formats table.
- [PATCH 3/7] wined3d: Move D24X4S4 to the formats table.
- [PATCH 6/7] wined3d: Move G16R16/R16G16F conversion to the formats table.
- 1/3 d3d9: Add an initial ColorFill regression test.
- [PATCH 2/7] wined3d: Move D24FS8 to formats table.
- [PATCH 7/7] wined3d: Move A4L4 conversion to the formats table.
Note the out-of-sequence arrival of number 6). The bot is not smart enough to handle two series from the same author arriving interleaved. It will see everything as one big series until all parts of a series have arrived. In this case, the patch attached to message 6) overwrote the patch attached to message 2) (both have part number 1).
Not sure how to handle this, except by asking people to leave some time before sending another series.
Dan's Patchwatcher had more or less the same issue in the beginning, but wasn't that solved?
I think the only issue left was when people sent 2 series of equal patchcounts and they arrive mixed.