Bug 223038 - Update port: mail/opendkim Fix rc script chmod
Summary: Update port: mail/opendkim Fix rc script chmod
Status: Closed Overcome By Events
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Daniel Austin
URL:
Keywords:
: 221878 (view as bug list)
Depends on:
Blocks:
 
Reported: 2017-10-15 23:01 UTC by Douglas Thrift
Modified: 2018-10-15 03:19 UTC (History)
7 users (show)

See Also:
bugzilla: maintainer-feedback? (freebsd-ports)


Attachments
mail/opendkim-2.10.3_8.diff (1.32 KB, patch)
2017-10-15 23:01 UTC, Douglas Thrift
no flags Details | Diff
mail/opendkim-2.10.3_8.diff (1.31 KB, patch)
2017-11-05 02:56 UTC, Douglas Thrift
douglas: maintainer-approval? (freebsd-ports)
Details | Diff
mail/opendkim-2.10.3_9 rc script (2.09 KB, patch)
2018-09-19 06:23 UTC, Matthias Pfaller
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Douglas Thrift 2017-10-15 23:01:21 UTC
Created attachment 187204 [details]
mail/opendkim-2.10.3_8.diff

The comments in the milter-opendkim rc script state that milteropendkim_socket is one of the parameters that can instead be set in opendkim.conf(5) config file. However, when running the rc script to start or restart the daemon, the usage for chmod is printed as if it was invoked with incorrect parameters. I tracked this down to milteropendkim_socket being unset and [ -S ... ] returning 0 in sh if its argument is blank and unquoted in the dkim_start_postcmd function. I have provided a patch which avoids this by first checking if milteropendkim_socket is not blank.

portlint: OK (0 fatal errors and 16 warnings found.)
testport: OK (poudriere: 103i386, 103amd64, 111i386, 111amd64)
Comment 1 Douglas Thrift 2017-11-05 02:56:03 UTC
Created attachment 187746 [details]
mail/opendkim-2.10.3_8.diff

Updated patch after whitespace changes from ports r453284.
Comment 2 Jonas Palm 2017-11-09 15:36:17 UTC
*** Bug 221878 has been marked as a duplicate of this bug. ***
Comment 3 mail 2017-12-01 15:07:31 UTC
Hi,

I'm using inet:port@address as "Socket" in opendkim.conf and was also suffering from this problem.

Your patch prevents the chmod from ever happening on my system, so the rc script does not throw any errors anymore.

TnX

Regards,

Ruben
Comment 4 Walter Schwarzenfeld 2018-01-16 15:20:35 UTC
Maintainer feedback, please!
Comment 5 Matthias Pfaller 2018-09-19 06:20:12 UTC
I just noticed the same and started a pr myself... The attached tries to cover a few additional cases where a empty milteropendkim_socket is not handled.
Comment 6 Matthias Pfaller 2018-09-19 06:23:54 UTC
Created attachment 197226 [details]
mail/opendkim-2.10.3_9 rc script
Comment 7 Douglas Thrift 2018-10-15 03:19:23 UTC
It looks like this is fixed by the fix for bug #208056 (ports r481909).