As outlined in issue #14639, there are some old and redundant 'running' instructions in two READMEs in /doc. This PR fixes that by instead linking to the official docs about running a node.
Also, changed the download link to point to the download page instead of the releases page.
This is a near-duplicate of PR14755 that was closed (maybe prematurely). Please see that PR for some discussion.
<!--e57a25ab6845829454e8d69fc972939a-->
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.
<!--174a7506f384e20aa4161008e828411d-->
No conflicts as of last run.
Tend to NACK. I prefer having this documentation self-contained and not refer to bitcoin.org. Yes, mentioning the full-node guide might be useful, but it shouldn't be the sole source of information.
@laanwj Thanks for reviewing, good to finally get some comments about this :) It sounds reasonable to keep some minimal self-contained documentation, I updated the commit.
There is also the question about some of text being misplaced. Phrases like "Bitcoin is a free open source peer-to-peer electronic cash system..." and "Bitcoin Core is the original Bitcoin client..." seem to be misplaced at this level in the tree. I think they are an heritage from before QT was merged upstream. These intro/setup texts are still removed in the updated commit.
After this, README_windows.txt contains only duplication, but removal causes Gitian build failure. This should be adressed but maybe in another PR.
Please re-review.
There is also the question about some of text being misplaced. Phrases like "Bitcoin is a free open source peer-to-peer electronic cash system..." and "Bitcoin Core is the original Bitcoin client..."
I agree about that! The project description belongs in the top-level README.md, not there.
In any case especially README_windows.txt is meant for people that have shortly ago downloaded the binaries, they probably already know what bitcoin is.
The contradictory thing about doc/README.md is that it serves two functions:
doc/ (as github displays this text when browsing to the directory)These functions are incompatible, and I think a lot of recent changes to these files are based on different ideas of what should be in there, but the real solution would be to split it up.
9 | +To download Bitcoin Core, visit [bitcoincore.org](https://bitcoincore.org/en/download/). 10 | 11 | Running 12 | --------------------- 13 | -The following are some helpful notes on how to run Bitcoin Core on your native platform. 14 | +The following are some helpful notes on how to run Bitcoin Core on your native platform. (For further details, please refer to https://bitcoin.org/en/full-node)
Surround the URL by angle brackets to make sure it is converted to a link by all markdown parsers :-)
Thanks, I updated the commit.