Skip to content

Conversation

@ghost
Copy link

@ghost ghost commented Feb 1, 2021

I wondered how one could enable debug logging with -debug=<category> for multiple categories. Found out solution is to specify that option multiple times for each wanted category.

This PR documents this behavior and uses the same wording for the same behavior of -debugexclude=<category> to make that also clear and stringent.

I wondered how one could enable debug logging with `-debug=<category>` for multiple categories. Found out solution is to specify that option multiple times for each wanted category.

This PR documents this behavior and uses the same wording for the same behavior of `-debugexclude=<category>` to make that also clear and stringent.
@DrahtBot DrahtBot added the Docs label Feb 1, 2021
@practicalswift
Copy link
Contributor

Concept ACK on clarifying this

@laanwj
Copy link
Member

laanwj commented Feb 2, 2021

ACK 572fd0f

@maflcko
Copy link
Member

maflcko commented Feb 2, 2021

ACK 572fd0f

Would be nice if the cardinality was auto-printed by the argsman, similar to how the python argsman does it

Copy link
Contributor

@theStack theStack left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK 572fd0f
Thanks for clarifying!

@fanquake fanquake merged commit ea96e17 into bitcoin:master Feb 3, 2021
@ghost ghost deleted the patch-1 branch February 3, 2021 07:20
sidhujag pushed a commit to syscoin/syscoin that referenced this pull request Feb 3, 2021
572fd0f doc: More precise -debug and -debugexclude doc (wodry)

Pull request description:

  I wondered how one could enable debug logging with `-debug=<category>` for multiple categories. Found out solution is to specify that option multiple times for each wanted category.

  This PR documents this behavior and uses the same wording for the same behavior of `-debugexclude=<category>` to make that also clear and stringent.

ACKs for top commit:
  laanwj:
    ACK 572fd0f
  MarcoFalke:
    ACK 572fd0f
  theStack:
    ACK 572fd0f

Tree-SHA512: 8d93db37602fd5ff4247e7c11478e55b99c0e3d47eaa2bb901937805d8f2a466b3a198b713b760981c5411576b74c52e2909c46c6d3f2e0e04215fd521b65cf7
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Aug 16, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants