Replace slog with log

This commit replaces the `slog` family of crates used by `wasm-pack`
with the `log` crate plus `env_logger`. This also means that by default
`wasm-pack` also won't create a `wasm-pack.log` file in the current
directory. Enabling logging will now be done through
`RUST_LOG=wasm_pack` instead of `-v` flags.

Closes rustwasm#425

Cargo.toml

@@ -15,21 +15,20 @@ cargo_metadata = "0.6.0"
 console = "0.6.1"
 curl = "0.4.13"
 dirs = "1.0.4"
+env_logger = { version = "0.5.13", default-features = false }
 failure = "0.1.2"
 flate2 = "1.0.2"
 hex = "0.3"
 human-panic = "1.0.1"
 indicatif = "0.9.0"
 lazy_static = "1.1.0"
+log = "0.4.6"
 openssl = { version = '0.10.11', optional = true }
 parking_lot = "0.6"
 serde = "1.0.74"
 serde_derive = "1.0.74"
 serde_json = "1.0.26"
 siphasher = "0.2.3"
-slog = "2.3"
-slog-term = "2.4"
-slog-async = "2.3"
 structopt = "0.2"
 tar = "0.4.16"
 toml = "0.4"

src/bindgen.rs

@@ -5,8 +5,9 @@ use child;
 use emoji;
 use failure::{self, ResultExt};
 use manifest::CrateData;
+use log::debug;
+use log::{info, warn};
 use progressbar::Step;
-use slog::Logger;
 use std::fs;
 use std::path::{Path, PathBuf};
 use std::process::Command;
