Commit graph

471 commits

Author SHA1 Message Date
Michael DeHaan
616d749ab3 Merge pull request #8156 from jirutka/fix-layman
layman: remove my email address
2014-07-16 11:22:13 -04:00
James Cammarata
19571d3f23 Merge pull request #7914 from hacosta/devel
Correctly handle .repo files in zypper_repository module
2014-07-16 09:55:35 -05:00
Jakub Jirutka
437dc0a9ef layman: remove my email address 2014-07-16 01:29:36 +02:00
Michael DeHaan
ca918cd3dd Merge pull request #8118 from oxyrox/devel
Typo
2014-07-15 18:15:08 -04:00
Michael DeHaan
ee179e1149 Merge pull request #8111 from hfinucane/gh-8108
Fix pip failure when version is parsed as a float
2014-07-15 18:07:03 -04:00
Michael DeHaan
50f51367f9 Merge pull request #8106 from muff1nman/pacman-update-checkmode-fix
Fix #8047
2014-07-15 18:04:50 -04:00
Aladin Jaermann
015a22ffba Correct typo 2014-07-14 10:49:08 +02:00
Henry Finucane
dfd1470c22 Ensure version is a string 2014-07-12 21:05:32 -07:00
Andrew DeMaria
5fdd2a7b01 Fix #8047
Ensure that json is returned when only updating the cache and in check mode
2014-07-12 10:57:30 -05:00
Dustin C. Hatch
dc304e6488 Correct regex in is_keyid
The `rpm_key` module claims to take a key ID as an argument, however it fails if one is actually given:

    msg: Not a valid key DEADBEEF

The reason for this is the regular expression in the `RpmKey.is_keyid` method is incorrect; it will only match the literal string "0-9a-f" exactly 8 times, rather than exactly 8 characters in the set 0-9 or a-f.
2014-07-11 14:02:25 -05:00
Hector Acosta
d3268d05ec Correctly handle .repo files in zypper_repository module
Before the changes, removing a repository required a repo url. This shouldn't be required since zypper allows removing a repo based on its alias (mapped to name in this module).

The name variable was always required, which is misleading since repofiles provide their own alias. So a runtime check was added to avoid this confusion.

Additionaly, running this module on .repo files weren't idempotent. e.g

Before:
$ ./hacking/test-module -m library/packaging/zypper_repository -a "repo=http://download.opensuse.org/repositories/devel:/languages:/python/SLE_11_SP3/devel:languages:python.repo name=foo"
{"repo": "http://download.opensuse.org/repositories/devel:/languages:/python/SLE_11_SP3/devel:languages:python.repo", "state": "present", "changed": true}
$ ./hacking/test-module -m library/packaging/zypper_repository -a "repo=http://download.opensuse.org/repositories/devel:/languages:/python/SLE_11_SP3/devel:languages:python.repo name=foo"
{"msg": "Repository named 'devel_languages_python' already exists. Please use another alias.\n", "failed": true}

After:
$ ./hacking/test-module -m library/packaging/zypper_repository -a "repo=http://download.opensuse.org/repositories/devel:/languages:/python/SLE_11_SP3/devel:languages:python.repo"
{"repo": "http://download.opensuse.org/repositories/devel:/languages:/python/SLE_11_SP3/devel:languages:python.repo", "state": "present", "changed": true}
$ ./hacking/test-module -m library/packaging/zypper_repository -a "repo=http://download.opensuse.org/repositories/devel:/languages:/python/SLE_11_SP3/devel:languages:python.repo"
{"repo": "http://download.opensuse.org/repositories/devel:/languages:/python/SLE_11_SP3/devel:languages:python.repo", "state": "present", "changed": false}

