The current FreeBSD Build documentation is a little outdated and underwhelming. This PR intends to keep the build-freebsd.md doc up to date. Here are the main improvements:
Before/Master: render
After/PR: render
57 | 58 | -In order to run the test suite (recommended), you will need to have Python 3 installed: 59 | +### 3. Install Optional Dependencies 60 | + 61 | +#### Wallet Dependencies 62 | +It is not necessary to build wallet functionality to run bitcoind or the GUI. To enable legacy wallets, you must run the provided `BerekelyDB` install script. To enable [descriptor wallets]((https://github.com/bitcoin/bitcoin/blob/master/doc/descriptors.md), `sqlite3` is required. Skip building `BerkeleyDB` if you intend to *exclusively* use descriptor wallets
It is not necessary to build wallet functionality to run bitcoind or the GUI. To enable legacy wallets, you must install `db5`. To enable [descriptor wallets](https://github.com/bitcoin/bitcoin/blob/master/doc/descriptors.md), `sqlite3` is required. Skip `db5` if you intend to *exclusively* use descriptor wallets
40 | 41 | ## Preparation 42 | 43 | -You will need the following dependencies, which can be installed as root via pkg: 44 | +### 1. Install Required Dependencies 45 | +You will need to install all of the required dependencies with `pkg install` as `root` or via `sudo`. If you want to use `sudo`, and you haven't set it up: [use this guide](http://www.freebsdwiki.net/index.php/Sudo%2C_configuring) to setup `sudo` access on FreeBSD.
Install the required dependencies the usual way you [install software on FreeBSD](https://www.freebsd.org/doc/en/books/handbook/ports.html) - either with `pkg` or via the Ports collection. The example commands below use `pkg` which is usually run as `root` or via `sudo`. If you want to use `sudo`, and you haven't set it up: [use this guide](http://www.freebsdwiki.net/index.php/Sudo%2C_configuring) to setup `sudo` access on FreeBSD.
27 | + 28 | +The following dependencies are **optional**: 29 | + 30 | + Library | Purpose | Description 31 | + ---------------------------------------------------------------------------|------------------|---------------------- 32 | + db4.8 | Berkeley DB | Wallet storage (only needed when wallet enabled)
[db5](https://svnweb.freebsd.org/ports/head/databases/db5/) | Berkeley DB | Wallet storage (only needed when wallet enabled)
69 | 70 | ```bash 71 | -pkg install python3 72 | +./contrib/install_db4.sh `pwd` 73 | +export BDB_PREFIX="$PWD/db4" 74 | ```
###### Legacy Wallet Support
`db5` is required to enable support for legacy wallets. Skip if you don't intend to use legacy wallets.
``` bash
pkg install db5
resolved this in 2e8b9a5, thanks for all of your suggestions!
120 | +There are many ways to configure Bitcoin Core, here are a few common examples: 121 | +##### Wallet (BDB + SQlite) Support, No GUI: 122 | +This explicitly enables legacy wallet support and disables the GUI. If `sqlite3` is installed, then descriptor wallet support will be built. 123 | ```bash 124 | ./autogen.sh 125 | ./configure --with-gui=no \
./configure --with-gui=no --with-incompatible-bdb \
BDB_LIBS="-ldb_cxx-5" \
BDB_CFLAGS="-I/usr/local/include/db5" \
111 | @@ -47,13 +112,20 @@ With wallet:
112 | MAKE=gmake
I never needed to append MAKE=gmake to ./configure. Just running gmake later suffices. If that works for you too, then I would suggest to remove it from here.
I don't seem to need MAKE=gmake either anymore—even on FreeBSD 12.1! It always crashed in the configure stage while resolving dependencies without that. Strange, must have changed with some autotools/autoconf update
Oh crap it's back. This seems to happen not initially, but after gmake automatically decides to re-run configuref
config.status: error: in `/home/user/src/bitcoin':
config.status: error: Something went wrong bootstrapping makefile fragments
for automatic dependency tracking. If GNU make was not used, consider
re-running the configure script with MAKE="gmake" (or whatever is
necessary). You can also try re-running configure with the
'--disable-dependency-tracking' option to at least be able to build
the package (albeit without support for automatic dependency tracking).
See `config.log' for more details
gmake: *** [Makefile:718: Makefile] Error 1
Using --disable-dependency-tracking is an extremely bad idea because it essentially removes make's ability to detect what files are affected by changes during development or when pulling from git. This means you can get 'fault ines' where one side of the binary is compiled against a certain data structure and another with the changed one. This has resulted in hours of debugging headaches for me. Dont' do it.
Ah, then I guess MAKE=gmake should stay.
Thanks for the hint!
Excellent work!
I think it is ok to recommend using db5 from the Ports/packages as installing that is a breeze compared to db4.8 which is not in the Ports/packages.
Some other minor suggestions below.
https://travis-ci.org/github/bitcoin/bitcoin/builds/748668149#L457
The subject line of commit hash 6de81d17b6e9b5b413fb354c9ac210a02a35d218 is followed by a non-empty line. Subject lines should always be followed by a blank line.
I think it is ok to squash everything in one commit.
The current FreeBSD Build documentation is a little outdated and underwhelming. This PR updates the doc to
be more informative. It also adds new instructions for building the GUI and adding support for descriptor
wallets.
on vasild's recommendation: it is ok to point the user to download db5 which has an active port package
instead of building db4 from the provided script.
Co-authored-by: Vasil Dimov <vd@freebsd.org>
Seems fine to me.
It does change the document to a different style than the others, e.g. we don't use --- (horizontal separator) after sections anywhere else, but I don't know if that's important.
utACK c17569056105d221053a839d5430df5b3e94f746