Bug #3643
closed
- Status changed from New to Code review
- % Done changed from 0 to 100
- Subject changed from Remove openssl.hpp to Remove unused openssl.hpp
- Assignee set to Muktadir Chowdhury
I will reject this. Please rebase your code on top of the latest commit in the master branch, as this issue has been already addressed.
openssl.hpp is only temporary not needed (will be needed after we merge the pending security branch)
- Status changed from Code review to Rejected
Please rebase your code on top of the latest commit in the master branch, as this issue has been already addressed.
Doesn't seem so.
ndn-cxx Build 3478 is a commit on top of the latest commit.
It's failing on OSX 10.11 with error:
+ brew link --force openssl
Error: No such keg: /usr/local/Cellar/openssl
Also, if OpenSSL is necessary, it should be listed as a dependency in docs/INSTALL.rst
.
- Subject changed from Remove unused openssl.hpp to Add installation of openssl into build scripts
- Category set to Build
- Status changed from Rejected to Code review
- Assignee changed from Muktadir Chowdhury to Alex Afanasyev
- Target version set to v0.4
I'm reopening a modified version of this issue. There are a few things that were forgotten:
- actual installation of openssl package with homebrew
- installation of libssl-dev package on ubuntu
- the same dependency installation in NFD build scripts.
- Status changed from Code review to Closed
Also available in: Atom
PDF