On 2017-12-01 18:30:59, Michael Biebl wrote:
> Am 01.12.2017 um 16:56 schrieb Antoine Beaupré:
>> On 2017-12-01 13:41:04, Michael Biebl wrote:
>>> Package: etckeeper
>>> Version: 1.18.7-1
>>> Severity: normal
>>>
>>> Hi,
>>>
>>> with the latest update, etckeeper.timer was enabled in
>>> multi-user.target.wants (most likely due to the compat bump).
>> 
>> Probably, yes.
>> 
>>> Please consider using WantedBy=timers.target instead which is supposed
>>> to be used for starting timers.
>> 
>> Where should that change be made?
>> 
>> I woudl very much appreciate a patch to do that.
>
>
> diff --git a/systemd/etckeeper.timer b/systemd/etckeeper.timer
> index 364c089..ac25400 100644
> --- a/systemd/etckeeper.timer
> +++ b/systemd/etckeeper.timer
> @@ -7,4 +7,4 @@ OnBootSec=15min
>  OnUnitActiveSec=1d
>
>  [Install]
> -WantedBy=multi-user.target
> +WantedBy=timers.target
>
>
> If you want to be nice to users of unstable, you should probably clean
> up /etc/systemd/system/multi-user.target.wants/etckeeper.timer manually
> on upgrades. I'd leave that up to your discretion.

Honestly, I'd love a little more hand-holding about this. :)

You opened three different issues with the etckeeper systemd support -
could you wrap that up in a little package for me? or even better,
report this upstream?

I certainly won't have time to deal with this until the new year
otherwise.

a.
-- 
Secrecy is the keystone to all tyranny. Not force, but secrecy and
censorship.
                       -  Robert A. Heinlein

Reply via email to