chore: debian system related deps

labasubagia · Jul 19, 2024 · 4dbd596 · 4dbd596
1 parent 293dd00
commit 4dbd596
Show file tree

Hide file tree

Showing 2 changed files with 21 additions and 9 deletions.
diff --git a/ansible/tasks/dev/python/setup-debian.yml b/ansible/tasks/dev/python/setup-debian.yml
@@ -4,8 +4,12 @@
   vars:
     is_remove: false
 
-    python_pkgs:
+    python_only_install_pkgs:
+      # debian system depends on python3
+      # removing this possibly break the system
       - python3
+
+    python_pkgs:
       - python3-pip
       - virtualenv
       - pipx
@@ -17,7 +21,7 @@
         - name: Setup Python > Install packages
           become: true
           ansible.builtin.apt:
-            name: "{{ python_pkgs }}"
+            name: "{{ python_only_install_pkgs + python_pkgs }}"
             state: present
 
         - name: Setup Python > Check pipx executable exists
@@ -68,3 +72,8 @@
           ansible.builtin.apt:
             name: "{{ python_pkgs }}"
             state: absent
+
+    - name: Setup scrcpy > remove unnecessary deps
+      become: true
+      ansible.builtin.apt:
+        autoremove: true
diff --git a/ansible/tasks/tools/scrcpy/setup-debian.yml b/ansible/tasks/tools/scrcpy/setup-debian.yml
@@ -3,7 +3,10 @@
   vars:
     is_remove: false
 
-    required_pkgs:
+    # ! do not remove it
+    # some packages might required by other system packages
+    # in the end of installation, just run `apt autoremove`
+    scrcpy_build_deps:
       - ffmpeg
       - adb
       - wget
@@ -39,7 +42,7 @@
         - name: Setup scrcpy > Install required deps
           become: true
           ansible.builtin.apt:
-            name: "{{ required_pkgs }}"
+            name: "{{ scrcpy_build_deps }}"
             state: present
 
         - name: Setup scrcpy > Clone repo
@@ -77,10 +80,10 @@
         # - name: Setup scrcpy > Remove required packages
         #   become: true
         #   ansible.builtin.apt:
-        #     name: "{{ required_pkgs }}"
+        #     name: "{{ scrcpy_build_deps }}"
         #     state: absent
 
-        - name: Setup scrcpy > remove unecessary deps
-          become: true
-          ansible.builtin.apt:
-            autoremove: true
+    - name: Setup scrcpy > remove unnecessary deps
+      become: true
+      ansible.builtin.apt:
+        autoremove: true