Module: tools
Branch: master
Commit: 34dd20b4de6c0971813b930b718ebe33ee086f31
URL: http://source.winehq.org/git/tools.git/?a=commit;h=34dd20b4de6c0971813b930b…
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>" .