Remove env_logger depedency

The env logger has been broken for over a year and is not used by anyone
as far as I know. This removes this option entirely in favor of
Alacritty's built-in logger level selection flags.

2 files changed, 5 insertions, 12 deletions

diff --git a/alacritty/Cargo.toml b/alacritty/Cargo.toml
index 5cd792ad..4033e581 100644
--- a/alacritty/Cargo.toml
+++ b/alacritty/Cargo.toml
@@ -13,7 +13,6 @@ alacritty_terminal = { path = "../alacritty_terminal", default-features = false
 clap = "2"
 log = "0.4"
 time = "0.1.40"
-env_logger = "0.7.1"
 fnv = "1"
 serde = { version = "1", features = ["derive"] }
 serde_yaml = "0.8"
diff --git a/alacritty/src/logging.rs b/alacritty/src/logging.rs
index defa4605..0bc8db9c 100644
--- a/alacritty/src/logging.rs
+++ b/alacritty/src/logging.rs
@@ -42,17 +42,11 @@ pub fn initialize(
 ) -> Result<Option<PathBuf>, log::SetLoggerError> {
     log::set_max_level(options.log_level);
 
-    // Use env_logger if RUST_LOG environment variable is defined. Otherwise,
-    // use the alacritty-only logger.
-    if std::env::var("RUST_LOG").is_ok() {
-        env_logger::try_init()?;
-        Ok(None)
-    } else {
-        let logger = Logger::new(event_proxy);
-        let path = logger.file_path();
-        log::set_boxed_logger(Box::new(logger))?;
-        Ok(path)
-    }
+    let logger = Logger::new(event_proxy);
+    let path = logger.file_path();
+    log::set_boxed_logger(Box::new(logger))?;
+
+    Ok(path)
 }
 
 pub struct Logger {