Merge pull request vlcty#9 from stuehmer/patch-1

Avoid setting User attribute in 'user' scope

Author: vlcty

README.md

@@ -23,7 +23,7 @@ timers:
 
 That's all the magic.
 
-### Existing variables
+### Existing variables per timer
 
 | Variable | Required |  Default value / Explanation |
 |----------|----------|------------------------------|
@@ -58,6 +58,24 @@ More about timers: https://www.freedesktop.org/software/systemd/man/systemd.time
 
 More about timespans: https://www.freedesktop.org/software/systemd/man/systemd.time.html
 
+### Existing variables globally, for the role
+| Variable | Required | Default value / Explanation                                                                                                                            |
+|----------|----------|--------------------------------------------------------------------------------------------------------------------------------------------------------|
+| systemd_scope | no       | Create system or user units. Default: `system`.                                                                                                        |
+| systemd_base_path | no    | Where to generate the systemd unit files. Set this to e.g. `~/.config/systemd/user` when using *systemd_scope*=`user`. Default: `/etc/systemd/system`. |
+
+You can create user timers for non-root services in combination with `become_user: '{{ my_user }}'`. Example:
+```
+systemd_base_path: ~/.config/systemd/user
+systemd_scope: user
+timers:
+  timer-one:
+    timer_command: ...
+    timer_OnCalendar: ...
+    timer_user: '{{ my_user }}'
+  ...
+```
+
 ## Working with shell redirection
 
 Shell redirection does not work out of the box. You have to work around that by calling `sh` or `bash`.   

templates/service.j2

@@ -14,7 +14,7 @@ EnvironmentFile={{ item.value.timer_envfile }}
 ExecStartPre={{ item.value.timer_precommand }}
 {% endif %}
 ExecStart={{ item.value.timer_command }}
-{% if item.value.timer_user is defined %}
+{% if item.value.timer_user is defined and (systemd_scope is not defined or systemd_scope != 'user') %}
 User={{ item.value.timer_user }}
 {% endif %}
 {% if item.value.timer_workingdir is defined %}