Module: tools Branch: master Commit: 34dd20b4de6c0971813b930b718ebe33ee086f31 URL: http://source.winehq.org/git/tools.git/?a=commit;h=34dd20b4de6c0971813b930b7...
Author: Alexandre Julliard julliard@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@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>" .