Skip to content

Conversation

jarolrod
Copy link
Member

@jarolrod jarolrod commented Jul 6, 2022

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.

@DrahtBot DrahtBot added the Docs label Jul 6, 2022
Copy link
Member

@fanquake fanquake left a 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.
@jarolrod
Copy link
Member Author

jarolrod commented Jul 7, 2022

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

Copy link
Contributor

@aureleoules aureleoules left a 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.

Copy link
Member

@fanquake fanquake left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK d3e9a1c

@maflcko maflcko merged commit 9945737 into bitcoin:master Jul 8, 2022
sidhujag pushed a commit to syscoin/syscoin that referenced this pull request Jul 8, 2022
@bitcoin bitcoin locked and limited conversation to collaborators Jul 8, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants