Bug 198857

Summary: [patch] mail/postfix does not properly build openldap with SASL
Product: Ports & Packages Reporter: redlamb19
Component: Individual Port(s)Assignee: Olli Hauer <ohauer>
Status: Closed FIXED    
Severity: Affects Many People CC: ohauer, redlamb19
Priority: --- Keywords: patch
Version: LatestFlags: bugzilla: maintainer-feedback? (sahil)
Hardware: amd64   
OS: Any   
Attachments:
Description Flags
Makefile patch for mail/postfix to honor LDAP_SASL option none

Description redlamb19 2015-03-23 23:04:43 UTC
Created attachment 154741 [details]
Makefile patch for mail/postfix to honor LDAP_SASL option

When building mail/postfix with LDAP_SASL option set, openldap is not properly configured (and built) with SASL support. While installation succeeds and postfix is linked against SASL libs, the openldap binaries are not.

Running 10.1-RELEASE

Fix: Apply patch which sets WANT_OPENLDAP_SASL option for openldap port when postfix LDAP_SASL option is set.

# cd /usr/ports
# patch -p1 < postfix_ldap_sasl.patch
Comment 1 redlamb19 2015-04-04 15:36:55 UTC
I'm sure everyone is able to figure this out, but I wanted to fix an error I had in the command to patch the Makefile.

# cd /usr/ports
# patch < postfix_ldap_sasl.patch (remove the -p1 flag)
Comment 2 commit-hook freebsd_committer freebsd_triage 2015-05-10 15:24:53 UTC
A commit references this bug:

Author: ohauer
Date: Sun May 10 15:24:33 UTC 2015
New revision: 385964
URL: https://svnweb.freebsd.org/changeset/ports/385964

Log:
  - update to version 3.0.1
  - rename OPTIONS to match default name of most ports
    - SASL2 -> SASL
    - OPENLDAP -> LDAP
  - install main.cf and master.cf with the @sample macro
  - rework pkg-install
  - fix wrong permission for /var/db/postfix [1]
  - sets WANT_OPENLDAP_SASL option for openldap port when
    postfix LDAP_SASL option is set [2]
  - make usage of new ${opt}_DEPENDS notation

  Release 3.0.1 or 3.0.2 is now a strong candidate to become the new
  default mail/postfix port (missing components are VDA and SPF).

  Changelog:
  20150211
  	Cleanup: strncasecmp_utf8() streamlining. Files: util/stringops.h,
  	util/allascii.c, util/strcasecmp_utf8.c.

  20150214
  	Bugfix (introduced: Postfix 3.0): missing #ifdef USE_TLS
  	inside #ifdef USE_SASL_AUTH.  Viktor Dukhovni. File:
  	smtpd/smtpd.c.

  20150217
  	Cleanup: missing <string.h> include. File: util/allascii.c.

  20150221
  	Bugfix (introduced: Postfix 3.0): don't append '.' to the
  	DNS resource record value, when converting TXT records to
  	the string form that is used used by xxx_dns_reply_filter.
  	File: dns/dns_strrecord.c.

  20150313
  	Documentation: incorrect Postfix version number for
  	postscreen_dnsbl_timeout. Quanah Gibson-Mount. File:
  	postscreen/postscreen.c.

  20150324
  	Bugfix (introduced: Postfix 2.6): sender_dependent_relayhost_maps
  	ignored the relayhost setting in the case of a DUNNO lookup
  	result.  It would use the recipient domain instead.  Viktor
  	Dukhovni. Wietse took the pieces of code that enforce the
  	precedence of a sender-dependent relayhost, the global
  	relayhost, and the recipient domain, and put that code
  	together in once place so that it is easier to maintain.
  	File: trivial-rewrite/resolve.c.

  20150328
  	Bugfix (introduced: Postfix 1.1.0): post-install expanded
  	macros in parameter values when trying to detect parameter
  	overrides, causing unnecessary main.cf updates during Postfix
  	start-up. Julian Reich, Viktor Dukhovni, and Wietse.  File:
  	conf/post-install.

  20150330
  	Bitrot: prepare for future changes in OpenSSL API. Viktor
  	Dukhovni. File: tls_dane.c.

  PR:	198215 [1]
  PR:	198857 [2]

Changes:
  head/mail/postfix-current/Makefile
  head/mail/postfix-current/distinfo
  head/mail/postfix-current/files/pkg-install.in
  head/mail/postfix-current/pkg-plist
Comment 3 Olli Hauer freebsd_committer freebsd_triage 2015-05-10 15:33:15 UTC
Thanks for the fix!