Summary: | mail/opendkim: PID file | ||
---|---|---|---|
Product: | Ports & Packages | Reporter: | Rob LA LAU <freebsd> |
Component: | Individual Port(s) | Assignee: | Chris Rees <crees> |
Status: | Closed FIXED | ||
Severity: | Affects Some People | CC: | crees, dewayne, freebsd-ports |
Priority: | --- | Flags: | bugzilla:
maintainer-feedback?
(freebsd-ports) |
Version: | Latest | ||
Hardware: | Any | ||
OS: | Any |
Description
Rob LA LAU
2021-04-04 14:05:21 UTC
Please take a look at this, which respects PidFile being set if set in the config file: https://reviews.freebsd.org/D29586 I haven't addressed the second comment part, but I think to be honest that /var/run/$name being created is so standard it doesn't really need special mention. It will cause conflicts if PidFile is set to the same as another, so perhaps a check if the pidfile exists might be in order, would be easy to add to dkim_get_pidfile. Looks good. Problem with the /var/run/* directory is that you normally expect settings like those to just work. However, currently, the opendkim.conf file points at /var/run/opendkim, while the rc script creates the directory /var/run/milteropendkim. Cheers, Rob The modified script creates the parent directory of $pidfile; ${pidfile%/*} removes the shortest suffix (i.e. is equivalent to $(dirname $pidfile)). I'm sorry, you are right. I had a look at it while I was doing other things. Yes, this looks good. Have a nice weekend, Rob *** Bug 230685 has been marked as a duplicate of this bug. *** Fixed, so close. Not committed yet! *Now* it's fixed :) Thanks for reporting! A commit in branch main references this bug: URL: https://cgit.FreeBSD.org/ports/commit/?id=0e46cdf18498118d3309812946e1df0245d62cb0 commit 0e46cdf18498118d3309812946e1df0245d62cb0 Author: Chris Rees <crees@FreeBSD.org> AuthorDate: 2022-01-14 17:56:38 +0000 Commit: Chris Rees <crees@FreeBSD.org> CommitDate: 2022-01-14 18:01:28 +0000 mail/opendkim: Respect PidFile setting in opendkim.conf While here, pet rclint a bit, though it doesn't like the profile parts not being in functions PR: ports/254762 Reported by: Rob LA LAU <freebsd@ohreally.nl> Reviewed by: skozlov (a previous iteration) Approved by: maintainer timeout, 9 months Differential Revision: https://reviews.freebsd.org/D29586 mail/opendkim/Makefile | 2 +- mail/opendkim/files/milter-opendkim.in | 98 ++++++++++++++++------------------ 2 files changed, 48 insertions(+), 52 deletions(-) |