[Docs] Update OpenBSD Build Instructions for OpenBSD 6.2 #11442

pull fanquake wants to merge 1 commits into bitcoin:master from fanquake:openbsd-doc-update changing 1 files +22 −81

fanquake commented at 2:34 AM on October 3, 2017: member
This updates the OpenBSD build docs to reflect building master on a OpenBSD 6.2 VM (using VirtualBox 5.1.28 r117968 on macOS 10.12.6).

Versions of installed packages were:
```
gmake 4.2.1
g++ 4.9.4
git 2.12.2
libevent 2.0.22
libtool 2.4.2
autoconf 2.69p2
automake 1.15p0
python 3.6.0
boost 1.58.0p3
llvm 4.0.0p2
```
The boost package installed via pkg_add now seems to work correctly. So we shouldn't require manual building + patching. I also wasn't required to make adjustments to any resource limits.

Building with g++ and Clang was successful, using:
```
./configure --disable-wallet --with-gui=no CC=egcc CXX=eg++ CPP=ecpp
```
and
```
./configure --disable-wallet --with-gui=no CC=clang CXX=clang++
```
Running make check worked for test/test_bitcoin but test/util/bitcoin-util-test.py failed with:
```
Running test/util/bitcoin-util-test.py...
../test/util/bitcoin-util-test.py
env: python3: No such file or directory
```
So that seems like a configuration issue, Python 3.6 is installed.

Still todo:
- Check if a manual installation of Berkeley DB is required
- Fix running test/util/bitcoin-util-test.py
- Have someone else verify building
cc @laanwj
fanquake added the label Docs and Output on Oct 3, 2017
fanquake added the label Linux/Unix on Oct 3, 2017
MarcoFalke commented at 7:59 AM on October 3, 2017: member

The test failure might be fixed after #11433
in doc/build-openbsd.md:95 in e6c9a4efd5 outdated
```
  48 | @@ -92,22 +49,6 @@ cd db-4.8.30.NC/build_unix/
  49 |  make install # do NOT use -jX, this is broken
  50 |  ```
  51 |  
  52 | -### Resource limits
```
laanwj commented at 8:43 AM on October 3, 2017:

I think we should keep this around just in case people run against these limits, as it's likely very close even if you didn't need it.

fanquake commented at 11:29 PM on October 3, 2017:

Restored.
laanwj commented at 8:45 AM on October 3, 2017: member

Thanks, will test this.
fanquake force-pushed on Oct 3, 2017

in doc/build-openbsd.md:26 in abb52ceee7 outdated

  23 | -See [dependencies.md](dependencies.md) for a complete overview.
  24 | -
  25 | -GCC
  26 | --------
  27 | -
  28 | -The default C++ compiler that comes with OpenBSD 5.9 is g++ 4.2. This version is old (from 2007), and is not able to compile the current version of Bitcoin Core, primarily as it has no C++11 support, but even before there were issues. So here we will be installing a newer compiler:

laanwj commented at 10:27 AM on October 9, 2017:

Not sure why you're removing the explicit description for GCC here - it's intentional that there is a "GCC" and "Clang" header, with the instructions above shared between both. When using clang it's not necessary to install g++ (better not to, probably).

fanquake commented at 11:46 AM on October 9, 2017:

Overzealous deleting. Fixed.

laanwj commented at 11:26 AM on October 9, 2017: member
The boost package installed via pkg_add now seems to work correctly. So we shouldn't require manual building + patching.
[gcc] I can confirm this. Good to know! Have run the make check successfully, as well as test/functional/test_runner.py as well as synced a testnet node part of the way w/ their boost package. This does go from boost 1.64 to 1.58, but I don't think that's so bad compared to the convenience of using a pre-built package.
```
ALL                            | ✓ Passed  | 1655 s (accumulated) 
Runtime: 574 s
```
(zmq test skipped, but we ignore zmq for this guide)

Check if a manual installation of Berkeley DB is required

Last time I checked, there's only a db-4.6.21p5v0 package, which is too old, and doesn't include the dbc++ stuff we need in any case.
fanquake force-pushed on Oct 9, 2017
fanquake force-pushed on Oct 9, 2017
fanquake force-pushed on Oct 9, 2017
fanquake force-pushed on Oct 9, 2017
fanquake commented at 11:52 AM on October 9, 2017: member

@laanwj Thanks for testing. I've fixed the GCC header, as well as a link to build-unix.md

laanwj commented at 12:49 PM on October 9, 2017: member

@fanquake Thanks!

[clang] Tests also passed:

ALL                            | ✓ Passed  | 1708 s (accumulated) 
Runtime: 574 s

Edit: travis issue is the whitespace watchdog

diff --git a/doc/build-openbsd.md b/doc/build-openbsd.md
@@ -29,39 +29,4 @@ The default C++ compiler that comes with OpenBSD 5.9 is g++ 4.2. This version is
+
^---- failure generated from contrib/devtools/lint-whitespace.sh

fanquake force-pushed on Oct 9, 2017
fanquake force-pushed on Oct 9, 2017
fanquake commented at 12:54 PM on October 9, 2017: member

Fixed the whitespace, and one other typo.

in doc/build-openbsd.md:26 in ce70109447 outdated

  27 |  See [dependencies.md](dependencies.md) for a complete overview.
  28 |  
  29 |  GCC
  30 |  -------
  31 | -
  32 | -The default C++ compiler that comes with OpenBSD 5.9 is g++ 4.2. This version is old (from 2007), and is not able to compile the current version of Bitcoin Core, primarily as it has no C++11 support, but even before there were issues. So here we will be installing a newer compiler:

laanwj commented at 2:18 PM on October 9, 2017:

I think this is also still true?

OpenBSD 6.1 (GENERIC.MP) [#20](/bitcoin-bitcoin/20/): Sat Apr  1 13:45:56 MDT 2017
...
user@cocytus user$ g++ --version
g++ (GCC) 4.2.1 20070719

fanquake commented at 12:46 AM on October 12, 2017:

Fixed.

fanquake force-pushed on Oct 12, 2017
laanwj commented at 11:49 AM on October 12, 2017: member

Looks merge-ready to me now, feel free to remove WIP tag (all three points have been addressed in above discussion).
fanquake renamed this:
~~[WIP] [Docs] Update OpenBSD Build Instructions for OpenBSD 6.1~~
[Docs] Update OpenBSD Build Instructions for OpenBSD 6.1
on Oct 12, 2017
fanquake force-pushed on Oct 25, 2017
[Docs] Update OpenBSD Build Instructions for OpenBSD 6.2 9d30f54ef1
fanquake force-pushed on Oct 25, 2017
fanquake renamed this:
~~[Docs] Update OpenBSD Build Instructions for OpenBSD 6.1~~
[Docs] Update OpenBSD Build Instructions for OpenBSD 6.2
on Oct 25, 2017
laanwj merged this on Nov 1, 2017
laanwj closed this on Nov 1, 2017
laanwj referenced this in commit e8f3c88133 on Nov 1, 2017
laanwj referenced this in commit cf18f42899 on Nov 1, 2017
laanwj referenced this in commit efae3663a7 on Dec 30, 2017
codablock referenced this in commit d19c74fd2a on Sep 26, 2019
codablock referenced this in commit 4096433ad9 on Sep 30, 2019
fanquake deleted the branch on Jan 22, 2020
barrystyle referenced this in commit 56fd9ffce3 on Jan 22, 2020
DrahtBot locked this on Feb 15, 2022

Contributors

Labels

Linked (view graph)

#11984 doc: Update OpenBSD build instructions for 6.2 (cont'd)