Fixing ubuntu installation description #17651

pull dr-orlovsky wants to merge 1 commits into bitcoin:master from dr-orlovsky:patch-1 changing 1 files +6 −2

dr-orlovsky commented at 5:18 PM on December 2, 2019: none

Adding required libssl-dev to Ubuntu installation guide section and adding details for BerkleyDB installation

Fixing ubuntu installation description

Adding required `libssl-dev` to Ubuntu installation guide section and adding details for BerkleyDB installation

43adde6155

fanquake added the label Docs on Dec 2, 2019

in doc/build-unix.md:87 in 43adde6155

  84 |      sudo apt-get install libevent-dev libboost-system-dev libboost-filesystem-dev libboost-chrono-dev libboost-test-dev libboost-thread-dev
  85 |  
  86 | -BerkeleyDB is required for the wallet.
  87 | +BerkeleyDB is required for the wallet. It can be build by doing
  88 | +
  89 | +    contrib/install_db4.sh \`pwd\`

hebasto commented at 5:34 PM on December 2, 2019:

This duplicates https://github.com/bitcoin/bitcoin/blob/master/doc/build-unix.md#berkeley-db

hebasto commented at 5:57 PM on December 2, 2019: member

@dr-orlovsky

Welcome!

Adding required libssl-dev to Ubuntu installation guide section...

Why do you think libssl-dev is required? Have you encountered a build issue?
dr-orlovsky commented at 6:44 PM on December 2, 2019: none
Thanks!

Yes, there was a build error on a clear Ubuntu 18.0.4 (bionic) machine during .configure:
```
checking for SSL... no
configure: error: openssl not found.
```
It seems it was shipped previously with Ubuntu and now is not.
fanquake commented at 6:46 PM on December 2, 2019: member

checking for SSL... no

OpenSSL is no longer required after #17265. What version of Bitcoin Core are you building?
dr-orlovsky commented at 6:48 PM on December 2, 2019: none
v0.19.0.1 by checking out the tag:
```
/usr/local/src/bitcoin$ git status
HEAD detached at v0.19.0.1
```
fanquake commented at 6:51 PM on December 2, 2019: member

v0.19.0.1 by checkout out the tag:

In that case you should be looking at the docs in the 0.19 branch. The Linux instructions there still correctly list libssl-dev as required.
dr-orlovsky commented at 6:52 PM on December 2, 2019: none

Right, sorry, read them through github and have not paid attention to the branch/tag, thinking that v0.19.0.1 is a very recent release and there were no dramatic changes after :)
dr-orlovsky closed this on Dec 2, 2019
DrahtBot locked this on Dec 16, 2021

Contributors

Labels