Bug 228991

Summary:	mail/postfix: missing cyrus_imapd in REQUIRE
Product:	Ports & Packages	Reporter:	Johan Bergström <bugs>
Component:	Individual Port(s)	Assignee:	Olli Hauer <ohauer>
Status:	Closed Works As Intended
Severity:	Affects Only Me	Flags:	bugzilla: maintainer-feedback? (ohauer)
Priority:	---
Version:	Latest
Hardware:	Any
OS:	Any

Description Johan Bergström 2018-06-13 16:32:25 UTC

Similarly to how postfix can lean on dovecot for smtpd_sasl_type, it can also speak to cyrus (mail/cyrus-imapdXX). Therefore I think it should be added to REQUIRE in the init script.

Comment 1 Olli Hauer freebsd_committer

2018-06-13 19:14:56 UTC

Thanks for your request but I don't think we need a REQUIRE line for cyrus_imapd, see below.

Looking at the REQUIRE and PROVIDE keywords of the following rc scripts postfix should always start after saslauthd and cyrus_imapXX.
But it would be a good idea to extend the dovecot script with "BEFORE: mail" and the cyrus_imapdXX scripts with "PROVIDE: cyrus_imapd imap" so the cyrus imap deamon always starts after saslauthd.

mail/postfix/files/postfix.in
 # PROVIDE: postfix mail
 # REQUIRE: %%REQUIRE%%


mail/dovecot/files/dovecot.in
 # PROVIDE: dovecot
 # REQUIRE: %%REQUIRE%%


mail/cyrus_imapXX/files/imapd.in 
 # PROVIDE: cyrus_imapd
 # REQUIRE: DAEMON
 # BEFORE: mail


security/cyrus-sasl2-saslauthd/files/saslauthd.in
 # PROVIDE: saslauthd
 # REQUIRE: DAEMON
 # BEFORE: mail imap

Comment 2 Johan Bergström 2018-06-13 19:21:10 UTC

(In reply to Olli Hauer from comment #1)

Ah, right. Fair point and +1 to proposed suggestion.