Bug 255809 - mail/postfix35: Add FLAVORS: ldap_sasl sasl
Summary: mail/postfix35: Add FLAVORS: ldap_sasl sasl
Status: Open
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Some People
Assignee: Juraj Lutter
Keywords: feature, needs-qa
Depends on:
Reported: 2021-05-12 11:22 UTC by Yasuhiro Kimura
Modified: 2021-08-01 19:25 UTC (History)
4 users (show)

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

Patch file (11.57 KB, patch)
2021-05-12 11:22 UTC, Yasuhiro Kimura
no flags Details | Diff
Updated patch file (11.57 KB, patch)
2021-05-31 04:58 UTC, Yasuhiro Kimura
no flags Details | Diff
Updated patch file (11.35 KB, patch)
2021-05-31 14:50 UTC, Yasuhiro Kimura
no flags Details | Diff
Updated patch file (11.59 KB, patch)
2021-08-01 19:04 UTC, Yasuhiro Kimura
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Yasuhiro Kimura 2021-05-12 11:22:19 UTC
Created attachment 224868 [details]
Patch file

* Flavorize this port and add two flavors ldap_sasl and sasl
* Pet portlint and portclippy
* Reformat Makefile with portfmt
Comment 1 Yasuhiro Kimura 2021-05-31 04:58:58 UTC
Created attachment 225396 [details]
Updated patch file

Chase update of ports tree.
Comment 2 Li-Wen Hsu freebsd_committer 2021-05-31 05:48:21 UTC
Comment 3 Stefan Eßer freebsd_committer 2021-05-31 07:38:13 UTC
Not tested, but should be working:

Use "CONFLICTS_INSTALL=" for all packages that cause conflicts independently of the flavor and add "default_CONFLICTS_INSTALL=" et.al. only for the package names for the other FLAVORs potentially created by this port.

This should make simplify maintenance of the CONFLICTS definitions and make it easier to verify their correctness.
Comment 4 Yasuhiro Kimura 2021-05-31 14:50:55 UTC
Created attachment 225409 [details]
Updated patch file

Divide COMMENT into common and FLAVOR dependent.
Comment 5 Yasuhiro Kimura 2021-05-31 14:52:41 UTC
(In reply to Stefan Eßer from comment #3)

Thanks for comment. I updated patch that way.
Comment 6 Jeremy Chadwick 2021-06-30 10:12:21 UTC
Any status on this (and getting a mail/postfix35-sasl stub port added?). It's been almost a month since the last update, and I see no response from the maintainer in this ticket, nor in 255641 or 255647.
Comment 7 Kubilay Kocak freebsd_committer freebsd_triage 2021-07-17 02:50:21 UTC
^Triage: Maintainer timeout, reset assignee
Comment 8 Yasuhiro Kimura 2021-08-01 19:04:02 UTC
Created attachment 226859 [details]
Updated patch file

Rename 'ldap-sasl' flavor to 'ldap' as SASL support for OpenLDAP ports is made permanent with ports 974e13b50148.