From 34ee1e126f7bccbb7a797d6855aabaeb027b5328 Mon Sep 17 00:00:00 2001
From: Johannes Rothe <mail@johannes-rothe.de>
Date: Tue, 31 Aug 2021 14:27:56 +0200
Subject: [PATCH] Make ansible-lint happy

---
 roles/dotfiles/tasks/main.yml | 81 ++++++++++++++++++-----------------
 1 file changed, 42 insertions(+), 39 deletions(-)

diff --git a/roles/dotfiles/tasks/main.yml b/roles/dotfiles/tasks/main.yml
index 419c7d5..bb270f6 100644
--- a/roles/dotfiles/tasks/main.yml
+++ b/roles/dotfiles/tasks/main.yml
@@ -31,43 +31,46 @@
 
 - name: Remove old backup
   file:
-    path: '{{home}}/tmp/dotfiles_backup'
+    path: '{{ home }}/tmp/dotfiles_backup'
     state: absent
 - name: Create new backup dir
   file:
-    path: '{{home}}/tmp/dotfiles_backup'
+    path: '{{ home }}/tmp/dotfiles_backup'
     state: directory
+    mode: 0700
 
 - name: Backup stuff
   copy:
     src: '{{ item }}'
-    dest: '{{home}}/tmp/dotfiles_backup'
+    dest: '{{ home }}/tmp/dotfiles_backup'
     remote_src: True  # copy the folder from the host instead of ansible
+    mode: 0600
   ignore_errors: yes
   loop:
     # dirs (without trailing '/' copies the whole dir)
-    - '{{home}}/.config/fish'
+    - '{{ home }}/.config/fish'
     # files
-    - '{{home}}/.tmux.conf'
-    - '{{home}}/.vimrc'
-    - '{{home}}/.ssh/config'
-    - '{{home}}/.config/i3/config'
-    - '{{home}}/.config/i3status/config'
-    - '{{home}}/.config/rofi/config'
-    - '{{home}}/.gitconfig'
-    - '{{home}}/.gnupg/gpg-agent.conf'
+    - '{{ home }}/.tmux.conf'
+    - '{{ home }}/.vimrc'
+    - '{{ home }}/.ssh/config'
+    - '{{ home }}/.config/i3/config'
+    - '{{ home }}/.config/i3status/config'
+    - '{{ home }}/.config/rofi/config'
+    - '{{ home }}/.gitconfig'
+    - '{{ home }}/.gnupg/gpg-agent.conf'
 
 - name: Make sure folders exist
   file:
-    path: '{{item}}'
+    path: '{{ item }}'
     state: directory
+    mode: 0700
   loop:
-    - '{{home}}/.config/i3/'
-    - '{{home}}/.config/i3status/'
-    - '{{home}}/.config/rofi/'
-    - '{{home}}/.ssh/'
-    - '{{home}}/pictures/shots/'
-    - '{{home}}/.gnupg/'
+    - '{{ home }}/.config/i3/'
+    - '{{ home }}/.config/i3status/'
+    - '{{ home }}/.config/rofi/'
+    - '{{ home }}/.ssh/'
+    - '{{ home }}/pictures/shots/'
+    - '{{ home }}/.gnupg/'
 
 - name: Link files
   file:
@@ -76,23 +79,23 @@
     state: link
     force: True
   loop:
-    - src: '{{files_dir}}/vimrc'
-      dest: '{{home}}/.vimrc'
-    - src: '{{files_dir}}/i3_config'
-      dest: '{{home}}/.config/i3/config'
-    - src: '{{files_dir}}/i3status_config'
-      dest: '{{home}}/.config/i3status/config'
-    - src: '{{files_dir}}/rofi_config'
-      dest: '{{home}}/.config/rofi/config'
-    - src: '{{files_dir}}/tmux.conf'
-      dest: '{{home}}/.tmux.conf'
-    - src: '{{files_dir}}/ssh_config'
-      dest: '{{home}}/.ssh/config'
-    - src: '{{files_dir}}/fish'
-      dest: '{{home}}/.config/fish'
-    - src: '{{files_dir}}/wallpaper'
-      dest: '{{home}}/pictures/wallpaper'
-    - src: '{{files_dir}}/gitconfig'
-      dest: '{{home}}/.gitconfig'
-    - src: '{{files_dir}}/gpg-agent.conf'
-      dest: '{{home}}/.gnupg/gpg-agent.conf'
+    - src: '{{ files_dir }}/vimrc'
+      dest: '{{ home }}/.vimrc'
+    - src: '{{ files_dir }}/i3_config'
+      dest: '{{ home }}/.config/i3/config'
+    - src: '{{ files_dir }}/i3status_config'
+      dest: '{{ home }}/.config/i3status/config'
+    - src: '{{ files_dir }}/rofi_config'
+      dest: '{{ home }}/.config/rofi/config'
+    - src: '{{ files_dir }}/tmux.conf'
+      dest: '{{ home }}/.tmux.conf'
+    - src: '{{ files_dir }}/ssh_config'
+      dest: '{{ home }}/.ssh/config'
+    - src: '{{ files_dir }}/fish'
+      dest: '{{ home }}/.config/fish'
+    - src: '{{ files_dir }}/wallpaper'
+      dest: '{{ home }}/pictures/wallpaper'
+    - src: '{{ files_dir }}/gitconfig'
+      dest: '{{ home }}/.gitconfig'
+    - src: '{{ files_dir }}/gpg-agent.conf'
+      dest: '{{ home }}/.gnupg/gpg-agent.conf'