qt: Don't log to console by default #13055

pull laanwj wants to merge 1 commits into bitcoin:master from laanwj:2018_04_gui_console changing 1 files +4 −0
  1. laanwj commented at 7:28 AM on April 23, 2018: member

    Default -printtoconsole to false for the GUI. GUI programs should not print to the console unnecessarily. For example, when launched by the window manager, the output might end up in the X session log file, resulting in duplicate logging. On Windows, it is pointless as well because bitcoin-qt isn't a console application.

    This same mechanism is used to set -server to true by default for bitcoind: https://github.com/bitcoin/bitcoin/blob/master/src/bitcoind.cpp#L116

    (fixes #13004)

  2. qt: Don't log to console by default
    Default `-printtoconsole` to false for the GUI. GUI programs should not
    print to the console unnecessarily. For example, when launched by the
    window manager, the output might end up in the X session log file,
    resulting in duplicate logging. On Windows, it is pointless as well
    because bitcoin-qt isn't a console application.
    aee80b0ef9
  3. laanwj added the label GUI on Apr 23, 2018
  4. jonasschnelli commented at 12:04 PM on April 23, 2018: contributor

    I'm not sure if changing the printtoconsole default in #13004 was reasonable. IMO forgetting to pass-in -noprintoconsole during debugging/developing quickly leads to lost data required for debugging.

    utACK aee80b0ef9f9e82fb696ca3a05c06f089926bf04.

  5. laanwj commented at 12:14 PM on April 23, 2018: member

    I'm not sure if changing the printtoconsole default in #13004 was reasonable. IMO forgetting to pass-in -noprintoconsole during debugging/developing quickly leads to lost data required for debugging.

    I think it's reasonable: the default (when not -daemon) is to log to both the log file and the console. After #13004 logging to the console no longer implies not logging the file. That's only done when -nodebuglogfile is provided.

  6. jnewbery commented at 9:50 PM on April 23, 2018: member

    utACK aee80b0ef9f9e82fb696ca3a05c06f089926bf04

  7. MarcoFalke merged this on Apr 23, 2018
  8. MarcoFalke closed this on Apr 23, 2018

  9. MarcoFalke referenced this in commit bdda14d1c0 on Apr 23, 2018
  10. rikur commented at 9:14 AM on June 21, 2018: none

    I stopped seeings logs in bitcoind configured with --no-gui after upgrading to 0.16.1. I'm wondering if this change is to "blame". I don't configure logging in my bitcoin.conf.

    Running macOS.

  11. jnewbery commented at 1:43 PM on June 21, 2018: member

    what is --no-gui? As far as I'm aware that's not valid config for bitcoind or the configure script.

    In any case, the change in this PR does not affect bitcoind, just bitcoin-qt.

  12. MarcoFalke commented at 1:51 PM on June 21, 2018: member

    @rikur Please file an issue and provide all the information if you can reproduce.

  13. jasonbcox referenced this in commit 21a017c20e on Dec 6, 2019
  14. random-zebra referenced this in commit b7c0c0e31d on Apr 15, 2020
  15. jonspock referenced this in commit 3202a6eac6 on Sep 28, 2020
  16. jonspock referenced this in commit 48dcb4608d on Sep 28, 2020
  17. jonspock referenced this in commit d75486c989 on Sep 29, 2020
  18. PastaPastaPasta referenced this in commit ce7b983ca4 on Nov 10, 2020
  19. PastaPastaPasta referenced this in commit d33911769b on Nov 12, 2020
  20. PastaPastaPasta referenced this in commit 4d10091cc8 on Nov 17, 2020
  21. 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 15:15 UTC

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