add systemd support to gentoo #32

Merged
merged 1 commit into from Apr 2, 2013

Projects

None yet

2 participants

@graysky2 graysky2 merged commit 642ed8d into graysky2:master Apr 2, 2013
@graysky2
Owner

Done... I thought gentoo prided itself on the non-system init stuff? Maybe I'm thinking of udev.... no matter. Merged.

@graysky2
Owner

I will push 5.32.1 once I update the docs. Might wanna wait and then point your ebuild to it.

@graysky2
Owner

I just noticed that your the Makefile-gentoo contains both the cronscript and the systemd timer... they will potentially "conflict" in the sense that users with the cron job will get the update on the hour whereas users with the resync timer will get the sync every 60 min regardless of the time. Is this the behavior you want? Please fork and modify to suit your distro specific wishes. I just updated and will update the manpage according to your decision.

@hasufell

We can just install the cronscript non-executable then. It's better anyway imo, so the user has to explicitly enable it.

Change line 39 to: install -Dm644 ...

@graysky2
Owner

Good idea. Now too sure how to handle this in the manpage... perhaps it's best not to at all. Is there an option of a post_install message you can invoke in the ebuild to the user informing him/her that the default is an INACTIVE cronjob and that if not using systemd, it is HIGHLY recommended to +x it?

@hasufell

I can, but that's handholding. The cronjob is installed in /etc/cron.hourly, thus indicating that a) it's a cronjob and b) it runs every hour...
That should be enough to figure out what it does. If you feel like, add a note to the readme. post_install messages are meant to print non-standard instructions to get software working, especially distro-specific ones, that are not covered by the readme.

@graysky2
Owner

OK... since this only affects gentoo users and since you are a gentoo packager, I will accept whatever you choose to do. I honestly don't know where to add this; the man page is pretty detail rich right now. If you're happy with the current state of the software from a Gentoo POV, I will tag this as 5.32.1 and call it good (just did it). Please update your ebuild accordingly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment