Fix upgrade cancel warnings #10998

pull TheBlueMatt wants to merge 1 commits into bitcoin:master from TheBlueMatt:2017-08-fix-upgrade-cancel-warnings changing 1 files +1 −1
  1. TheBlueMatt commented at 1:29 AM on August 7, 2017: member

    Based on #10919. Without this, if you cancel upgrade, you get a needless error: ERROR: VerifyDB(): *** irrecoverable inconsistency in block data at

  2. TheBlueMatt renamed this:
    2017 08 fix upgrade cancel warnings
    Fix upgrade cancel warnings
    on Aug 7, 2017
  3. fanquake added this to the milestone 0.15.0 on Aug 7, 2017
  4. laanwj commented at 7:06 AM on August 7, 2017: member
  5. Skip remainder of init if upgrade is cancelled 861f9a28bc
  6. TheBlueMatt force-pushed on Aug 7, 2017
  7. TheBlueMatt commented at 3:31 PM on August 7, 2017: member

    Rebased on master.

  8. jnewbery commented at 4:16 PM on August 7, 2017: member

    Seems like reasonable behaviour to me, although I don't see how strLoadError here can ever get used:

                        strLoadError = _("Error upgrading chainstate database");
    

    since we'll never reach https://github.com/bitcoin/bitcoin/blob/318392ca7cda813f4117688ce622fcef252744bc/src/init.cpp#L1543 if fRequestShutdown is set to true. We should probably remove that assignment and improve the logging at https://github.com/bitcoin/bitcoin/blob/318392ca7cda813f4117688ce622fcef252744bc/src/txdb.cpp#L424.

    utACK 861f9a28bcb2b7bcdb602d319a0fd844bf519a1d.

  9. btcdrak commented at 12:44 AM on August 8, 2017: contributor

    utACK 861f9a28bcb2b7bcdb602d319a0fd844bf519a1d

  10. laanwj merged this on Aug 8, 2017
  11. laanwj closed this on Aug 8, 2017

  12. laanwj referenced this in commit 2507fd5556 on Aug 8, 2017
  13. codablock referenced this in commit eeabdb36cd on Sep 27, 2017
  14. codablock referenced this in commit 33bbf1e2be on Oct 12, 2017
  15. codablock referenced this in commit 426e3e0fb5 on Oct 26, 2017
  16. codablock referenced this in commit da5217f43a on Oct 26, 2017
  17. codablock referenced this in commit 83945a6380 on Oct 26, 2017
  18. codablock referenced this in commit 7e3e9e433b on Oct 30, 2017
  19. codablock referenced this in commit bff8b16aa8 on Oct 31, 2017
  20. codablock referenced this in commit ceb0d19b2d on Oct 31, 2017
  21. codablock referenced this in commit 64755903e2 on Oct 31, 2017
  22. codablock referenced this in commit 4cd19913dc on Oct 31, 2017
  23. UdjinM6 referenced this in commit bea88da7aa on Nov 8, 2017
  24. DrahtBot locked this on Sep 8, 2021

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2026-04-24 15:15 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me