8000 [Upstream] [Core] Add -debugexclude option by lyricidal · Pull Request #123 · PRCYCoin/PRCYCoin · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

[Upstream] [Core] Add -debugexclude option #123

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Sep 28, 2021
Merged

[Upstream] [Core] Add -debugexclude option #123

merged 1 commit into from
Sep 28, 2021

Conversation

lyricidal
Copy link

Implemented on top of:

Backports bitcoin#10123

setting -debug can lead to very noisy debug.logs with subcomponents filling up the log file. See for example https://travis-ci.org/bitcoin/bitcoin/jobs/216767286 where there are hundreds of libevent debug logs.

This commit adds an option to exclude certain components from debug logging. The usage is:

pivxd -debug -debugexclude=<component1> -debugexclude=<component2> ...

This finally reduces spammy logs in the functional tests debug, due to libevent and leveldb (bitcoin#10124 was already included backporting the test suite).

from PIVX-Project/PIVX#1439

@lyricidal
[Core] Add -debugexclude option
A067
3647f27 
to switch off logging for specified components
Backports bitcoin/bitcoin@3bde556
@lyricidal lyricidal requested review from lopeed and anprdev September 28, 2021 20:54
@lopeed lopeed merged commit ba06cbd into develop Sep 28, 2021
@lopeed lopeed deleted the upstream-debug_exclude branch September 28, 2021 20:55
@lyricidal lyricidal added the Upstream Upstream backports/fixes label Sep 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lopeed lopeed lopeed approved these changes

@anprdev anprdev Awaiting requested review from anprdev

Assignees
No one assigned
Labels
Upstream Upstream backports/fixes
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@lyricidal @lopeed
0