more package fixes

tasks/main.yml

@@ -33,23 +33,31 @@
 - name: Flush handlers to ensure dependencies are installed
   ansible.builtin.meta: flush_handlers
 
+- name: Add needed MacOS packages
+  when:
+    - ansible_distribution == 'MacOSX'
+  ansible.builtin.set_fact:
+    pkg_sys: "{{ pkg_sys + ['gnu-tar', 'virtualenv'] }}"
+
 - name: Install sys_pkgs list using system package manager
   become: "{{ sys_pkg_become }}"
   ansible.builtin.package:
-    name: "{{ sys_pkgs | unique }}"
+    name: "{{ pkg_sys | unique }}"
     state: present
 
-- name: Linux based OS
-  when: ansible_system == 'Linux'
+- name: Linux specific tasks
+  when:
+    - ansible_system == 'Linux'
   ansible.builtin.include_tasks:
     file: linux.yml
 
-- name: Darwin/macOS based OS
-  when: ansible_distribution == 'MacOSX'
+- name: MacOS specific tasks
+  when:
+    - ansible_distribution == 'MacOSX'
   ansible.builtin.include_tasks:
     file: macos.yml
 
-- name: Install archive_pkgs
+- name: Install pkg_archive
   when:
     - pkg_archive|length > 0
   loop: "{{ pkg_archive }}"
@@ -58,7 +66,7 @@
   ansible.builtin.include_tasks:
     file: "archive/{{ pkg }}.yml"
 
-- name: Install appimages
+- name: Install pkg_appimage
   when:
     - pkg_appimage|length > 0
   loop: "{{ pkg_appimage }}"
@@ -69,8 +77,8 @@
 
 - name: Build and install source packages
   when:
-    - src_pkgs|length > 0
-  loop: "{{ src_pkgs | unique }}"
+    - pkg_src|length > 0
+  loop: "{{ pkg_src | unique }}"
   loop_control:
     loop_var: pkg
   ansible.builtin.include_tasks:
@@ -78,8 +86,8 @@
 
 - name: Install cargo packages
   when:
-    - cargo_pkgs|length > 0
-  loop: "{{ cargo_pkgs | unique }}"
+    - pkg_cargo|length > 0
+  loop: "{{ pkg_cargo | unique }}"
   loop_control:
     loop_var: pkg
   ansible.builtin.include_tasks:
@@ -87,24 +95,25 @@
 
 - name: Install go packages
   when:
-    - go_pkgs|length > 0
-  loop: "{{ go_pkgs | unique }}"
+    - pkg_go|length > 0
+  loop: "{{ pkg_go | unique }}"
   loop_control:
     loop_var: pkg
   ansible.builtin.include_tasks:
     file: go.yml
 
 - name: Install local npm packages
-  loop: "{{ npmpkgs | unique }}"
+  loop: "{{ pkg_npm | unique }}"
   loop_control:
     loop_var: pkg
   ansible.builtin.include_tasks:
     file: npm.yml
 
 - name: Install python pipx packages for user
-  loop: "{{ pipx_pkgs | unique }}"
+  when:
+    - pkg_pipx|length > 0
+  loop: "{{ pkg_pipx | unique }}"
   loop_control:
     loop_var: pkg
-  when: pipx_pkgs|length > 0
   ansible.builtin.include_tasks:
     file: pipx.yml