[patch] Add repoman check for period only

0001-Have-repoman-warn-when-DESCRIPTION-ends-with-a-perio.patch (text/plain), 1.67 KB, created by Michael Orlitzky on 2014-04-19 00:26:29 UTC

From 5c6eb45f45490da1ba2874d68ba14b02973bacc5 Mon Sep 17 00:00:00 2001
From: Michael Orlitzky <[e-mail]>
Date: Fri, 18 Apr 2014 20:20:47 -0400
Subject: [PATCH 1/1] Have repoman warn when $DESCRIPTION ends with a period.

---
 bin/repoman | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/bin/repoman b/bin/repoman
index 888892b..5ef5f0c 100755
--- a/bin/repoman
+++ b/bin/repoman
@@ -318,6 +318,7 @@ qahelp = {
 	"LICENSE.missing": "Ebuilds that have a missing or empty LICENSE variable",
 	"LICENSE.virtual": "Virtuals that have a non-empty LICENSE variable",
 	"DESCRIPTION.missing": "Ebuilds that have a missing or empty DESCRIPTION variable",
+    "DESCRIPTION.punctuation": "DESCRIPTION ends with a period character",
 	"DESCRIPTION.toolong": "DESCRIPTION is over %d characters" % max_desc_len,
 	"EAPI.definition": "EAPI definition does not conform to PMS section 7.3.1 (first non-comment, non-blank line)",
 	"EAPI.deprecated": "Ebuilds that use features that are deprecated in the current EAPI",
@@ -385,6 +386,7 @@ qawarnings = set((
 "dependency.badindev",
 "dependency.badmaskedindev",
 "dependency.badtilde",
+"DESCRIPTION.punctuation",
 "DESCRIPTION.toolong",
 "EAPI.deprecated",
 "HOMEPAGE.virtual",
@@ -1890,6 +1892,11 @@ for x in effective_scanlist:
 					stats[myqakey] += 1
 					fails[myqakey].append(relative_path)
 
+		if myaux['DESCRIPTION'] and myaux['DESCRIPTION'][-1] == '.':
+			stats['DESCRIPTION.punctuation'] += 1
+			fails['DESCRIPTION.punctuation'].append(
+				"%s: DESCRIPTION ends with a period character" % relative_path)
+
 		# 14 is the length of DESCRIPTION=""
 		if len(myaux['DESCRIPTION']) > max_desc_len:
 			stats['DESCRIPTION.toolong'] += 1
-- 
1.8.3.2