Update doc/build-unix.txt #1755

pull osmosis79 wants to merge 1 commits into bitcoin:master from osmosis79:patch-1 changing 1 files +22 −17

osmosis79 commented at 3:21 AM on August 30, 2012: none

how to build on ubuntu 12.04
in doc/build-unix.txt:None in 29ab4042b6 outdated
```
  80 | + sudo apt-get install libboost-all-dev
  81 | +
  82 | +for other Ubuntu & Debian:
  83 | + sudo apt-get install libdb4.8-dev 
  84 | + sudo apt-get install libdb4.8++-dev
  85 | + sudo apt-get install libboost1.37-dev
```
Diapolo commented at 4:59 AM on August 30, 2012:

I'm rather sure it would be better to not use such an old Boost version, but if we use that for the Linux build this just reflects it ;).

osmosis79 commented at 6:20 AM on August 30, 2012:

if thats the case, better update line 59 too. It specifies "Versions used in this release: Boost 1.37". I tested on ubuntu 12.04, so added the name of the newer package accordingly.
laanwj commented at 5:32 AM on August 30, 2012: member

Is this only for the bitcoind build? Or also bitcoin-qt? Because it includes libqrencode et al which are only used by the GUI. In that case you also need to add qt4-qmake libqt4-dev
osmosis79 commented at 6:23 AM on August 30, 2012: none

thats a good point. Yes, strangely bitcoind does require libqrencode...unless the disable flag is specified. I havent built bitcoin-qt.
laanwj commented at 6:28 AM on August 30, 2012: member

You must be wrong. I cannot find any mention of even the word 'qr' in makefile.unix.
osmosis79 commented at 6:28 AM on August 30, 2012: none

scratch that... I meant libminiupnpc-dev. No, libqrencode is not required. Ill go ahead and add the qt libs to the list.
osmosis79 commented at 6:38 AM on August 30, 2012: none

ok, removed libqrencode from build-unix.txt, and left other qt libs out. QR instructions are in readme-qt.rst
gmaxwell commented at 1:48 PM on August 30, 2012: contributor

Hm. Is it wise to add instructions that will have people building binaries which will write wallets the project builds can't read?
laanwj commented at 2:03 PM on August 30, 2012: member

Afaik there is no other way on ubuntu 12.04. It doesn't have the 4.8 bdb / dbcxx package anymore.

Edit: well, it still has libdb4.8-dev, but not libdb4.8++-dev
BitcoinPullTester commented at 10:01 PM on August 30, 2012: none

Automatic sanity-testing: PASSED, see http://jenkins.bluematt.me/pull-tester/9386e61695cab47d8cd7037489c2cc86ce5ff864 for binaries and test log.
osmosis79 commented at 10:17 PM on August 30, 2012: none

I agree, there is no other supported way to build on ubuntu 12.04. For 'wallets the project builds can't read', do we fail sanely?
in doc/build-unix.txt:None in 9386e61695 outdated
```
  66 | +Build requirements:
  67 | + sudo apt-get install build-essential 
  68 | + sudo apt-get install libssl-dev
  69 | +
  70 | +for Ubuntu 12.04:
  71 | + sudo apt-get install libdb5.1-dev 
```
TheBlueMatt commented at 5:03 PM on August 31, 2012:

NACK, if anything, tell them to add the ppa and use db4.*++ from there.

laanwj commented at 10:41 AM on September 1, 2012:

Maybe add a warning that it is recommended to build with db 4.x, because wallet compatibility with the binary builds will be lost when converting to db 5.x. If you don't care about compatibility there is no reason to go through the extra trouble of finding a db4.x++ package.
laanwj commented at 5:26 AM on September 8, 2012: member

This change seems to be accepted apart from the dbc++ version change. If you remove that change for now (and squash into one commit), we can at least merge this.
osmosis79 commented at 8:28 AM on September 11, 2012: none

added info for PPA and db compatibility. squashed changes to single commit. should be good to go.
Cleanup on doc/build-unix.txt a21bdda7ad
TheBlueMatt commented at 6:53 PM on September 11, 2012: member

ACK
laanwj referenced this in commit 8b1eb5687d on Sep 11, 2012
laanwj merged this on Sep 11, 2012
laanwj closed this on Sep 11, 2012
BitcoinPullTester commented at 3:57 AM on September 12, 2012: none

Automatic sanity-testing: FAILED MERGE, see http://jenkins.bluematt.me/pull-tester/224222746698db200d4c47e1611219f25fc5aa71 for test log.

This pull does not merge cleanly onto current master
DrahtBot locked this on Sep 8, 2021

Contributors