#883262 Please start etckeeper.timer via timers.target

Package:
etckeeper
Source:
etckeeper
Submitter:
Michael Biebl
Date:
2017-12-03 01:57:04 UTC
Severity:
normal
#883262#5
Date:
2017-12-01 12:41:04 UTC
From:
To:
Hi,

with the latest update, etckeeper.timer was enabled in
multi-user.target.wants (most likely due to the compat bump).

Please consider using WantedBy=timers.target instead which is supposed
to be used for starting timers.
#883262#10
Date:
2017-12-01 15:56:34 UTC
From:
To:
Probably, yes.

Where should that change be made?

I woudl very much appreciate a patch to do that.

Thanks!

A.
#883262#15
Date:
2017-12-01 17:30:59 UTC
From:
To:
Am 01.12.2017 um 16:56 schrieb Antoine Beaupré:


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.

Michael
#883262#20
Date:
2017-12-01 17:46:49 UTC
From:
To:
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.
#883262#25
Date:
2017-12-02 21:30:16 UTC
From:
To:
Am 01.12.2017 um 18:46 schrieb Antoine Beaupré:

Not quite sure what you mean by that. If the patch is not sufficient, do
you want me to NMU the package?
If you tell me what's unclear, maybe I can help you.

Michael
#883262#30
Date:
2017-12-03 01:54:45 UTC
From:
To:
Feel free to NMU the package - you can also push straight to git on
collab-maint as well - but i was refering to the other two bug reports
here...

a.