| 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: | Latest | Flags: | bugzilla:
maintainer-feedback?
(linpct) |
||||
| Hardware: | Any | ||||||
| OS: | Any | ||||||
| Attachments: |
|
||||||
I think it's great to provide user another option, please keep default on is a good choice. Thanks. 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. 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. |
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?