diff options
author | Michael DeHaan <michael@ansibleworks.com> | 2014-03-11 08:39:28 -0400 |
---|---|---|
committer | Michael DeHaan <michael@ansible.com> | 2014-03-11 10:48:42 -0400 |
commit | 3bb3ce4467545e2e124c913a8c22bd39b0a7b7d6 (patch) | |
tree | e141c51e118ef6c500a02e772bd8bdaf359fdd66 | |
parent | 9d4589d57437df8f43099630038bcdc3e714430a (diff) | |
download | ansible-3bb3ce4467545e2e124c913a8c22bd39b0a7b7d6.tar.gz |
Update apt module so that environment variables are set correctly since not going through shell. Very sorry folks, it will be addressed.
-rw-r--r-- | library/packaging/apt | 17 |
1 files changed, 14 insertions, 3 deletions
diff --git a/library/packaging/apt b/library/packaging/apt index f143c8f7b7..f11ab73de1 100644 --- a/library/packaging/apt +++ b/library/packaging/apt @@ -138,7 +138,11 @@ import datetime import fnmatch # APT related constants -APT_ENVVARS = "DEBIAN_FRONTEND=noninteractive DEBIAN_PRIORITY=critical" +APT_ENV_VARS = dict( + DEBIAN_FRONTEND = 'noninteractive', + DEBIAN_PRIORITY = 'critical' +) + DPKG_OPTIONS = 'force-confdef,force-confold' APT_GET_ZERO = "0 upgraded, 0 newly installed" APTITUDE_ZERO = "0 packages upgraded, 0 newly installed" @@ -260,7 +264,10 @@ def install(m, pkgspec, cache, upgrade=False, default_release=None, else: check_arg = '' - cmd = "%s %s -y %s %s %s install %s" % (APT_ENVVARS, APT_GET_CMD, dpkg_options, force_yes, check_arg, packages) + for (k,v) in APT_ENV_VARS.iteritems(): + os.environ[k] = v + + cmd = "%s -y %s %s %s install %s" % (APT_GET_CMD, dpkg_options, force_yes, check_arg, packages) if default_release: cmd += " -t '%s'" % (default_release,) @@ -292,7 +299,11 @@ def remove(m, pkgspec, cache, purge=False, purge = '--purge' else: purge = '' - cmd = "%s %s -q -y %s %s remove %s" % (APT_ENVVARS, APT_GET_CMD, dpkg_options, purge, packages) + + for (k,v) in APT_ENV_VARS.iteritems(): + os.environ[k] = v + + cmd = "%s -q -y %s %s remove %s" % (APT_GET_CMD, dpkg_options, purge, packages) if m.check_mode: m.exit_json(changed=True) |