-
Notifications
You must be signed in to change notification settings - Fork 37.7k
doc: update for NetBSD 9.2, add GUI Build Instructions #25549
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
concept ACK
NetBSD doc has not seen any meaningful contribution since 2018. This PR intends to update the docs so that one can successfully build on the latest NetBSD release. It also adds dependency information and instructions to build the GUI.
updated from d43fc74 to d3e9a1c (pr25549.01 -> pr25549.02, diff) Addressed @fanquake comments: thanks for the review, I feel it is now more concise and to the point |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ACK d3e9a1c.
I was able to compile Bitcoin Core (with GUI) on a NetBSD 9.2 virtual machine using jarolrod's guide and followed the build guide without any issue.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ACK d3e9a1c
For reviewer: as I suppose few have a NetBSD system available, I wrote a guide to setup a VM for testing purposes.
This attempts to update the NetBSD docs so one can successfully build on the latest release. It also adds instructions to build the GUI.
Additionally, it includes a note and an example on how one could update the gcc version bundled with NetBSD 9.2 and prior to be able to actually compile. This note can be updated with the release of NetBSD 10, as it will package an acceptable gcc version.
Master: render
PR: render
Related to #20610, but reworked.