Signed-off-by: Hector Acosta <hector.acosta@gmail.com>
2014-06-26 13:17:27 -07:00
James Cammarata
29434b06b8 Merge pull request #7942 from sidred10/devel
Fixed FreeBSD ports install issue
2014-06-26 11:50:33 -05:00
sid
08370b3caf Fixed FreeBSD ports install issue 2014-06-26 20:52:57 +05:30
Maxim Odinintsev
82c026387f Successfull exit if sync=yes and no package provided 2014-06-26 15:02:15 +03:00
Maxim Odinintsev
a58912773d Success exit if sync=yes and no package provided 2014-06-26 14:55:16 +03:00
James Cammarata
2305ac9881 Merge pull request #7930 from trbs/apt_key_fetch_url_failure_codes
apt_key clear message when downloading key returns an (http) error code
2014-06-25 22:27:08 -05:00
George Yoshida
779c6cb7f8 In the yum module example, use present/absent instead of installed/removed 2014-06-25 23:21:43 +09:00
trbs
dfc625a7df apt_key clear message when downloading key returns an (http) error code 2014-06-25 15:30:34 +02:00
Sam Hanes
769ee61f66 Restore #6484 reverted by c751168. 2014-06-23 17:12:19 -07:00
James Cammarata
67fb2e653e Merge branch 'upgrade-release' of https://github.com/amenonsen/ansible into amenonsen-upgrade-release 2014-06-23 10:35:48 -05:00
yuyak
5482208eee Fix typo in apt module documentation 2014-06-21 21:27:57 +08:00
James Cammarata
57a2104c40 Merge pull request #7801 from koenpunt/npm-rbenv-exec
Allow gem executable to contain spaces
2014-06-17 11:16:42 -05:00
Koen Punt
d9f9234084 Allow gem executable to contain spaces
rbenv has a special exec function which loads the correct ruby version before executing a command.
The syntax for this is `/usr/local/rbenv/bin/rbenv exec gem ...` for example.
But previously when specifying executable='/usr/local/rbenv/bin/rbenv exec gem' this would not work because the string was treated as one executable. This PR fixes that by splitting the executable on spaces.
2014-06-17 11:58:10 +02:00
Koen Punt
2748095a8e Allow executable to contain spaces
NVM has a special script which loads the correct node version before executing a command.
The syntax for this is `/usr/local/nvm/nvm-exec ...`, so `nvm-exec npm list --json` for example.
But previously when specifying `executable='/usr/local/nvm/nvm-exec nvm'` this would not work because the string was treated as one executable.
2014-06-17 01:31:24 +02:00
James Cammarata
ddb946beb1 Minor cleanup of openbsd_pkg example comment 2014-06-16 11:35:57 -05:00
James Cammarata
df15990e0c Merge pull request #7774 from mwpher/devel
[openbsd-pkg] Add note regarding package flavors
2014-06-16 11:33:53 -05:00
Todd Owen
247f70cf5c Fix in pkgutil: don't call pipes.quote() on None.
In this function, 'site' may be None, which will cause an exception
if it is passed to pipes.quote().
2014-06-15 10:20:35 +08:00
Matthew Pherigo
127201559f [openbsd-pkg] Add note regarding package flavors
This adds a note to the openbsd-pkg module which explains how to specify
package flavors as well as how to avoid ambiguity errors on a package
with multiple flavors.

Technically, this is not Ansible-specific, but it's pretty likely the
user will encounter this problem.
2014-06-14 19:42:09 -05:00
James Cammarata
61325f5817 Cleaning up svr4pkg commit fix for #7645 2014-06-09 10:48:06 -05:00
James Cammarata
19d6f4d177 Merge branch '7645' of https://github.com/kcghost/ansible into kcghost-7645 2014-06-09 10:45:29 -05:00
Casey Fitzpatrick
fdbb77da51 fix for svr4pkg module failure reporting, issue #7645 2014-06-08 19:59:44 -04:00
Jean-Frédéric
c42f8907fd Fix documentation of deb option in apt module
The apt module may install a .deb package on the remote machine, not on the local.
Fix the misleading documentation.
2014-06-06 18:18:44 +02:00
Andrew Resch
00389af1c2 Fix typo in variable name 2014-05-23 16:09:57 -07:00
Andrew Resch
fb1385bbd9 Replace CommandException with Exception 2014-05-23 12:44:01 -07:00
Daniel Jaouen
37fff3dc38 Add upgrade_all option to homebrew module. 2014-05-23 11:09:18 -04:00
James Cammarata
d44ed533b3 Default use_proxy to True for fetch_url()
Also added some error handling to the fetch_url() call in the
apt_repository module, so that failures to look up the PPA info
are properly handled.

Fixes #7322
2014-05-19 23:04:13 -05:00
Abhijit Menon-Sen
0dea177e77 Allow '-t ' to be passed to apt: upgrade 2014-05-17 23:02:20 +05:30
James Cammarata
923baa6772 Merge branch 'yum_module_typos' of https://github.com/mpeters/ansible into mpeters-yum_module_typos 2014-05-16 09:24:47 -05:00
Michael Peters
15b3b42525 slight language change based on feedback from jimi-c 2014-05-16 09:44:32 -04:00
Michael Peters
69923cc413 fixing some typos in the docs of the yum module 2014-05-16 09:22:12 -04:00
Alan Pearce
670068aaad Update composer example
Remove unsupported 'install' argument
2014-05-16 14:06:55 +01:00
Yuichi TANIKAWA
2b644da4ca Add dots to valid path characters 2014-05-10 18:13:16 +09:00
James Cammarata
537494c2d6 Make sure distro variable is set if python-apt is installed
Fixes #7295
2014-05-06 09:42:50 -05:00
Michael DeHaan
5c75ce1133 In apt module show 'name' vs pkg to standardize with the rest of the modules. 'pkg' is an alias for name. 2014-04-29 15:48:51 -04:00
mkosmach
fbf3b78ace add simplejson as alternative to json 2014-04-25 02:44:51 -07:00
Michael DeHaan
31f7a0693e Add missing license headers to two modules and standardize formatting of another. 2014-04-24 10:32:54 -04:00
James Cammarata
83f300ddd6 Merge branch 'apt_key_format' of https://github.com/andreasf/ansible into andreasf-apt_key_format 2014-04-24 00:02:10 -05:00
James Cammarata
46c0eb2f3e Merge pull request #7022 from jjshoe/patch-1
apt_key lists all keys in uppercase
2014-04-23 15:58:23 -05:00
James Cammarata
afee14173e Merge pull request #7095 from radeksimko/pokemon-exc-fix
Prevent catching unrelated exceptions
2014-04-21 11:49:04 -05:00
Radek Simko
1513d2bd64 Prevention for catching unrelated exceptions added 2014-04-21 15:35:49 +01:00