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

Closed
wants to merge 1 commit into from

Commits on Aug 5, 2024

  1. Merge bitcoin#22626: Remove txindex migration code

    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
    laanwj authored and vijaydasmp committed Aug 5, 2024
    Configuration menu
    Copy the full SHA
    783e8ad View commit details
    Browse the repository at this point in the history