search

leamas / ddupdate

Update DNS Data for Dynamic IP Addresses
MIT License
40 stars 28 forks source link

systemd units are now user units #30

Closed aerusso closed 5 years ago

aerusso commented 5 years ago

As of b49006e, systemd units are not in /lib/systemd/system, instead they are in /usr/lib/systemd/user. This MR anticipates the merge of that commit into the debian branch. Moreover, with this change, debian builds after merging master into it.

aerusso commented 5 years ago

@leamas I gave master a test, and this is all I needed to get it to work!

I do think that removing system-level scheduling in favor of user-level scheduling is less than ideal (because it requires keeping another scheduling process around for each daemon user). It's easy enough to set back up, so I cannot complain.

leamas commented 5 years ago

hm... I have actually made the exact same patches already (but not pushed). If we both do the same thing it cant possibly be wrong, right ?

do think that removing system-level scheduling in favor of user-level scheduling is less than ideal

I still think it's the right thing. Being able to configure and run as ordinary user is a huge benefit.

I seriously doubt the runtime impact. A timer-based service doesn't really stay around, I guess. Havn't looked into it, to be honest.

Thanks for testing! New release under way

leamas commented 5 years ago

Clöosing without merge since the changes are already in place.