fix removal case where first pkg in a list is not installed at all

and the next one gets ignored despite being installed.

also clean up unnecessary second check.
This commit is contained in:
Seth Vidal 2012-10-29 13:48:20 -04:00
parent 3fcc591f18
commit 7c1e0de141

View file

@ -440,13 +440,9 @@ def remove(module, items, repoq, yum_basecmd, conf_file, en_repos, dis_repos):
else:
pkglist = is_installed(module, repoq, spec, conf_file, en_repos=en_repos, dis_repos=dis_repos)
if not pkglist:
res['msg'] += "No Package matching '%s' found installed" % spec
module.exit_json(**res)
found = False
for this in pkglist:
if is_installed(module, repoq, this, conf_file, en_repos=en_repos, dis_repos=dis_repos):
found = True
found = False
else:
found = True
if not found:
res['results'].append('%s is not installed' % spec)