Alexandre Julliard : patches: Add a patch status for missing authorship information.

[Alexandre Julliard]

Module: tools
Branch: master
Commit: 34dd20b4de6c0971813b930b718ebe33ee086f31
URL:    http://source.winehq.org/git/tools.git/?a=commit;h=34dd20b4de6c0971813b930b7...

Author: Alexandre Julliard <julliard(a)winehq.org>
Date:   Tue Mar  7 10:49:32 2017 +0100

patches: Add a patch status for missing authorship information.

Signed-off-by: Alexandre Julliard <julliard(a)winehq.org>

---

 patches/expire      | 15 +++++++++++----
 patches/patches.css |  1 +
 patches/update      |  5 +++--
 3 files changed, 15 insertions(+), 6 deletions(-)

diff --git a/patches/expire b/patches/expire
index 6bbd5c8..fd5ba22 100755
--- a/patches/expire
+++ b/patches/expire
@@ -31,6 +31,7 @@ my %status_expiry =
  "applyfail"  => 7,
  "buildfail"  => 7,
  "deferred"   => 7,
+ "authorship" => 7,
  "formatting" => 7,
  "other"      => 7,
  "nopatch"    => 3,
@@ -54,6 +55,7 @@ my %status_descr =
  "applyfail"  => "Apply failure",
  "buildfail"  => "Build failure",
  "deferred"   => "Deferred",
+ "authorship" => "Authorship",
  "formatting" => "Formatting",
  "other"      => "Other project",
  "nopatch"    => "No patch",
@@ -98,12 +100,17 @@ mode implies -Werror). You need to resend a fixed patch.",
 upcoming release. You need to either resend after the release, make
 the patch less intrusive, or make a convincing argument that it needs
 to be included in the release.",
+ "authorship" =>
+"This means that there are some authorship issues with your patch. The
+author's full name, email address, or Signed-off-by headers are
+missing or incorrectly formatted. You need to resend a patch with
+corrected author information.",
  "formatting" =>
 "This means that there are some formatting issues with your
-patch. Either the author's full name, email address, or Signed-off-by
-headers are missing, indentation is messed up (usually from editing
-with a non-standard tab size), or there are gratuitous formatting
-changes to the code. You need to resend a fixed patch.",
+patch. Either the indentation is messed up (usually from editing with
+a non-standard tab size), some characters got mangled in transit, or
+there are gratuitous formatting changes to the code. You need to
+resend a fixed patch.",
  "other"      =>
 "This means that the patch belongs to another WineHQ project (website,
 appdb, etc.) and will be applied by the respective maintainer.",
diff --git a/patches/patches.css b/patches/patches.css
index 020d48c..2799827 100644
--- a/patches/patches.css
+++ b/patches/patches.css
@@ -162,6 +162,7 @@ tr.odd { background-color: #f8e8e8; }
 .other,      .other :link,      .other :visited      { color: #e08000; }
 .applyfail,  .applyfail :link,  .applyfail :visited  { color: red; }
 .buildfail,  .buildfail :link,  .buildfail :visited  { color: red; }
+.authorship, .authorship :link, .authorship :visited { color: red; }
 .formatting, .formatting :link, .formatting :visited { color: red; }
 .nopatch,    .nopatch :link,    .nopatch :visited    { color: red; }
 .rejected,   .rejected :link,   .rejected :visited   { color: red; }
diff --git a/patches/update b/patches/update
index ddf1396..ad2b82a 100755
--- a/patches/update
+++ b/patches/update
@@ -31,6 +31,7 @@ my %status_descr =
  "applyfail"  => "Apply failure",
  "buildfail"  => "Build failure",
  "deferred"   => "Deferred",
+ "authorship" => "Authorship",
  "formatting" => "Formatting",
  "other"      => "Other project",
  "nopatch"    => "No patch",
@@ -66,8 +67,8 @@ my @legend =
  [ "buildfail",  "<ul><li>Syntax error or other compile problem.</li>" .
                  "<li>You forgot to include some changes or new files required for compilation.</li>" .
                  "<li>The patch causes compiler warnings (maintainer mode implies -Werror).</li></ul>" ],
- [ "formatting", "<ul><li>The patch author's full name, email address, or Signed-off-by headers are incorrect or missing.</li>" .
-                 "<li>Whitespace got mangled in transit.</li>" .
+ [ "authorship", "<ul><li>The patch author's full name, email address, or Signed-off-by headers are incorrect or missing.</li></ul>" ],
+ [ "formatting", "<ul><li>Whitespace got mangled in transit.</li>" .
                  "<li>Indentation is messed up (usually from editing with a non-standard tab size).</li>" .
                  "<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>" .