-
Notifications
You must be signed in to change notification settings - Fork 50
openssl minimum required version is missing in the doc #267
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
|
||
| # Tested Platforms | ||
| Our CI tests are run cont'nously on the follow'ng platforms and compilers? | ||
| - Linux: CentOs5 gcc 3.4.6, CentOs5.11 gcc 4.1.2, centos 7 gcc 4.8.2 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
...run continuously on the following platforms and compilers:
README.md
Outdated
| For compiling with SSL support: | ||
|
|
||
| - You need to have the openssl installed in your development environment: (i) Add openssl include directory to include directories, (ii) Add openssl library directory to the link directories list (This is the directory named tls. e.g. cpp/Linux_64/hazelcast/lib/tls), (iii) Set the openssl libraries to link. | ||
| - You need to have the openssl (version 1.0.2) installed in your development environment: (i) Add openssl include directory to include directories, (ii) Add openssl library directory to the link directories list (This is the directory named tls. e.g. cpp/Linux_64/hazelcast/lib/tls), (iii) Set the openssl libraries to link. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
...openssl include directory....library directory......directory named tls, e.g., `cpp/Lin.......
README.md
Outdated
|
|
||
| - hazelcast/ | ||
| - lib/ => Contains both shared and static library of hazelcast. | ||
| - lib/tls => Contains the library with tls (ssl) support enabled. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
.....with TLS (SSL) support.....
Serdaro
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reviewed.
Serdaro
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approved.
|
Test PASSed. |
2 similar comments
|
Test PASSed. |
|
Test PASSed. |
Update test environment and openssl version info.