diff options
| author |   Zac Medico <zmedico@gentoo.org> | 2009-03-10 05:16:11 +0000 |
|---|---|---|
| committer | Zac Medico <zmedico@gentoo.org> | 2009-03-10 05:16:11 +0000 |
| commit | e40282011279ed59bc87c7ee3d1811a0cfcf19e2 (patch) | |
| tree | 230f699d108105e16d4866154a33662865668bb9 /pym | |
| parent | Fix incorrect slice inside _src_uri_validate() and add new test case. (diff) | |
| download | portage-multirepo-e40282011279ed59bc87c7ee3d1811a0cfcf19e2.tar.gz portage-multirepo-e40282011279ed59bc87c7ee3d1811a0cfcf19e2.tar.bz2 portage-multirepo-e40282011279ed59bc87c7ee3d1811a0cfcf19e2.zip | |
Remove redundant conditional inside _src_uri_validate().
svn path=/main/trunk/; revision=12819
Diffstat (limited to 'pym')
| -rw-r--r-- | pym/portage/dbapi/porttree.py | 22 |
1 files changed, 13 insertions, 9 deletions
diff --git a/pym/portage/dbapi/porttree.py b/pym/portage/dbapi/porttree.py index 8dcdf1b8..caff7f80 100644 --- a/pym/portage/dbapi/porttree.py +++ b/pym/portage/dbapi/porttree.py @@ -69,15 +69,19 @@ def _src_uri_validate(cpv, eapi, src_uri): if operator is None: uri = x continue - if operator is not None: - if "/" in x: - raise portage.exception.InvalidDependString( - ("getFetchMap(): '%s' SRC_URI '/' character in " + \ - "file name: '%s'") % (cpv, x)) - if x[-1:] == "?": - raise portage.exception.InvalidDependString( - ("getFetchMap(): '%s' SRC_URI arrow missing " + \ - "right operand") % (cpv,)) + + # This should be the right operand of an arrow operator. + if "/" in x: + raise portage.exception.InvalidDependString( + ("getFetchMap(): '%s' SRC_URI '/' character in " + \ + "file name: '%s'") % (cpv, x)) + + if x[-1:] == "?": + raise portage.exception.InvalidDependString( + ("getFetchMap(): '%s' SRC_URI arrow missing " + \ + "right operand") % (cpv,)) + + # Found the right operand, so reset state. uri = None operator = None |