change name to be a list type

remove implicit split that expects a , separated string, let list type
deal with multiple possible compatible input types.
also removed unused imports
This commit is contained in:
Brian Coca 2016-03-22 15:27:23 -07:00
parent 745df06abc
commit 3b95400a59

View file

@ -63,10 +63,6 @@ EXAMPLES = '''
''' '''
import shlex
import os
import sys
import pipes
import re import re
def query_package(module, pkgin_path, name): def query_package(module, pkgin_path, name):
@ -214,14 +210,14 @@ def main():
module = AnsibleModule( module = AnsibleModule(
argument_spec = dict( argument_spec = dict(
state = dict(default="present", choices=["present","absent"]), state = dict(default="present", choices=["present","absent"]),
name = dict(aliases=["pkg"], required=True)), name = dict(aliases=["pkg"], required=True, type='list')),
supports_check_mode = True) supports_check_mode = True)
pkgin_path = module.get_bin_path('pkgin', True, ['/opt/local/bin']) pkgin_path = module.get_bin_path('pkgin', True, ['/opt/local/bin'])
p = module.params p = module.params
pkgs = p["name"].split(",") pkgs = p["name"]
if p["state"] == "present": if p["state"] == "present":
install_packages(module, pkgin_path, pkgs) install_packages(module, pkgin_path, pkgs)