Add bash as a role and use it in role dependencies when bash is needed

8 files changed, 32 insertions, 15 deletions

diff --git a/roles/bash/meta/main.yml b/roles/bash/meta/main.yml
new file mode 100644
index 00000000..f8dac374
--- /dev/null
+++ b/roles/bash/meta/main.yml
@@ -0,0 +1,18 @@
+---
+dependencies: []
+galaxy_info:
+  author: "a14m"
+  description: "Install bash"
+  company: "kartoffeln.work GmbH."
+  license: "MIT"
+  min_ansible_version: "2.18"
+  platforms:
+    - name: "ArchLinux"
+      versions:
+        - "all"
+    - name: "Ubuntu"
+      versions:
+        - "noble"
+    - name: "Debian"
+      versions:
+        - "bookworm"
diff --git a/roles/bash/tasks/main.yml b/roles/bash/tasks/main.yml
new file mode 100644
index 00000000..cf064512
--- /dev/null
+++ b/roles/bash/tasks/main.yml
@@ -0,0 +1,6 @@
+---
+- name: "Ensure bash is installed"
+  become: true
+  ansible.builtin.package:
+    name: "bash"
+    state: "present"
diff --git a/roles/go/meta/main.yml b/roles/go/meta/main.yml
index 17484257..28741fea 100644
--- a/roles/go/meta/main.yml
+++ b/roles/go/meta/main.yml
@@ -1,6 +1,7 @@
 ---
 dependencies:
   - role: "homebrew"
+  - role: "bash"
 galaxy_info:
   author: "a14m"
   description: "Install Go (versions) on Linux distributions"
diff --git a/roles/go/tasks/main.yml b/roles/go/tasks/main.yml
index 73f065f3..99b00be1 100644
--- a/roles/go/tasks/main.yml
+++ b/roles/go/tasks/main.yml
@@ -1,10 +1,4 @@
 ---
-- name: "Ensure bash is installed"
-  become: true
-  ansible.builtin.package:
-    name: "bash"
-    state: "present"
-
 - name: "Add Go to shell profile"
   ansible.builtin.blockinfile:
     path: "{{ ansible_env.HOME }}/.bashrc"
diff --git a/roles/locales/meta/main.yml b/roles/locales/meta/main.yml
index ccac3d31..74b5d5fb 100644
--- a/roles/locales/meta/main.yml
+++ b/roles/locales/meta/main.yml
@@ -1,5 +1,6 @@
 ---
-dependencies: []
+dependencies:
+  - role: "bash"
 galaxy_info:
   author: "a14m"
   description: "Configure the linux distro locales"
diff --git a/roles/locales/tasks/main.yml b/roles/locales/tasks/main.yml
index cb8175bb..e87bf5fb 100644
--- a/roles/locales/tasks/main.yml
+++ b/roles/locales/tasks/main.yml
@@ -2,14 +2,11 @@
 - name: "Include OS-specific variables"
   ansible.builtin.include_vars: "{{ ansible_os_family | lower }}.yml"
 
-- name: "Ensure locales dependencies are installed"
+- name: "Ensure locales package is installed"
   become: true
   ansible.builtin.package:
-    name: "{{ item }}"
+    name: "{{ locales_pkg }}"
     state: "present"
-  with_items:
-    - "{{ locales_pkg }}"
-    - bash
 
 - name: "Read locale.gen file"
   ansible.builtin.slurp:
diff --git a/roles/user/meta/main.yml b/roles/user/meta/main.yml
index dda07e47..996a5fc3 100644
--- a/roles/user/meta/main.yml
+++ b/roles/user/meta/main.yml
@@ -1,5 +1,6 @@
 ---
-dependencies: []
+dependencies:
+  - "bash"
 galaxy_info:
   author: "a14m"
   description: "Configure user and sudo"
diff --git a/roles/user/tasks/main.yml b/roles/user/tasks/main.yml
index 41e0b5b1..72318325 100644
--- a/roles/user/tasks/main.yml
+++ b/roles/user/tasks/main.yml
@@ -1,10 +1,9 @@
 ---
-- name: "Ensure sudo installed"
+- name: "Ensure sudo is installed"
   become: true
   ansible.builtin.package:
     name:
       - "sudo"
-      - "bash"
     state: "present"
 
 - name: "Set sudo group fact"