Possibility of adding a new patch status

Hugh McMaster hugh.mcmaster at masterindexing.com
Wed Jul 31 03:30:07 CDT 2013


On Tuesday, 30 July 2013 9:04 PM, Ken Sharp wrote:

>There's also "Pending".

Hi Ken,

Yes, I know about the 'Pending' status.  The patches page describes this status in two ways:

1. The patch is not obviously correct at first glance. Making a more convincing argument, preferably in the form of a test case, may help.
2. Waiting for feedback from the main developer in that area.

Both are quite different. Perhaps these should be split with different labels as well - particularly as the first requires revisions, while the second implies that the patch may be accepted as is.

Anyway, to come back to the point, leaving a patch as 'New' is problematic.  'Pending' is more informative, yet perhaps not descriptive enough to cover patches that stay as 'New' over a 24-hour patch cycle.



More information about the wine-devel mailing list