Bug 205672

Summary: [patch] www/node: option to disable SSL
Product: Ports & Packages Reporter: Adam Weinberger <adamw>
Component: Individual Port(s)Assignee: Adam Weinberger <adamw>
Status: Closed FIXED    
Severity: Affects Only Me CC: linpct
Priority: --- Keywords: patch
Version: LatestFlags: bugzilla: maintainer-feedback? (linpct)
Hardware: Any   
OS: Any   
Attachments:
Description Flags
SSL option none

Description Adam Weinberger freebsd_committer freebsd_triage 2015-12-28 20:21:31 UTC
Created attachment 164777 [details]
SSL option

For those of us who use node only for tasks, it'd be nice not to have yet another version of OpenSSL to look after (not to mention build).

Attached patch allows disabling the ssl build (defaults to on so generated package won't change).

What are your thoughts on this?
Comment 1 linpct 2015-12-30 09:01:55 UTC
I think it's great to provide user another option, please keep default on is a good choice.

Thanks.
Comment 2 Adam Weinberger freebsd_committer freebsd_triage 2015-12-30 17:21:15 UTC
I'm having second thoughts about this.

www/npm won't even install, much less run, so this option would probably require warning messages when enabled.
Comment 3 Adam Weinberger freebsd_committer freebsd_triage 2016-03-12 19:19:44 UTC
I'm going to close out this PR. It was a bad idea and it breaks almost everything. Leaving bug #203528 open which is a better idea than this one :-)

Daniel if you feel strongly about this I am happy to commit this, but I suspect it just causes problems at this point.