Allow dnf to remove dependent packages of a package that is being removed

Fixes #1517
reviewable/pr18780/r1
Toshio Kuratomi 9 years ago
parent 1b5c9ee7f9
commit b1e9dc0b52

@ -234,6 +234,7 @@ def _mark_package_install(module, base, pkg_spec):
def ensure(module, base, state, names): def ensure(module, base, state, names):
allow_erasing = False
if names == ['*'] and state == 'latest': if names == ['*'] and state == 'latest':
base.upgrade_all() base.upgrade_all()
else: else:
@ -280,6 +281,7 @@ def ensure(module, base, state, names):
_mark_package_install(module, base, pkg_spec) _mark_package_install(module, base, pkg_spec)
else: else:
# state == absent
if filenames: if filenames:
module.fail_json( module.fail_json(
msg="Cannot remove paths -- please specify package name.") msg="Cannot remove paths -- please specify package name.")
@ -291,8 +293,11 @@ def ensure(module, base, state, names):
for pkg_spec in pkg_specs: for pkg_spec in pkg_specs:
if installed.filter(name=pkg_spec): if installed.filter(name=pkg_spec):
base.remove(pkg_spec) base.remove(pkg_spec)
# Like the dnf CLI we want to allow recursive removal of dependent
# packages
allow_erasing = True
if not base.resolve(): if not base.resolve(allow_erasing=allow_erasing):
module.exit_json(msg="Nothing to do") module.exit_json(msg="Nothing to do")
else: else:
if module.check_mode: if module.check_mode:

Loading…
Cancel
Save