Skip to content

Remove bootstrap.md#5469

Closed
fanquake wants to merge 1 commit intobitcoin:masterfrom
fanquake:remove_bootstrap_md
Closed

Remove bootstrap.md#5469
fanquake wants to merge 1 commit intobitcoin:masterfrom
fanquake:remove_bootstrap_md

Conversation

@fanquake
Copy link
Copy Markdown
Member

See discussion in #5455. Can be tagged for 0.11

@laanwj laanwj added this to the 0.11.0 milestone Dec 13, 2014
@laanwj
Copy link
Copy Markdown
Member

laanwj commented Dec 13, 2014

I didn't do this yet because people are reading the documentation on github, and having doc/remove_bootstrap.md still be there in the default branch is mildly useful as long as 0.9.x is still most common. But tagging for 0.11 is OK. By that time I assume parallel block download is commonly used and bootstrapping is, for most purposes, a thing of the past.

@paveljanik
Copy link
Copy Markdown
Contributor

If it is the thing of the past, we should also remove the code behind it.

We should probably also define some mechanism to deprecate features, announce to-be-deprecated features somehow. This is one thing, accounts the other etc.

@sipa
Copy link
Copy Markdown
Member

sipa commented Dec 13, 2014

Linearize may remain useful for longer, especially with 0.10+'s out-of-order downloading and storage. Perhaps block chain analysis tools will no longer be able to work with the raw block files anymore, but linearize should always give very clean data to work with.

@ghost
Copy link
Copy Markdown

ghost commented Dec 13, 2014

I second @sipa's comments. The reports of bootstrap's death have been greatly exaggerated. Removing its documentation, even the code itself, appears premature.

@paveljanik
Copy link
Copy Markdown
Contributor

Don't get me wrong. I do not agree with the removal. It can be useful in many cases.

@sipa
Copy link
Copy Markdown
Member

sipa commented Dec 13, 2014

My only point is that there is a distinction between removing the concept/recommendation of "using linearize to create a linear block data file" and "using said file to speed up initial download".

@makhmet
Copy link
Copy Markdown

makhmet commented Dec 13, 2014

Дорогой друг,что я удалил,Это не умышленно просто у меня мало опыта.Я даже
не понимаю,что не правильно сделал.Больше нечего не буду нажимать.С
ИСКРЕННИМ УВЖЕНИЕМ МАХМЕТ!!!

2014-12-14 1:24 GMT+03:00 paveljanik notifications@github.com:

Don't get me wrong. I do not agree with the removal. It can be useful in
many cases.


Reply to this email directly or view it on GitHub
#5469 (comment).

@paveljanik
Copy link
Copy Markdown
Contributor

@sipa Yes. I was talking about the import from the .dat file (https://github.com/bitcoin/bitcoin/blob/master/src/init.cpp#L429) in this case.

@laanwj
Copy link
Copy Markdown
Member

laanwj commented Dec 15, 2014

No one is arguing for the linearize scripts to be removed. I'd certainly be opposed. But there is no use for a guide encouraging using the bootstrapping torent for normal users anymore.

Importing from a .dat can also still be useful in exceptional circumstances, ie testing and troubleshooting. NACK on removing that.

@laanwj
Copy link
Copy Markdown
Member

laanwj commented Jan 16, 2015

This is part of @theuni's trivial tree (https://github.com/theuni/bitcoin-trivial/commits/trivial-rebase) and will be merged via that route, so closing this pull.

@laanwj laanwj closed this Jan 16, 2015
@fanquake fanquake deleted the remove_bootstrap_md branch January 22, 2020 11:28
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Feb 15, 2022
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