From 11bdb6c878452985878d5b45d962d0596ef124b2 Mon Sep 17 00:00:00 2001 From: Adrian Malacoda Date: Thu, 28 Dec 2017 00:39:10 -0600 Subject: [PATCH] use status() instead of output() to capture the output of the lessc/browserify process, panic if it fails --- src/lib.rs | 18 ++++++++++++++---- 1 file changed, 14 insertions(+), 4 deletions(-) diff --git a/src/lib.rs b/src/lib.rs index b2a116d..edd53b0 100644 --- a/src/lib.rs +++ b/src/lib.rs @@ -16,10 +16,15 @@ pub fn browserify (infile: &str, outfile: &str) -> PathBuf { let out_dir = env::var("OUT_DIR").unwrap(); let out_dir = Path::new(&out_dir); let full_outfile = out_dir.join(outfile); - Command::new("node_modules/.bin/browserify") + let status = Command::new("node_modules/.bin/browserify") .args(&[infile, "-o", full_outfile.to_str().expect("failed to build output path")]) - .output() + .status() .expect("failed to build js bundle"); + + if !status.success() { + panic!("Failed to browserify input file {}", infile); + } + full_outfile } @@ -27,10 +32,15 @@ pub fn lessc (infile: &str, outfile: &str) -> PathBuf { let out_dir = env::var("OUT_DIR").unwrap(); let out_dir = Path::new(&out_dir); let full_outfile = out_dir.join(outfile); - Command::new("node_modules/.bin/lessc") + let status = Command::new("node_modules/.bin/lessc") .args(&[infile, full_outfile.to_str().expect("failed to build output path")]) - .output() + .status() .expect("failed to build css bundle"); + + if !status.success() { + panic!("Failed to lessc input file {}", infile); + } + full_outfile }