Julius Schwartzenberg wrote:
Paul Vriens wrote:
Do any of these patches fix any of the tests? If so you should also change the tests by removing one or more todo_wine()'s.
This could become tricky though if your tests patches are not committed before these ones.
Yes, all the tests work properly with my patches. I had understood that todo_wine should be based on the state of current GIT. But I understand now that normally a fix should remove an existing todo_wine in GIT. Right now I'm indeed not sure how to deal with this since I don't know whether either the tests and/or the fixes will be committed. I kinda hoped Alexandre had some kind of automated way to removed fixed todo_wines...
Nope. AJ does not have an automatic 'todo wine' removal tool. However, you can submit patches as a series, the first patch implements the function, the second removes the todo wine. If the first patch is rejected, then the second one is automatically removed. Again, if the second patch is not changed and you do change the first, you can 'resubmit' the second patch as is with the changed first patch.
James McKenzie