diff options
| author | Sebastian Luther <SebastianLuther@gmx.de> | 2010-04-13 16:33:28 +0200 |
|---|---|---|
| committer | Zac Medico <zmedico@gentoo.org> | 2010-08-10 18:27:37 -0700 |
| commit | 4930057bcb1b659a3075739dd46edafbbdd6deb9 (patch) | |
| tree | e0f4814cb24f67ca911f13d1430ac1da42ba1e58 | |
| parent | c4d225f0bdedd7be72a5b6b2f10345773233cead (diff) | |
| download | portage-4930057bcb1b659a3075739dd46edafbbdd6deb9.tar.gz portage-4930057bcb1b659a3075739dd46edafbbdd6deb9.tar.bz2 portage-4930057bcb1b659a3075739dd46edafbbdd6deb9.zip | |
Add repoman check for REQUIRED_USE
| -rwxr-xr-x | bin/repoman | 17 | ||||
| -rw-r--r-- | pym/portage/dep/__init__.py | 4 |
2 files changed, 21 insertions, 0 deletions
diff --git a/bin/repoman b/bin/repoman index 0115c2885..2a53bd27e 100755 --- a/bin/repoman +++ b/bin/repoman @@ -322,6 +322,7 @@ qahelp={ "PROVIDE.syntax":"Syntax error in PROVIDE (usually an extra/missing space/parenthesis)", "PROPERTIES.syntax":"Syntax error in PROPERTIES (usually an extra/missing space/parenthesis)", "RESTRICT.syntax":"Syntax error in RESTRICT (usually an extra/missing space/parenthesis)", + "REQUIRED_USE.syntax":"Syntax error in REQUIRED_USE (usually an extra/missing space/parenthesis)", "SRC_URI.syntax":"Syntax error in SRC_URI (usually an extra/missing space/parenthesis)", "SRC_URI.mirror":"A uri listed in profiles/thirdpartymirrors is found in SRC_URI", "ebuild.syntax":"Error generating cache entry for ebuild; typically caused by ebuild syntax error or digest verification failure", @@ -1785,6 +1786,22 @@ for x in scanlist: stats["RESTRICT.invalid"] += len(mybadrestrict) for mybad in mybadrestrict: fails["RESTRICT.invalid"].append(x+"/"+y+".ebuild: %s" % mybad) + #REQUIRED_USE check + required_use = myaux["REQUIRED_USE"] + if required_use: + if eapi in ("0", "1", "2", "3"): + stats['EAPI.incompatible'] += 1 + fails['EAPI.incompatible'].append( + relative_path + ": REQUIRED_USE" + \ + " not supported with EAPI='%s'" % (eapi,)) + try: + portage.dep.check_required_use(required_use, "", myaux["IUSE"].split()) + except portage.exception.InvalidRequiredUseString as e: + stats["REQUIRED_USE.syntax"] = stats["REQUIRED_USE.syntax"] + 1 + fails["REQUIRED_USE.syntax"].append( + "%s: REQUIRED_USE: %s" % (relative_path, e)) + del e + # Syntax Checks relative_path = os.path.join(x, y + ".ebuild") full_path = os.path.join(repodir, relative_path) diff --git a/pym/portage/dep/__init__.py b/pym/portage/dep/__init__.py index a2dbbe9d0..30b181988 100644 --- a/pym/portage/dep/__init__.py +++ b/pym/portage/dep/__init__.py @@ -1450,6 +1450,10 @@ def _check_required_use(constraints, use, iuse): ("check_required_use(): '%s' constraint list without " + \ "use conditional or operator") % (constraints,)) + if not constraint: + raise portage.exception.InvalidRequiredUseString( + ("check_required_use(): '%s' syntax error") % (constraints,) ) + if constraint[-1] == "?": #a use conditional skip_next = True |