@@ -25,20 +26,15 @@ pub fn install_wasm_bindgen(
     version: &str,
     install_permitted: bool,
     step: &Step,
-    log: &Logger,
 ) -> Result<Download, failure::Error> {
     // If `wasm-bindgen` is installed globally and it has the right version, use
     // that. Assume that other tools are installed next to it.
     //
     // This situation can arise if `wasm-bindgen` is already installed via
     // `cargo install`, for example.
     if let Ok(path) = which("wasm-bindgen") {
-        debug!(
-            log,
-            "found global wasm-bindgen binary at: {}",
-            path.display()
-        );
-        if wasm_bindgen_version_check(&path, version, log) {
+        debug!("found global wasm-bindgen binary at: {}", path.display());
+        if wasm_bindgen_version_check(&path, version) {
             return Ok(Download::at(path.parent().unwrap()));
         }
     }
@@ -51,14 +47,13 @@ pub fn install_wasm_bindgen(
         Ok(dl) => return Ok(dl),
         Err(e) => {
             warn!(
-                log,
                 "could not download pre-built `wasm-bindgen`: {}. Falling back to `cargo install`.",
                 e
             );
         }
     }
 
-    cargo_install_wasm_bindgen(log, &cache, version, install_permitted)
+    cargo_install_wasm_bindgen(&cache, version, install_permitted)
 }
 
 /// Downloads a precompiled copy of wasm-bindgen, if available.
@@ -101,7 +96,6 @@ fn prebuilt_url(version: &str) -> Option<String> {
 /// Use `cargo install` to install the `wasm-bindgen` CLI locally into the given
 /// crate.
 pub fn cargo_install_wasm_bindgen(
-    logger: &Logger,
     cache: &Cache,
     version: &str,
     install_permitted: bool,
@@ -131,7 +125,7 @@ pub fn cargo_install_wasm_bindgen(
         .arg("--root")
         .arg(&tmp);
 
-    child::run(logger, cmd, "cargo install").context("Installing wasm-bindgen with cargo")?;
+    child::run(cmd, "cargo install").context("Installing wasm-bindgen with cargo")?;
 
     fs::rename(&tmp, &destination)?;
     Ok(Download::at(&destination))
@@ -147,7 +141,6 @@ pub fn wasm_bindgen_build(
     target: &str,
     debug: bool,
     step: &Step,
-    log: &Logger,
 ) -> Result<(), failure::Error> {
     let msg = format!("{}Running WASM-bindgen...", emoji::RUNNER);
     PBAR.step(step, &msg);
@@ -185,26 +178,24 @@ pub fn wasm_bindgen_build(
         cmd.arg("--debug");
     }
 
-    child::run(log, cmd, "wasm-bindgen").context("Running the wasm-bindgen CLI")?;
+    child::run(cmd, "wasm-bindgen").context("Running the wasm-bindgen CLI")?;
     Ok(())
 }
 
 /// Check if the `wasm-bindgen` dependency is locally satisfied.
-fn wasm_bindgen_version_check(bindgen_path: &PathBuf, dep_version: &str, log: &Logger) -> bool {
+fn wasm_bindgen_version_check(bindgen_path: &PathBuf, dep_version: &str) -> bool {
     let mut cmd = Command::new(bindgen_path);
     cmd.arg("--version");
-    child::run(log, cmd, "wasm-bindgen")
+    child::run(cmd, "wasm-bindgen")
         .map(|stdout| {
             stdout
                 .trim()
                 .split_whitespace()
                 .nth(1)
                 .map(|v| {
                     info!(
-                        log,
                         "Checking installed `wasm-bindgen` version == expected version: {} == {}",
-                        v,
-                        dep_version
+                        v, dep_version
                     );
                     v == dep_version
                 })

src/build.rs

@@ -4,7 +4,6 @@ use child;
 use emoji;
 use failure::{Error, ResultExt};
 use progressbar::Step;
-use slog::Logger;
 use std::path::Path;
 use std::process::Command;
 use std::str;
@@ -51,18 +50,17 @@ fn rustc_minor_version() -> Option<u32> {
 
 /// Ensure that `rustup` has the `wasm32-unknown-unknown` target installed for
 /// current toolchain
-pub fn rustup_add_wasm_target(log: &Logger, step: &Step) -> Result<(), Error> {
+pub fn rustup_add_wasm_target(step: &Step) -> Result<(), Error> {
     let msg = format!("{}Adding WASM target...", emoji::TARGET);
     PBAR.step(step, &msg);
     let mut cmd = Command::new("rustup");
     cmd.arg("target").arg("add").arg("wasm32-unknown-unknown");
-    child::run(log, cmd, "rustup")
-        .context("Adding the wasm32-unknown-unknown target with rustup")?;
+    child::run(cmd, "rustup").context("Adding the wasm32-unknown-unknown target with rustup")?;
     Ok(())
 }
 
 /// Run `cargo build` targetting `wasm32-unknown-unknown`.
-pub fn cargo_build_wasm(log: &Logger, path: &Path, debug: bool, step: &Step) -> Result<(), Error> {
+pub fn cargo_build_wasm(path: &Path, debug: bool, step: &Step) -> Result<(), Error> {
     let msg = format!("{}Compiling to WASM...", emoji::CYCLONE);
     PBAR.step(step, &msg);
     let mut cmd = Command::new("cargo");
@@ -71,18 +69,18 @@ pub fn cargo_build_wasm(log: &Logger, path: &Path, debug: bool, step: &Step) ->
         cmd.arg("--release");
     }
     cmd.arg("--target").arg("wasm32-unknown-unknown");
-    child::run(log, cmd, "cargo build").context("Compiling your crate to WebAssembly failed")?;
+    child::run(cmd, "cargo build").context("Compiling your crate to WebAssembly failed")?;
     Ok(())
 }
 
 /// Run `cargo build --tests` targetting `wasm32-unknown-unknown`.
-pub fn cargo_build_wasm_tests(log: &Logger, path: &Path, debug: bool) -> Result<(), Error> {
+pub fn cargo_build_wasm_tests(path: &Path, debug: bool) -> Result<(), Error> {
     let mut cmd = Command::new("cargo");
     cmd.current_dir(path).arg("build").arg("--tests");
     if !debug {
         cmd.arg("--release");
     }
     cmd.arg("--target").arg("wasm32-unknown-unknown");
-    child::run(log, cmd, "cargo build").context("Compilation of your program failed")?;
+    child::run(cmd, "cargo build").context("Compilation of your program failed")?;
     Ok(())
 }

src/child.rs

@@ -4,7 +4,7 @@
 //! properly logged and their output is logged as well.
 
 use failure::Error;
-use slog::Logger;
+use log::info;
 use std::{
     io::{self, Read},
     mem, process, string,
@@ -116,11 +116,10 @@ where
 
 /// Run the given command and return its stdout.
 pub fn run(
-    logger: &Logger,
     mut command: process::Command,
     command_name: &str,
 ) -> Result<String, Error> {
-    info!(logger, "Running {:?}", command);
+    info!("Running {:?}", command);
 
     let mut child = command
         .stdout(process::Stdio::piped())
@@ -144,11 +143,11 @@ pub fn run(
         thread::spawn(move || read_and_send(stderr, stderr_send, OutputFragment::Stderr));
 
     let mut stdout = OutputAccumulator::new(|line| {
-        info!(logger, "{} (stdout): {}", command_name, line);
+        info!("{} (stdout): {}", command_name, line);
         PBAR.message(line)
     });
     let mut stderr = OutputAccumulator::new(|line| {
-        info!(logger, "{} (stderr): {}", command_name, line);
+        info!("{} (stderr): {}", command_name, line);
         PBAR.message(line)
     });