Alexandre Julliard : patches: Add a patch status for replies to a patch.

[Alexandre Julliard]

Module: tools
Branch: master
Commit: 045638407bea42e8cdc68b1b354d7ecdaf862390
URL:    http://source.winehq.org/git/tools.git/?a=commit;h=045638407bea42e8cdc68b1b3...

Author: Alexandre Julliard 
Date:   Mon Apr  3 13:10:54 2017 +0200

patches: Add a patch status for replies to a patch.

Signed-off-by: Alexandre Julliard

---

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>" .