Why is getinfo being removed? #11382

issue thequillmagazine opened this issue on September 21, 2017
  1. thequillmagazine commented at 9:45 PM on September 21, 2017: none

    "WARNING: getinfo is deprecated and will be fully removed in 0.16. Projects should transition to using getblockchaininfo, getnetworkinfo, and getwalletinfo before upgrading to 0.16",

    Why? This command gives me a concise summary of the running node. getnetworkinfo is too detailed, getwalletinfo doesn't apply, and getblockchainifo is a redundant command. Why force the use of three commands (that already exist) when one command is doing good? I've never understood this principal.

  2. promag commented at 9:47 PM on September 21, 2017: member

    See #8780 for further details.

  3. jnewbery commented at 9:51 PM on September 21, 2017: member

    From PR #8780:

    Some of the more serious problems with it are:

    • It combines information from different subsystems (wallet, networking, blockchain, ...), so its implementation is messy by definition and has to lock all over the place
    • It is ill-defined and without focus - what information should be added, what shouldn't
    • It behaves differently with and without wallet (and what should it do with multiple wallets)

    It basically still exists as a convenience for people using bitcoin-cli manually

    There is an open PR #10871 to implement getinfo in the bitcoin-cli client. If you find bitcoin-cli getinfo useful I recommend you review that PR to help it get merged.

  4. thequillmagazine commented at 10:00 PM on September 21, 2017: none

    @jnewbery I'm using the bitcoin-cli client, I'll take a look at the code. Thanks for the instant reply!

  5. fanquake added the label Questions and Help on Sep 21, 2017
  6. kangasbros commented at 11:21 AM on September 28, 2017: none

    Why not leave it for legacy purposes, what's the harm? Much code probably uses this command. It can be put to deprecated list or something, but I don't really understand the argumentation of removing it entirely.

  7. sipa commented at 11:25 AM on September 28, 2017: member

    @kangasbros Guaranteeing that its output remains consistent will become increasingly difficult as subsystems (wallet and blockchain, in particular) become more independent.

  8. kangasbros commented at 6:10 AM on September 29, 2017: none

    Maybe leave the fields out that are difficult to keep consistent? I can't believe leaving the whole function outright out is the best decision.

  9. sipa commented at 6:18 AM on September 29, 2017: member

    If we have to break backward compatibility anyway, what is the point in keeping it? Anything relying on it needs to change anyway.

    Also see #10871 to still get the convenience for users who want a quick overview.

  10. fanquake commented at 11:37 AM on October 7, 2017: member

    Closing, this is happening and has been explained sufficiently.

  11. fanquake closed this on Oct 7, 2017

  12. meshcollider referenced this in commit aece8a4637 on Jan 24, 2018
  13. 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-13 21:15 UTC

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