Module: tools Branch: master Commit: 045638407bea42e8cdc68b1b354d7ecdaf862390 URL: http://source.winehq.org/git/tools.git/?a=commit;h=045638407bea42e8cdc68b1b3...
Author: Alexandre Julliard julliard@winehq.org Date: Mon Apr 3 13:10:54 2017 +0200
patches: Add a patch status for replies to a patch.
Signed-off-by: Alexandre Julliard julliard@winehq.org
---
patches/expire | 3 +++ patches/patches.css | 1 + patches/update | 5 +++-- 3 files changed, 7 insertions(+), 2 deletions(-)
diff --git a/patches/expire b/patches/expire index fd5ba22..cba6246 100755 --- a/patches/expire +++ b/patches/expire @@ -35,6 +35,7 @@ my %status_expiry = "formatting" => 7, "other" => 7, "nopatch" => 3, + "reply" => 7, "rejected" => 7, "split" => 7, "depend" => 7, @@ -59,6 +60,7 @@ my %status_descr = "formatting" => "Formatting", "other" => "Other project", "nopatch" => "No patch", + "reply" => "Reply", "rejected" => "Rejected", "split" => "Needs splitting", "depend" => "Dependency", @@ -141,6 +143,7 @@ the patch is correct.", # No notifications are sent for the following: # "nil" # "signoff" +# "reply" # "superseded" # "testfail" ); diff --git a/patches/patches.css b/patches/patches.css index 2799827..c1ef34f 100644 --- a/patches/patches.css +++ b/patches/patches.css @@ -159,6 +159,7 @@ tr.odd { background-color: #f8e8e8; } .deferred, .deferred :link, .deferred :visited { color: #e08000; } .testcase, .testcase :link, .testcase :visited { color: #e08000; } .signoff, .signoff :link, .signoff :visited { color: #e08000; } +.reply, .reply :link, .reply :visited { color: #e08000; } .other, .other :link, .other :visited { color: #e08000; } .applyfail, .applyfail :link, .applyfail :visited { color: red; } .buildfail, .buildfail :link, .buildfail :visited { color: red; } diff --git a/patches/update b/patches/update index ad2b82a..cdc172d 100755 --- a/patches/update +++ b/patches/update @@ -43,6 +43,7 @@ my %status_descr = "testcase" => "Needs tests", "testfail" => "Test failure", "signoff" => "Sign-off", + "reply" => "Reply", );
my @legend = @@ -59,6 +60,7 @@ my @legend = [ "testcase", "<ul><li>You need to write some test cases demonstrating that the patch is correct.</li></ul>" ], [ "deferred", "<ul><li>The patch is deferred because of code freeze for an upcoming release. Either resend after the release, make the patch less intrusive, or make a convincing argument that it needs to be included in the release.</li></ul>" ], [ "signoff", "<ul><li>This is a Signed-off-by reply to another patch.</li></ul>" ], + [ "reply", "<ul><li>The mail is a reply to another patch.</li></ul>" ], [ "other", "<ul><li>The patch belongs to another WineHQ project (website, appdb, etc.) and will be applied by the respective maintainer.</li></ul>" ], [ "applyfail", "<ul><li>The patch got mangled in transit.</li>" . "<li>It's not relative to the latest git.</li>" . @@ -73,8 +75,7 @@ my @legend = "<li>8-bit chars got mangled in transit (usually when sending patches to translation files).</li>" . "<li>You are making gratuitous formatting changes to the code.</li>" . "<li>You are using C++ comments.</li></ul>" ], - [ "nopatch", "<ul><li>You hit 'send' before attaching the patch.</li>" . - "<li>The mail is a reply to another patch, or spam.</li></ul>" ], + [ "nopatch", "<ul><li>You hit 'send' before attaching the patch.</li></ul>" ], [ "rejected", "<ul><li>The patch has been rejected with a comment on wine-devel or #winehackers.</li>" . "<li>The patch contains an obvious error that you are expected to figure out yourself.</li></ul>" ], [ "split", "<ul><li>A single mail contained multiple patch files.</li>" .