svn commit: r382595 - head/net/hostapd

[Mathieu Arnold]

+--On 29 mars 2015 14:29:07 +0200 John Marino <freebsd.contact at marino.st>
wrote:
| On 3/29/2015 15:20, Mathieu Arnold wrote:
|> +--On 29 mars 2015 12:15:41 +0000 John Marino <marino at FreeBSD.org> wrote:
|> | +OPTIONS_DEFINE=	PORTS_SSL
|> | +
|> | +PORTS_SSL_DESC= Build with OpenSSL from ports (instead of base system)
|> | +
|> | +.include <bsd.port.options.mk>
|> | +
|> | +.if ${PORT_OPTIONS:MPORTS_SSL}
|> | +.if !defined(WITH_OPENSSL_PORT)
|> | +IGNORE=		the PORTS_SSL option not allowed when WITH_OPENSSL_PORT is
|> | not set +.endif
|> | +CONFIGURE_ARGS+= --with-ssl-dir=${PREFIX}
|> | +.else
|> | +.if defined(WITH_OPENSSL_PORT)
|> | +IGNORE=		the PORTS_SSL option is required when WITH_OPENSSL_PORT is
|> | set +.endif
|> | +.endif
|> 
|> That seems way overly complicated, why not just do:
|> 
|> .if defined(WITH_OPENSSL_PORT)
|> CONFIGURE_ARGS+= --with-ssl-dir=${PREFIX}
|> .endif
| 
| 
| I assume the maintainer wants to preserve the possibility to not include
| openssl capability at all...  (neither port nor base)

Mmmm, well, no, the only thing what you committed does is force the one
building the port is to say that he's using, or not, OpenSSL from ports,
and have it broken if one choose wrong.

-- 
Mathieu Arnold