Commit Graph

11 Commits (da95dd56521a3c92e88bb00a496760322c6f66dc)

Author SHA1 Message Date
Michael Scherer 49aa069c94 Add proper type to path and link
Since both of them are path, it should be checked using the
proper type.
2016-12-08 11:33:58 -05:00
Greg DeKoenigsberg c65a612d1f Changes to author formatting, remove emails 2016-12-08 11:32:51 -05:00
Christian Thiemann c20ab3abef Fix alternatives module in non-English locale
The alternatives module parses the output of update-alternatives, but the expected English phrases may not show up if the system locale is not English. Setting LC_ALL=C when invoking update-alternatives fixes this problem.
2016-12-08 11:32:45 -05:00
Brian Coca a7b46316c1 removed executable bit 2016-12-08 11:32:42 -05:00
Brian Coca 2753cd186a moar quotes 2016-12-08 11:32:39 -05:00
Matt Martz 8bd5757720 Fix up modules that have python24 syntax error 2016-12-08 11:32:36 -05:00
David Wittman ef0f852041 Fix multiple issues with alternatives module
- Changes are no longer erroneously reported on RHEL (#12)
 - Adding new link groups on Debian works again.
   - This was broken in a previous commit by assuming the OS was RHEL
     if `update-alternatives --query <name>` had a return code of 2
 - Prefer `--display` over `--query` for determining available
   alternatives
   - --display is more distro-agnostic and simplifies the code
 - Fix missing `msg=` in `fail_json` call when `link` is missing
 - Document that `link` is required on RHEL-based distros

Tested on Ubuntu 12.04+ and CentOS 6/7
2016-12-08 11:32:29 -05:00
Brian Coca f5a1196cc7 now properly checks that link is suplied before trying to use it should fix #46 2016-12-08 11:32:25 -05:00
Aurélien Bompard 27afdc9566 Module alternatives: support RedHat-based OSes
RedHat-based OSes have a version of update-alternatives which comes from
the chkconfig package and does not support the --query parameter. Work
around that.
2016-12-08 11:32:20 -05:00
Aurélien Bompard 2d5c2dabe8 Module alternatives: support check mode 2016-12-08 11:32:20 -05:00
Michael DeHaan 5879c503bb File extensions! 2016-12-08 11:32:18 -05:00