diff --git a/changelogs/fragments/61961-pacman_remove_recurse_option.yaml b/changelogs/fragments/61961-pacman_remove_recurse_option.yaml new file mode 100644 index 00000000000..0f64a8968cf --- /dev/null +++ b/changelogs/fragments/61961-pacman_remove_recurse_option.yaml @@ -0,0 +1,2 @@ +removed_features: + - pacman - Removed deprecated ``recurse`` option, use ``extra_args=--recursive`` instead diff --git a/docs/docsite/rst/porting_guides/porting_guide_2.10.rst b/docs/docsite/rst/porting_guides/porting_guide_2.10.rst index 5a2c70d9b96..07f87b494e6 100644 --- a/docs/docsite/rst/porting_guides/porting_guide_2.10.rst +++ b/docs/docsite/rst/porting_guides/porting_guide_2.10.rst @@ -62,6 +62,7 @@ Noteworthy module changes * :ref:`vmware_host_ntp ` now returns ``host_ntp_status`` instead of Ansible internal key ``results``. * :ref:`vmware_host_service_manager ` now returns ``host_service_status`` instead of Ansible internal key ``results``. * :ref:`vmware_tag ` now returns ``tag_status`` instead of Ansible internal key ``results``. +* The deprecated ``recurse`` option in :ref:`pacman ` module has been removed, you should use ``extra_args=--recursive`` instead. Plugins diff --git a/lib/ansible/modules/packaging/os/pacman.py b/lib/ansible/modules/packaging/os/pacman.py index 6a6ef126bb9..6ab812123b9 100644 --- a/lib/ansible/modules/packaging/os/pacman.py +++ b/lib/ansible/modules/packaging/os/pacman.py @@ -37,17 +37,6 @@ options: default: present choices: [ absent, latest, present ] - recurse: - description: - - When removing a package, also remove its dependencies, provided - that they are not required by other packages and were not - explicitly installed by a user. - This option is deprecated since 2.8 and will be removed in 2.10, - use `extra_args=--recursive`. - default: no - type: bool - version_added: "1.3" - force: description: - When removing package, force remove package, without any checks. @@ -429,7 +418,6 @@ def main(): argument_spec=dict( name=dict(type='list', aliases=['pkg', 'package']), state=dict(type='str', default='present', choices=['present', 'installed', 'latest', 'absent', 'removed']), - recurse=dict(type='bool', default=False), force=dict(type='bool', default=False), extra_args=dict(type='str', default=''), upgrade=dict(type='bool', default=False), @@ -452,13 +440,6 @@ def main(): elif p['state'] in ['absent', 'removed']: p['state'] = 'absent' - if p['recurse']: - module.deprecate('Option `recurse` is deprecated and will be removed in ' - 'version 2.10. Please use `extra_args=--recursive` ' - 'instead', '2.10') - if p['state'] == 'absent': - p['extra_args'] += " --recursive" - if p["update_cache"] and not module.check_mode: update_package_db(module, pacman_path) if not (p['name'] or p['upgrade']):