From 5c6eb45f45490da1ba2874d68ba14b02973bacc5 Mon Sep 17 00:00:00 2001 From: Michael Orlitzky 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