Skip to content

Conversation

@bnoordhuis
Copy link
Member

Don't include tls_wrap.h in stream_base.cc. It's not used and it breaks
the build when --without-ssl is passed to configure.

Fixes the following build error:

In file included from ../src/tls_wrap.h:5:0,
                 from ../src/stream_base.cc:10:
../src/node_crypto.h:20:25: fatal error: openssl/ssl.h:
No such file or directory
 #include <openssl/ssl.h>

R=@indutny

@jbergstroem
Copy link
Member

LGTM. Tested both with/without --without-ssl.

Don't include tls_wrap.h in stream_base.cc.  It's not used and it breaks
the build when --without-ssl is passed to configure.

Fixes the following build error:

    In file included from ../src/tls_wrap.h:5:0,
                     from ../src/stream_base.cc:10:
    ../src/node_crypto.h:20:25: fatal error: openssl/ssl.h:
    No such file or directory
     #include <openssl/ssl.h>

PR-URL: nodejs#1027
Reviewed-By: Johan Bergström <[email protected]>
@bnoordhuis bnoordhuis closed this Mar 2, 2015
@bnoordhuis bnoordhuis deleted the fix-without-ssl branch March 2, 2015 23:21
@bnoordhuis bnoordhuis merged commit 3d6440c into nodejs:v1.x Mar 2, 2015
@bnoordhuis
Copy link
Member Author

@jbergstroem Thanks for the review, landed in 3d6440c.

@rvagg rvagg mentioned this pull request Mar 2, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants