Merge pull request #248 from bcoca/fix_alternatives

now properly checks that link is suplied before trying to use it
reviewable/pr18780/r1
Brian Coca 10 years ago
commit 83c051dfd5

@ -136,10 +136,13 @@ def main():
# install the requested path if necessary
# (unsupported on the RedHat version)
if path not in all_alternatives and os_family == "Debian":
module.run_command(
[UPDATE_ALTERNATIVES, '--install', link, name, path, str(DEFAULT_LINK_PRIORITY)],
check_rc=True
)
if link:
module.run_command(
[UPDATE_ALTERNATIVES, '--install', link, name, path, str(DEFAULT_LINK_PRIORITY)],
check_rc=True
)
else:
module.fail_json("Needed to install the alternative, but unable to do so, as we are missking the link")
# select the requested path
module.run_command(

Loading…
Cancel
Save