Bug #224271 - New check 'IUSE.undefined', renamed from generic

'ebuild.minorsyn'.

svn path=/main/trunk/; revision=10593

diff --git a/bin/repoman b/bin/repoman
index 3cabe0a73368781547e98c61f923321356ae469e..b9710be0fadd16c0779e79bd2bfd63c0f145a5f9 100755 (executable)
--- a/bin/repoman
+++ b/bin/repoman
@@ -293,6 +293,7 @@ qahelp={
        "variable.readonly":"Assigning a readonly variable",
        "LIVEVCS.stable":"This ebuild is a live checkout from a VCS but has stable keywords.",
        "IUSE.invalid":"This ebuild has a variable in IUSE that is not in the use.desc or use.local.desc file",
+       "IUSE.undefined":"This ebuild does not define IUSE (style guideline says to define IUSE even when empty)",
        "LICENSE.invalid":"This ebuild is listing a license that doesnt exist in portages license/ dir.",
        "KEYWORDS.invalid":"This ebuild contains KEYWORDS that are not listed in profiles/arch.list or for which no valid profile was found",
        "RDEPEND.suspect":"RDEPEND contains a package that usually only belongs in DEPEND.",
@@ -330,6 +331,7 @@ qawarnings=[
 "KEYWORDS.dropped",
 "KEYWORDS.stupid",
 "KEYWORDS.missing",
+"IUSE.undefined",
 "RDEPEND.suspect",
 "RESTRICT.invalid",
 "ebuild.minorsyn",

diff --git a/man/repoman.1 b/man/repoman.1
index eddba39197b9160768e3ee87c30ce5409a31155c..7f03365c206840e6978eed9bf676508c9aa019f4 100644 (file)
--- a/man/repoman.1
+++ b/man/repoman.1
@@ -114,6 +114,9 @@ Ebuilds that have a missing or empty HOMEPAGE variable
 .B IUSE.invalid
 This build has a variable in IUSE that is not in the use.desc or use.local.desc file
 .TP
+.B IUSE.undefined
+This ebuild does not define IUSE (style guideline says to define IUSE even when empty)
+.TP
 .B KEYWORDS.dropped
 Ebuilds that appear to have dropped KEYWORDS for some arch
 .TP

diff --git a/pym/repoman/checks.py b/pym/repoman/checks.py
index bfd06bc34d8772bc7e63e469f7ca654ba3b753c0..7bc9d990b4cedc7a580e4e5d0b7a300d64792a22 100644 (file)
--- a/pym/repoman/checks.py
+++ b/pym/repoman/checks.py
@@ -241,6 +241,6 @@ def run_checks(contents, st_mtime, inherited=None):
                                if e:
                                        yield lc.repoman_check_name, e % (num + 1)
        if iuse_def is None:
-               yield 'ebuild.minorsyn', 'IUSE is not defined'
+               yield 'IUSE.undefined', 'IUSE is not defined'
        if inherit_autotools and autotools_func_call is None:
                yield 'inherit.autotools', 'no eauto* function called'