Skip to content
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

Fixing ubuntu installation description #17651

Closed
wants to merge 1 commit into from

Conversation

dr-orlovsky
Copy link

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

Adding required `libssl-dev` to Ubuntu installation guide section and adding details for BerkleyDB installation
@fanquake fanquake added the Docs label Dec 2, 2019
BerkeleyDB is required for the wallet.
BerkeleyDB is required for the wallet. It can be build by doing

contrib/install_db4.sh \`pwd\`
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@hebasto
Copy link
Member

hebasto commented Dec 2, 2019

@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
Copy link
Author

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
Copy link
Member

fanquake commented Dec 2, 2019

checking for SSL... no

OpenSSL is no longer required after #17265. What version of Bitcoin Core are you building?

@dr-orlovsky
Copy link
Author

dr-orlovsky commented Dec 2, 2019

v0.19.0.1 by checking out the tag:

/usr/local/src/bitcoin$ git status
HEAD detached at v0.19.0.1

@fanquake
Copy link
Member

fanquake commented Dec 2, 2019

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
Copy link
Author

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 dr-orlovsky closed this Dec 2, 2019
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Dec 16, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants