Skip to content

Commit 4beb054

Browse files
committed
use tracing
1 parent 17bef09 commit 4beb054

File tree

5 files changed

+25
-29
lines changed

5 files changed

+25
-29
lines changed

Cargo.toml

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -38,9 +38,10 @@ path = "src/main.rs"
3838
cast = "0.3"
3939
clap = "2.33"
4040
clap_conf = "0.1.5"
41-
env_logger = "0.9"
4241
inflections = "1.1"
43-
log = { version = "~0.4", features = ["std"] }
42+
tracing = { version = "0.1", default-features = false, features = ["std"] }
43+
tracing-subscriber = { version = "0.3", default-features = false, features = ["std", "ansi", "fmt", "env-filter", "tracing-log"] }
44+
4445
quote = "1.0"
4546
proc-macro2 = "1.0"
4647
anyhow = "1.0"

src/generate/device.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ use crate::svd::Device;
22
use proc_macro2::{Ident, Span, TokenStream};
33
use quote::{quote, ToTokens};
44

5-
use log::debug;
5+
use tracing::debug;
66
use std::fs::File;
77
use std::io::Write;
88

src/generate/peripheral.rs

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,8 @@ use crate::svd::{
66
Cluster, ClusterInfo, DeriveFrom, DimElement, Peripheral, Register, RegisterCluster,
77
RegisterProperties,
88
};
9-
use log::{debug, trace, warn};
9+
10+
use tracing::{debug, trace, warn};
1011
use proc_macro2::{Ident, Punct, Spacing, Span, TokenStream};
1112
use quote::{quote, ToTokens};
1213
use syn::{parse_str, Token};

src/generate/register.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ use crate::svd::{
44
};
55
use cast::u64;
66
use core::u64;
7-
use log::warn;
7+
use tracing::warn;
88
use proc_macro2::{Ident, Punct, Spacing, Span, TokenStream};
99
use quote::{quote, ToTokens};
1010

src/main.rs

Lines changed: 18 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
11
#![recursion_limit = "128"]
22

3-
use log::{error, info};
43
use std::path::PathBuf;
54
use svd_parser::svd;
5+
use tracing::{error, info};
66

77
mod generate;
88
mod util;
@@ -90,10 +90,9 @@ fn run() -> Result<()> {
9090
.short("l")
9191
.help(&format!(
9292
"Choose which messages to log (overrides {})",
93-
env_logger::DEFAULT_FILTER_ENV
93+
tracing_subscriber::EnvFilter::DEFAULT_ENV
9494
))
9595
.takes_value(true)
96-
.possible_values(&["off", "error", "warn", "info", "debug", "trace"]),
9796
)
9897
.version(concat!(
9998
env!("CARGO_PKG_VERSION"),
@@ -193,29 +192,24 @@ fn run() -> Result<()> {
193192
}
194193

195194
fn setup_logging<'a>(getter: &'a impl clap_conf::Getter<'a, String>) {
196-
// * Log at info by default.
195+
// * Log at `info` by default.
197196
// * Allow users the option of setting complex logging filters using
198-
// env_logger's `RUST_LOG` environment variable.
199-
// * Override both of those if the logging level is set via the `--log`
200-
// command line argument.
201-
let env = env_logger::Env::default().filter_or(env_logger::DEFAULT_FILTER_ENV, "info");
202-
let mut builder = env_logger::Builder::from_env(env);
203-
builder.format_timestamp(None);
204-
205-
let log_lvl_from_env = std::env::var_os(env_logger::DEFAULT_FILTER_ENV).is_some();
206-
207-
if log_lvl_from_env {
208-
log::set_max_level(log::LevelFilter::Trace);
209-
} else {
210-
let level = match getter.grab().arg("log_level").conf("log_level").done() {
211-
Some(lvl) => lvl.parse().unwrap(),
212-
None => log::LevelFilter::Info,
213-
};
214-
log::set_max_level(level);
215-
builder.filter_level(level);
216-
}
197+
// the `RUST_LOG` environment variable.
198+
// * Override both of those if the logging level is set via
199+
// the `log_level` config setting.
200+
201+
let filter = match getter.grab().arg("log_level").conf("log_level").done() {
202+
Some(lvl) => tracing_subscriber::EnvFilter::from(lvl),
203+
None => tracing_subscriber::EnvFilter::from_default_env()
204+
.add_directive(tracing::Level::INFO.into()),
205+
};
217206

218-
builder.init();
207+
tracing_subscriber::fmt()
208+
.without_time()
209+
.with_target(true)
210+
.with_env_filter(filter)
211+
.with_ansi(true)
212+
.init();
219213
}
220214

221215
fn main() {

0 commit comments

Comments
 (0)