Mirrored_Repos/veloren
1
0
Fork 0
mirror of https://gitlab.com/veloren/veloren.git synced 2024-08-30 18:12:32 +00:00
Code Issues Packages Projects Releases Wiki Activity

change(log): only print out log file errors, no panic

Browse Source
This commit is contained in:
Songtronix 2019-12-20 15:59:50 +01:00
parent b03e4ac261
commit f070c139e9
2 changed files with 11 additions and 18 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

21
voxygen/src/logging.rs
View File

@ -40,17 +40,12 @@ pub fn init(
))
});
// Based on settings ignore errors incase log file can't be created
if !settings.log.ignore_errors {
file_cfg = file_cfg.chain(
fern::log_file(&format!("voxygen-{}.log", time.format("%Y-%m-%d-%H")))
.expect("Failed to create log file!"),
);
} else {
if let Ok(log_file) = fern::log_file(&format!("voxygen-{}.log", time.format("%Y-%m-%d-%H")))
{
file_cfg = file_cfg.chain(log_file);
}
// Try to create the log file.
// Incase of it failing we simply print it out to the console.
let mut log_file_created = Ok(());
match fern::log_file(&format!("voxygen-{}.log", time.format("%Y-%m-%d-%H"))) {
Ok(log_file) => file_cfg = file_cfg.chain(log_file),
Err(e) => log_file_created = Err(e),
}
let stdout_cfg = fern::Dispatch::new()
@ -70,4 +65,8 @@ pub fn init(
base.chain(stdout_cfg)
.apply()
.expect("Failed to setup logging!");
if let Err(e) = log_file_created {
log::error!("Failed to create log file! {}", e);
}
}

8
voxygen/src/settings.rs
View File

@ -158,17 +158,11 @@ pub struct Log {
// Whether to create a log file or not.
// Default is to create one.
pub log_to_file: bool,
// Should we ignore errors if we are unable to create the log file
// Default is to panic if log file can't be created.
pub ignore_errors: bool,
}
impl Default for Log {
fn default() -> Self {
Self {
log_to_file: true,
ignore_errors: false,
}
Self { log_to_file: true }
}
}