On 10.03.2017 21:23, Fabian Maurer wrote:
Yes, you should send your patches in smaller chunks. Also, you shouldn't rely on the assumption that all of your patches are applied at once. The patchset should still make sense, even if each commit is applied separately. If it doesn't, that would mean you have to reorder or merge them. As long as you mark failing tests with todo_wine it is not a problem to have a temporary state with "less features" than the original implementation though.
Sorry, I still don't understand what's wrong. The patches depend on each other, so just applying patch 9 would result in an error. But only applying 1-5 should pose no problem. Except for missing functionality of course, but the tests should be fine. But I have to send these 12 patches as single patchset, since it's supposed to replace the old taskdialog-hack and I don't want to introduce a regression. Only with all those patches the functionality is the same.
Of course they should be applied in the correct order, but introducing a temporary regression because of unimplemented functionality is not a problem, and basically expected when you are working on such a task. ;)
Again, the usual and recommended process for upstreaming stuff is to focus on a set of about 5-7 patches, improve them until they fulfill all the requirements and have been accepted, and then proceed with next ones. Following these rules will make things easier for you (no need to resend all 12 patches when you missed some testbot errors, for example), and also for reviewers. As a reviewer you cannot be sure that a patch which previously was fine hasn't been changed, so you have to start from scratch and review the same patch over and over again - noone wants to do that. ;) If it is still not clear what I mean, maybe someone else can try and explain it a bit better.
Best regards, Sebastian