Re: CVS commit: pkgsrc/mk/platform

[Jonathan Perkin <jperkin%joyent.com@localhost>]

* On 2013-08-27 at 06:43 BST, Richard PALO wrote:

> Module Name:  pkgsrc
> Committed By: richard
> Date:         Tue Aug 27 05:43:20 UTC 2013
> 
> Modified Files:
>       pkgsrc/mk/platform: SunOS.mk
> 
> Log Message:
> fix openssl builtin support, at least for solaris

Please don't make such changes to mk without discussion.  This isn't a
'fix', it's changing default behaviour, and the reason I added it in
the first place was to ensure that by default users got accelerated
AES-NI support.

If you want to use builtin OpenSSL, you can simply add
PREFER_NATIVE=openssl or PREFER.openssl=native to your mk.conf, that's
the reason these variables are set with '?='.

Unless there is a specific reason for doing this, please change it
back, else let's discuss which setting makes the most sense and ensure
that is documented alongside the setting rather than simply commenting
it out.

Thanks,

-- 
Jonathan Perkin  -  Joyent, Inc.  -  www.joyent.com