Skip to content
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

backport: Merge bitcoin#22626 #6173

Draft
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

vijaydasmp
Copy link

No description provided.

@vijaydasmp vijaydasmp changed the title Bp23 95 2 backport: Aug 5, 2024
@vijaydasmp vijaydasmp changed the title backport: backport: Merge bitcoin#19833, 22626 Aug 5, 2024
fa20f81 Remove txindex migration code (MarcoFalke)
fae8786 doc: Fix validation typo (MarcoFalke)
fab8900 Add missing includes and forward declarations, remove unused ones (MarcoFalke)

Pull request description:

  No supported version of Bitcoin Core used the legacy txindex, so all relevant nodes can be assumed to have upgraded. Thus, there is no need to keep this code any longer.

  As a temporary courtesy, provide a one-time warning on how to free the disk space used by the legacy txindex.

  Fixes bitcoin#22615

ACKs for top commit:
  laanwj:
    Code review ACK fa20f81
  hebasto:
    ACK fa20f81, tested on Linux Mint 20.2 (x86_64).
  Zero-1729:
    crACK fa20f81
  theStack:
    Approach ACK fa20f81

Tree-SHA512: 68aa32d064d1e3932e6e382816a4b5de417bd7e82861fea1ee50660e8c397f4efeb88ae4ed54a8ad1952c3563eb0b8449d7ccf883c353cc4d4dc7e15c53d78e8
@vijaydasmp vijaydasmp changed the title backport: Merge bitcoin#19833, 22626 backport: Merge bitcoin#22626 Aug 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants