-
Notifications
You must be signed in to change notification settings - Fork 285
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
Implements "Log to the Unix syslog" example #70
Conversation
src/app.md
Outdated
Messages are logged to [UNIX syslog]. Logger backend is initialized | ||
with [`syslog::init`]. | ||
[`syslog::Facility`] indicates type of program submitting log. | ||
[`log::LogLevelFilter`] denotes alowed log verbosity |
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.
"alowed" should be "allowed"
src/app.md
Outdated
with [`syslog::init`]. | ||
[`syslog::Facility`] indicates type of program submitting log. | ||
[`log::LogLevelFilter`] denotes alowed log verbosity | ||
and `Option<&str>` holds optional aplication name. |
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.
"aplication" should be "application"
src/app.md
Outdated
[`log::LogLevelFilter`] denotes alowed log verbosity | ||
and `Option<&str>` holds optional aplication name. | ||
|
||
```rust,no_run |
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.
Why is this no_run
? Is there some reason it would fail if run in CI?
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.
Thanks for your time :)
it is marked as no_run
in order not to clobber the syslog. I guess that most people would prefer not to have noise in their syslog at every testrun.
The example is marked as no_run in order not to clobber the actual on each test run
@dtolnay typos fixed (sorry for that 😞). The example itself is still |
Please see https://github.com/brson/rust-cookbook/issues/61