diff options
| author | Zac Medico <zmedico@gentoo.org> | 2008-01-11 07:22:07 +0000 |
|---|---|---|
| committer | Zac Medico <zmedico@gentoo.org> | 2008-01-11 07:22:07 +0000 |
| commit | 573ce721bd4ed67b979b56fdd996aa21fdb8faf2 (patch) | |
| tree | 64c42533b3de4727afc2a76cbd0c335543a0133e | |
| parent | 322441e4f44aeb130b37a030bd8b6fa6549894eb (diff) | |
| download | portage-573ce721bd4ed67b979b56fdd996aa21fdb8faf2.tar.gz portage-573ce721bd4ed67b979b56fdd996aa21fdb8faf2.tar.bz2 portage-573ce721bd4ed67b979b56fdd996aa21fdb8faf2.zip | |
Add a DESCRIPTION.toolong warning for when the DESCRIPTION line is
over 80 characters long. Thanks to Betelgeuse for the patch.
svn path=/main/trunk/; revision=9177
| -rwxr-xr-x | bin/repoman | 11 |
1 files changed, 11 insertions, 0 deletions
diff --git a/bin/repoman b/bin/repoman index ebeaea915..51fc8a395 100755 --- a/bin/repoman +++ b/bin/repoman @@ -73,6 +73,8 @@ from portage.output import ConsoleStyleFile, StyleWriter initialize_logger() +# 14 is the length of DESCRIPTION="" +max_desc_len = 80 - 14 allowed_filename_chars="a-zA-Z0-9._-+:" allowed_filename_chars_set = {} map(allowed_filename_chars_set.setdefault, map(chr, range(ord('a'), ord('z')+1))) @@ -259,6 +261,7 @@ qahelp={ "KEYWORDS.stupid":"Ebuilds that use KEYWORDS=-* instead of package.mask", "LICENSE.missing":"Ebuilds that have a missing or empty LICENSE variable", "DESCRIPTION.missing":"Ebuilds that have a missing or empty DESCRIPTION variable", + "DESCRIPTION.toolong":"DESCRIPTION line is over 80 characters", "EAPI.unsupported":"Ebuilds that have an unsupported EAPI version (you must upgrade portage)", "SLOT.missing":"Ebuilds that have a missing or empty SLOT variable", "HOMEPAGE.missing":"Ebuilds that have a missing or empty HOMEPAGE variable", @@ -320,6 +323,7 @@ qawarnings=[ "DEPEND.badmasked","RDEPEND.badmasked","PDEPEND.badmasked", "DEPEND.badindev","RDEPEND.badindev","PDEPEND.badindev", "DEPEND.badmaskedindev","RDEPEND.badmaskedindev","PDEPEND.badmaskedindev", +"DESCRIPTION.toolong", "IUSE.invalid", "KEYWORDS.stupid", "KEYWORDS.missing", @@ -1286,6 +1290,13 @@ for x in scanlist: stats[myqakey]=stats[myqakey]+1 fails[myqakey].append(x+"/"+y+".ebuild") + # 14 is the length of DESCRIPTION="" + if len(myaux['DESCRIPTION']) > max_desc_len: + stats['DESCRIPTION.toolong'] += 1 + fails['DESCRIPTION.toolong'].append( + "%s: DESCRIPTION is %d characters (max %d)" % \ + (relative_path, len(myaux['DESCRIPTION']), max_desc_len)) + keywords = myaux["KEYWORDS"].split() stable_keywords = [] for keyword in keywords: |