Skip to content

Commit

Permalink
nodejs6: use bundled openssl
Browse files Browse the repository at this point in the history
This is a preparation step for OpenSSL 1.1 migration. Node.js 6 does not
build with OpenSSL 1.1:

:info:build ../src/node_crypto.h:91:40: error: invalid application of 'sizeof' to an incomplete type 'SSL_CTX' (aka 'ssl_ctx_st')

I switched to bundled openssl instead of using the openssl10 port to
lower the maintenance burden. nodejs6 is EOL'ed upstream and will be
considered to be dropped once nodejs8 works on OS X 10.6, anyway.

Closes: https://trac.macports.org/ticket/58224
  • Loading branch information
Chih-Hsuan Yen authored and ci42 committed May 4, 2019
1 parent 0a231f9 commit dd890d2
Showing 1 changed file with 2 additions and 5 deletions.
7 changes: 2 additions & 5 deletions devel/nodejs6/Portfile
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@ PortGroup compiler_blacklist_versions 1.0

name nodejs6
version 6.17.1
revision 1

categories devel net
platforms darwin
Expand Down Expand Up @@ -33,8 +34,7 @@ distname node-v${version}
depends_build port:pkgconfig

depends_lib port:icu \
port:python27 \
path:lib/libssl.dylib:openssl
port:python27

proc rec_glob {basedir pattern} {
set files [glob -directory $basedir -nocomplain -type f $pattern]
Expand Down Expand Up @@ -78,9 +78,6 @@ if { ${os.platform} eq "darwin" && ${os.major} < 11 } {

configure.args-append --without-npm
configure.args-append --with-intl=system-icu
configure.args-append --shared-openssl
configure.args-append --shared-openssl-includes=${prefix}/include/openssl
configure.args-append --shared-openssl-libpath=${prefix}/lib

# V8 only supports ARM and IA-32 processors
supported_archs i386 x86_64
Expand Down

0 comments on commit dd890d2

Please sign in to comment.