From d4c33dfe4b46ea6a056edd761a16c9a359fd5bc5 Mon Sep 17 00:00:00 2001
From: Adar Nimrod <nimrod@shore.co.il>
Date: Sun, 13 Dec 2015 09:21:00 +0200
Subject: [PATCH] Bash is needed for login shell (so only for the ldap_login
 role), moved tasks accordingly.

---
 tasks/ldap_login.yml        | 14 ++++++++++++++
 tasks/openbsd_bootstrap.yml |  5 -----
 2 files changed, 14 insertions(+), 5 deletions(-)

diff --git a/tasks/ldap_login.yml b/tasks/ldap_login.yml
index e03b087..30ec8cb 100644
--- a/tasks/ldap_login.yml
+++ b/tasks/ldap_login.yml
@@ -3,3 +3,17 @@
 - name: Assert
   assert:
     that: ansible_os_family in [ 'Debian', 'OpenBSD' ]
+
+- name: apt install
+  when: ansible_pkg_mgr == 'apt'
+  apt:
+    name: bash
+    state: present
+    update_cache: yes
+    cache_valid_time: 3600
+
+- name: pkg install
+  when: ansible_pkg_mgr == 'openbsd_pkg'
+  openbsd_pkg:
+    name: bash
+    state: present
diff --git a/tasks/openbsd_bootstrap.yml b/tasks/openbsd_bootstrap.yml
index 9d14194..7d0ca7b 100644
--- a/tasks/openbsd_bootstrap.yml
+++ b/tasks/openbsd_bootstrap.yml
@@ -23,8 +23,3 @@
     regexp: '^installpath ='
     state: present
     create: yes
-
-- name: Install Bash
-  openbsd_pkg:
-    name: bash
-    state: latest
-- 
GitLab