-
Notifications
You must be signed in to change notification settings - Fork 2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
net-dns/ddclient: fix systemd unit #9773
Conversation
Pull Request assignment Areas affected: ebuilds net-dns/ddclient: @titanofold, @ConiKost, @gentoo/proxy-maint Bugs linked: 665150 In order to force reassignment and/or bug reference scan, please append |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's fix the whole systemd unit (issues from PR #9762) instead of just this one thing.
Also, avoid mkdir -p
as it doesn't complain if something doesn't get made.
8e853f8
to
d5836d3
Compare
Yep. Please have a look, I've created now tmpfiles for systemd, which should be more elegant than those chmod stuff in the unit file itself. Added also |
18e5c0c
to
357df6c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good. And some extra things I didn't notice yesterday in my excitement about someone working on this.
src_install() { | ||
dobin ddclient | ||
|
||
newinitd "${FILESDIR}"/ddclient.initd-r5 ddclient |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Redundant.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed.
|
||
diropts -o ddclient -g ddclient | ||
keepdir /var/cache/ddclient | ||
diropts |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This keepdir section is unnecessary.
Now that we're going to have a fancy tmpfiles.d, it should be created at run time by systemd.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah. I wasn't sure, if keepdir shouldn't be as some sort of a backup. Anyway, removed now again.
diropts | ||
|
||
newinitd "${FILESDIR}"/ddclient.initd-r5 ddclient | ||
systemd_dounit "${FILESDIR}"/ddclient.service |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we want to install the old revision of ddclient.service?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No, fixed.
Closes: https://bugs.gentoo.org/665150 Package-Manager: Portage-2.3.48, Repoman-2.3.10
Pull request CI report Report generated at: 2018-09-03 19:18 UTC Issues already there before the PR (double-check them): |
Closes: https://bugs.gentoo.org/665150
Package-Manager: Portage-2.3.48, Repoman-2.3.10