summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorZac Medico <zmedico@gentoo.org>2008-07-28 01:42:06 +0000
committerZac Medico <zmedico@gentoo.org>2008-07-28 01:42:06 +0000
commit1033d4f65b7a126576bb6edf1b4a2a4c0f4f9471 (patch)
treefed1d6ec943bb4d744bc71797b1c318ecafb604e
parent6b96ba70a98478b00a6f50337970a6c3136d665f (diff)
downloadportage-1033d4f65b7a126576bb6edf1b4a2a4c0f4f9471.tar.gz
portage-1033d4f65b7a126576bb6edf1b4a2a4c0f4f9471.tar.bz2
portage-1033d4f65b7a126576bb6edf1b4a2a4c0f4f9471.zip
* Add support in dep_getusedeps() and isvalidatom() for comma separated USE
deps that only have one set of square brackets. * Add test cases for the new comma separated USE deps syntax. svn path=/main/trunk/; revision=11229
-rw-r--r--pym/portage/dep.py31
-rw-r--r--pym/portage/tests/dep/test_isvalidatom.py5
2 files changed, 33 insertions, 3 deletions
diff --git a/pym/portage/dep.py b/pym/portage/dep.py
index 737c009a9..15000dd84 100644
--- a/pym/portage/dep.py
+++ b/pym/portage/dep.py
@@ -588,16 +588,35 @@ def dep_getusedeps( depend ):
use_list = []
open_bracket = depend.find('[')
# -1 = failure (think c++ string::npos)
+ comma_separated = False
+ bracket_count = 0
while( open_bracket != -1 ):
+ bracket_count += 1
close_bracket = depend.find(']', open_bracket )
if close_bracket == -1:
raise InvalidAtom("USE Dependency with no closing bracket: %s" % depend )
use = depend[open_bracket + 1: close_bracket]
# foo[1:1] may return '' instead of None, we don't want '' in the result
- if len(use):
- use_list.append(use)
+ if not use:
+ raise InvalidAtom("USE Dependency with " + \
+ "no use flag ([]): %s" % depend )
+ if not comma_separated:
+ comma_separated = "," in use
+
+ if comma_separated and bracket_count > 1:
+ raise InvalidAtom("USE Dependency contains a mixture of " + \
+ "comma and bracket separators: %s" % depend )
+
+ if comma_separated:
+ for x in use.split(","):
+ if x:
+ use_list.append(x)
+ else:
+ raise InvalidAtom("USE Dependency with no use " + \
+ "flag next to comma: %s" % depend )
else:
- raise InvalidAtom("USE Dependency with no use flag ([]): %s" % depend )
+ use_list.append(use)
+
# Find next use flag
open_bracket = depend.find( '[', open_bracket+1 )
return tuple(use_list)
@@ -634,6 +653,12 @@ def isvalidatom(atom, allow_blockers=False):
return 0
if allow_blockers and atom.startswith("!"):
atom = atom[1:]
+
+ try:
+ dep_getusedeps(atom)
+ except InvalidAtom:
+ return 0
+
cpv = dep_getcpv(atom)
cpv_catsplit = catsplit(cpv)
mycpv_cps = None
diff --git a/pym/portage/tests/dep/test_isvalidatom.py b/pym/portage/tests/dep/test_isvalidatom.py
index dfcf17898..d2210b8e9 100644
--- a/pym/portage/tests/dep/test_isvalidatom.py
+++ b/pym/portage/tests/dep/test_isvalidatom.py
@@ -26,6 +26,11 @@ class IsValidAtom(TestCase):
( "sys-apps/portage-2.1:foo", False ),
( "sys-apps/portage-2.1:", False ),
( "=sys-apps/portage-2.2*:foo[bar][-baz][doc?][-build?]", True ),
+ ( "=sys-apps/portage-2.2*:foo[bar,-baz,doc?,-build?]", True ),
+ ( "=sys-apps/portage-2.2*:foo[bar,-baz,doc?,-build?,]", False ),
+ ( "=sys-apps/portage-2.2*:foo[,bar,-baz,doc?,-build?]", False ),
+ ( "=sys-apps/portage-2.2*:foo[bar,-baz][doc?,-build?]", False ),
+ ( "=sys-apps/portage-2.2*:foo[bar][doc,build]", False ),
( ">~cate-gory/foo-1.0", False ),
( ">~category/foo-1.0", False ),
( "<~category/foo-1.0", False ),