no strong opinion

There's sometimes unsaid knowledge in some of these docs, and we can do a much better job of figuring out what is all the information people need to know and organize it effectively. As someone who's worked on the bsd docs, in my head the doc recommends db5 because the doc is meant to guide a user through a dynamic compile using packages they install with their package manager. There's many ways to compile core, and people need to make their own decisions as to how they want to accomplish the task.

The install_db4 script is meant for when there's no easy way to get a bin from a package manager and as a last resort option because you really don't want any static builds. Additionally, There are plans to move away from any bdb support, but that's further out. Descriptor wallets use sqlite.

Debating whether to use db5 or db4 without patches that we need (in depends) isn't so useful when, given the circumstances of this dependency, the recommendation should be to build it using depends.

Now, I would say that we could include a note here to build bdb with depends and then configure with the bdb prefix, but this would end up duplicated across the build docs for other systems where there is no db4 package.

ACK c572eae - have not tested, but looks ok.