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.
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
Set log level at runtime #1056
Set log level at runtime #1056
Conversation
@@ -79,7 +79,8 @@ int main(int argc, char *argv[]) | |||
"(http://www.opengeosys.org) " | |||
"Distributed under a Modified BSD License. " | |||
"See accompanying file LICENSE.txt or " | |||
"http://www.opengeosys.org/project/license", | |||
"http://www.opengeosys.org/project/license\n" | |||
"version: " + BaseLib::BuildInfo::git_describe, | |||
' ', | |||
BaseLib::BuildInfo::git_describe); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just checking. Will the version number displayed only once?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, the second BaseLib::BuildInfo::git_describe
is only shown when running ogs --version
.
LOGOG_SHUTDOWN(); | ||
} | ||
|
||
void SetFormatter(std::unique_ptr<logog::Formatter> formatter) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There should be &&
added. Otherwise it will not work if you pass non-temporaries to that method. ✅
But in general I like the idea! |
👍 |
See LogogSetup.h for possible values. tests-target has a log level of warn.
OpenGeoSys development has been moved to GitLab. |
-l
command line arguments to OGS cli and the testrunner. Possible values:none
,error
,warn
,info
,debug
,all
.make tests
log-level is nowwarn
(this suppresses verbose logging in recent Jenkins builds).