Re: Questions about postfix/options.mk

[HIRAMATSU Yoshifumi <hiramatsu%jgoamakf.info@localhost>]

Hello,

On Mon, 30 Mar 2015 00:29:32 +0900,
Greg Troxel <gdt%ir.bbn.com@localhost> wrote:
> 
> I am assuming this is discussion for a change that might happen after
> the freeze.   We're too close to the end to change things (when they've
> been stable for years and not obviously an issue).

Yes. Change for bdb is just a clean up. And change for pgsql option, while it fixes
build on Mac OS X, "postfix start" still fails so the sitiation is not much improved.

> > 2. When "pgsql" option is enabled, options.mk adds "-lcrypt -lssl -lcrypto" to
> >    the linker flags. But can we remove the "-lcrypt -lssl -lcrypto"?
> >
> >    The option leads to build fail on Mac OS X 10.10.1 because libcrypt
> >    does not exist.
> >
> >    Removing "-lcrypt -lssl -lcrypto" fixes it, and on NetBSD(7.99.6, checked out on Mar 11)
> >    and SmartOS(Joyent Public Cloud base64 image), postfix is built and run on succeeded
> >    even if those flags are removed.
> 
> I'm assuming you also mean removing the openssl bl3 line.  Did you test
> with tls disabled?

Yes, I have tested both

 - all option enabled
   PKG_OPTIONS.postfix+=   bdb ldap mysql pcre pgsql sasl sqlite tls cdb

 - only pgsql enabled ( tls option disabled )
   PKG_OPTIONS.postfix+=  -tls pgsql

and confirmed "postfix start" succeeded.


Actual diff for options.mk is here.

Index: options.mk
===================================================================
RCS file: /cvsroot/pkgsrc/mail/postfix/options.mk,v
retrieving revision 1.36
diff -b -u -r1.36 options.mk
--- options.mk  10 May 2011 13:38:23 -0000      1.36
+++ options.mk  30 Mar 2015 02:11:54 -0000
@@ -71,10 +71,8 @@
 ###
  .if !empty(PKG_OPTIONS:Mpgsql)
   .  include "../../mk/pgsql.buildlink3.mk"
   -.  include "../../security/openssl/buildlink3.mk"
    CCARGS+=       -DHAS_PGSQL -I${PGSQL_PREFIX}/include/pgsql
    -AUXLIBS+=      -L${PGSQL_PREFIX}/lib -lpq \
    -               -L${BUILDLINK_PREFIX.openssl}/lib -lcrypt -lssl -lcrypto
    +AUXLIBS+=      -L${PGSQL_PREFIX}/lib -lpq
     .endif

 ###

I did not begin to work on bdb option change. I noticed that "-bdb" does not affect
when I test pgsql option.

Kind regards,
HIRAMATSU, Yoshifumi