refactor(ansible): declare become only when false

ansible/roles/dev_tools/tasks/commands/difft.yaml

@@ -8,12 +8,10 @@
         dest: /tmp/difftastic.tar.gz
 
     - name: install difftastic
-      become: false
       ansible.builtin.unarchive:
         src: /tmp/difftastic.tar.gz
         # NOTE: only contains "difft" file
         dest: "{{ lookup('env','HOME') }}/.local/bin/"
 
     - name: check difft
       ansible.builtin.command: which difft
-      changed_when: false

ansible/roles/dev_tools/tasks/commands/docker.yaml

@@ -1,5 +1,4 @@
 - name: setup docker
-  become: true
   block:
     # apt
     - block:

ansible/roles/dev_tools/tasks/commands/emacs.yaml

@@ -16,11 +16,13 @@
 
     # file is backed-up, but existing symlink is kept
     - name: backup .emacs if already exists
+      become: false
       ansible.builtin.shell: >-
         mv "{{ lookup('env', 'HOME') }}/.emacs" "{{ lookup('env', 'HOME') }}/.emacs.bak"
       when: dot_emacs_exists.stat.exists and not dot_emacs_exists.stat.islnk
 
     - name: create symlink to .emacs(unless symlink does not already exist)
+      become: false
       file:
         src: "{{ (playbook_dir + '/../.emacs') | realpath }}"
         dest: "{{ lookup('env', 'HOME') }}/.emacs"
@@ -34,11 +36,13 @@
       changed_when: false
 
     - name: backup .emacs.d if already exists
+      become: false
       ansible.builtin.shell: >-
         mv "{{ lookup('env', 'HOME') }}/.emacs.d" "{{ lookup('env', 'HOME') }}/.emacs.d.bak"
       when: dot_emacs_d_exists.stat.exists and not dot_emacs_d_exists.stat.islnk
 
     - name: create symlink to .emacs.d(unless symlink does not already exist)
+      become: false
       file:
         src: "{{ (playbook_dir + '/../.emacs.d') | realpath }}"
         dest: "{{ lookup('env', 'HOME') }}/.emacs.d"

ansible/roles/dev_tools/tasks/commands/fish.yaml

@@ -19,11 +19,13 @@
       changed_when: false
 
     - name: backup .config/fish if already exists
+      become: false
       ansible.builtin.shell: >-
         mv "{{ lookup('env', 'HOME') }}/.config/fish" "{{ lookup('env', 'HOME') }}/.config/fish.bak"
       when: fish_config_exists.stat.exists and not fish_config_exists.stat.islnk
 
     - name: create symlink to .config/fish(unless symlink does not already exist)
+      become: false
       file:
         src: "{{ (playbook_dir + '/../.config/fish') | realpath }}"
         dest: "{{ lookup('env', 'HOME') }}/.config/fish"

ansible/roles/dev_tools/tasks/commands/gh.yaml

@@ -1,5 +1,4 @@
 - name: setup gh command
-  become: true
   block:
     - name: get gpg key
       ansible.builtin.get_url:

ansible/roles/init_snap/tasks/main.yaml

@@ -1,5 +1,4 @@
 - name: refresh snap
-  become: true
   command: snap refresh
   when: not is_ci
   changed_when: false

ansible/roles/system_tools/tasks/commands/tmux.yaml

@@ -13,11 +13,13 @@
       changed_when: false
 
     - name: backup .tmux.conf if already exists
+      become: false
       ansible.builtin.shell: >-
         mv "{{ lookup('env', 'HOME') }}/.tmux.conf" "{{ lookup('env', 'HOME') }}/.tmux.conf.bak"
       when: tmux_conf_exists.stat.exists and not tmux_conf_exists.stat.islnk
 
     - name: create symlink to .tmux.conf(unless symlink does not already exist)
+      become: false
       file:
         src: "{{ (playbook_dir + '/../.tmux.conf') | realpath }}"
         dest: "{{ lookup('env', 'HOME') }}/.tmux.conf"
@@ -31,11 +33,13 @@
       changed_when: false
 
     - name: backup .config/tmux if already exists
+      become: false
       ansible.builtin.shell: >-
         mv "{{ lookup('env', 'HOME') }}/.config/tmux" "{{ lookup('env', 'HOME') }}/.config/tmux.bak"
       when: config_tmux_d_exists.stat.exists and not config_tmux_d_exists.stat.islnk
 
     - name: create symlink to .config/tmux(unless symlink does not already exist)
+      become: false
       file:
         src: "{{ (playbook_dir + '/../.config/tmux') | realpath }}"
         dest: "{{ lookup('env', 'HOME') }}/.config/tmux"

ansible/roles/xdg_base_directory/tasks/main.yaml

@@ -16,6 +16,8 @@
   ansible.builtin.shell: echo $PATH
   register: out
 
+- debug: var=out.stdout_lines
+
 - name: check ls
   ansible.builtin.shell: ls -a {{ lookup('env','HOME') }}
   register: out