Skip to content
Permalink
Browse files

Directly set `name:` to list instead of using per-task variable.

  • Loading branch information...
riccardomurri committed Mar 7, 2019
1 parent b704938 commit 70dd1df3130bd42013d8302c9b8ede055041c17b
@@ -110,28 +110,23 @@

- name: Ensure additional packages are installed
apt:
name: '{{packages}}'
state: '{{ pkg_install_state }}'
allow_unauthenticated: '{{ not insecure_https_downloads|default("no")|bool }}'
vars:
packages:
name:
- apt-transport-https
# these are required by Ansible's `debconf` module
- debconf
- debconf-utils
state: '{{ pkg_install_state }}'
allow_unauthenticated: '{{ not insecure_https_downloads|default("no")|bool }}'


# Apparently some systemd commands (e.g., `hostnamectl` -- possibly others)
# *require* D-Bus but the Debian 8 cloud VM image does not have it installed
# (see issues #392, #393). So, ensure it is installed.
- name: Ensure additional packages are installed (Debian 8)
apt:
name: '{{packages}}'
name: 'dbus'
state: '{{ pkg_install_state }}'
allow_unauthenticated: '{{ not insecure_https_downloads|default("no")|bool }}'
vars:
packages:
- dbus
when: 'is_debian_8_or_later'


@@ -12,10 +12,7 @@

- name: Install commonly needed software (Debian-family)
package:
name: '{{packages}}'
state: '{{ pkg_install_state }}'
vars:
packages:
name:
# compression and archivers
- bzip2
- cpio
@@ -42,3 +39,4 @@
- screen
- tmux
- vim
state: '{{ pkg_install_state }}'
@@ -12,10 +12,7 @@

- name: Install commonly needed software (RHEL-family)
package:
name: "{{ packages }}"
state: '{{ pkg_install_state }}'
vars:
packages:
name:
# compression and archivers
- bzip2
- cpio
@@ -44,3 +41,4 @@
- screen
- tmux
- vim
state: '{{ pkg_install_state }}'

0 comments on commit 70dd1df

Please sign in to comment.
You can’t perform that action at this time.