Bug 228991 - mail/postfix: missing cyrus_imapd in REQUIRE
Summary: mail/postfix: missing cyrus_imapd in REQUIRE
Status: Closed Works As Intended
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Olli Hauer
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-06-13 16:32 UTC by Johan Bergström
Modified: 2018-06-13 19:42 UTC (History)
0 users

See Also:
bugzilla: maintainer-feedback? (ohauer)

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
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 freebsd_triage 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.