mirror of
https://github.com/ansible-collections/community.general.git
synced 2024-09-14 20:13:21 +02:00
opkg module: allow installing a package in a certain version (#5688)
* opkg: allow installing a package in a certain version example: - name: Install foo in version 1.2 community.general.opkg: name: foo=1.2 state: present Signed-off-by: Joerg Hofrichter <joerg.hofrichter@ni.com> * opkg: use list for passing arguments to run_command Signed-off-by: Joerg Hofrichter <joerg.hofrichter@ni.com> Signed-off-by: Joerg Hofrichter <joerg.hofrichter@ni.com>
This commit is contained in:
parent
28969c61ad
commit
b3485b8fca
2 changed files with 53 additions and 11 deletions
|
@ -0,0 +1,2 @@
|
|||
minor_changes:
|
||||
- opkg - allow installing a package in a certain version (https://github.com/ansible-collections/community.general/pull/5688).
|
|
@ -22,6 +22,9 @@ options:
|
|||
name:
|
||||
description:
|
||||
- Name of package(s) to install/remove.
|
||||
- C(NAME=VERSION) syntax is also supported to install a package
|
||||
in a certain version. See the examples. This is supported since
|
||||
community.general 6.2.0.
|
||||
aliases: [pkg]
|
||||
required: true
|
||||
type: list
|
||||
|
@ -64,6 +67,11 @@ EXAMPLES = '''
|
|||
name: foo
|
||||
state: present
|
||||
|
||||
- name: Install foo in version 1.2
|
||||
community.general.opkg:
|
||||
name: foo=1.2
|
||||
state: present
|
||||
|
||||
- name: Update cache and install foo
|
||||
community.general.opkg:
|
||||
name: foo
|
||||
|
@ -96,22 +104,39 @@ from ansible.module_utils.six.moves import shlex_quote
|
|||
def update_package_db(module, opkg_path):
|
||||
""" Updates packages list. """
|
||||
|
||||
rc, out, err = module.run_command("%s update" % opkg_path)
|
||||
rc, out, err = module.run_command([opkg_path, "update"])
|
||||
|
||||
if rc != 0:
|
||||
module.fail_json(msg="could not update package db")
|
||||
|
||||
|
||||
def query_package(module, opkg_path, name, state="present"):
|
||||
def query_package(module, opkg_path, name, version=None, state="present"):
|
||||
""" Returns whether a package is installed or not. """
|
||||
|
||||
if state == "present":
|
||||
rc, out, err = module.run_command([opkg_path, "list-installed", name])
|
||||
if rc != 0:
|
||||
return False
|
||||
# variable out is one line if the package is installed:
|
||||
# "NAME - VERSION - DESCRIPTION"
|
||||
if version is not None:
|
||||
if not out.startswith("%s - %s " % (name, version)):
|
||||
return False
|
||||
else:
|
||||
if not out.startswith(name + " "):
|
||||
return False
|
||||
return True
|
||||
else:
|
||||
raise NotImplementedError()
|
||||
|
||||
rc, out, err = module.run_command("%s list-installed | grep -q \"^%s \"" % (shlex_quote(opkg_path), shlex_quote(name)), use_unsafe_shell=True)
|
||||
if rc == 0:
|
||||
return True
|
||||
|
||||
return False
|
||||
def split_name_and_version(module, package):
|
||||
""" Split the name and the version when using the NAME=VERSION syntax """
|
||||
splitted = package.split('=', 1)
|
||||
if len(splitted) == 1:
|
||||
return splitted[0], None
|
||||
else:
|
||||
return splitted[0], splitted[1]
|
||||
|
||||
|
||||
def remove_packages(module, opkg_path, packages):
|
||||
|
@ -125,11 +150,16 @@ def remove_packages(module, opkg_path, packages):
|
|||
remove_c = 0
|
||||
# Using a for loop in case of error, we can report the package that failed
|
||||
for package in packages:
|
||||
package, version = split_name_and_version(module, package)
|
||||
|
||||
# Query the package first, to see if we even need to remove
|
||||
if not query_package(module, opkg_path, package):
|
||||
continue
|
||||
|
||||
rc, out, err = module.run_command("%s remove %s %s" % (opkg_path, force, package))
|
||||
if force:
|
||||
rc, out, err = module.run_command([opkg_path, "remove", force, package])
|
||||
else:
|
||||
rc, out, err = module.run_command([opkg_path, "remove", package])
|
||||
|
||||
if query_package(module, opkg_path, package):
|
||||
module.fail_json(msg="failed to remove %s: %s" % (package, out))
|
||||
|
@ -154,13 +184,23 @@ def install_packages(module, opkg_path, packages):
|
|||
install_c = 0
|
||||
|
||||
for package in packages:
|
||||
if query_package(module, opkg_path, package) and (force != '--force-reinstall'):
|
||||
package, version = split_name_and_version(module, package)
|
||||
|
||||
if query_package(module, opkg_path, package, version) and (force != '--force-reinstall'):
|
||||
continue
|
||||
|
||||
rc, out, err = module.run_command("%s install %s %s" % (opkg_path, force, package))
|
||||
if version is not None:
|
||||
version_str = "=%s" % version
|
||||
else:
|
||||
version_str = ""
|
||||
|
||||
if not query_package(module, opkg_path, package):
|
||||
module.fail_json(msg="failed to install %s: %s" % (package, out))
|
||||
if force:
|
||||
rc, out, err = module.run_command([opkg_path, "install", force, package + version_str])
|
||||
else:
|
||||
rc, out, err = module.run_command([opkg_path, "install", package + version_str])
|
||||
|
||||
if not query_package(module, opkg_path, package, version):
|
||||
module.fail_json(msg="failed to install %s%s: %s" % (package, version_str, out))
|
||||
|
||||
install_c += 1
|
||||
|
||||
|
|
Loading…
Reference in a new issue