Made nj-cli exit non-zero when cargo build fails. (#105)

Author: simlay

Cargo.lock

@@ -1,6 +1,6 @@
 [package]
 name = "nj-cli"
-version = "0.4.0"
+version = "0.4.1"
 authors = ["fluvio.io"]
 edition = "2018"
 description = "build tool for node-bindgen"
@@ -12,4 +12,4 @@ log = "0.4.8"
 structopt = { version = "0.3.18", default-features = false }
 cargo_metadata = "0.11.3"
 toml = "0.5.6"
-serde = { version = "1.0.114", features = ["derive"] }
+serde = { version = "1.0.114", features = ["derive"] }

nj-cli/Cargo.toml

@@ -82,7 +82,7 @@ fn init(opt: InitOpt) {
         .stdout(Stdio::inherit())
         .spawn()
         .expect("Failed to execute command");
-    
+
     build_command.wait()
         .expect("failed to wait on child");
 
@@ -97,7 +97,7 @@ fn init(opt: InitOpt) {
                 .args(&["fmt", &manifest_path])
                 .spawn()
                 .expect("Failed to execute command");
-            
+
             fmt.wait()
                 .expect("Failed to execute command");
         };
@@ -119,9 +119,30 @@ fn build(opt: BuildOpt) {
         .spawn()
         .expect("Failed to execute command");
 
-    build_command.wait()
+    let status = build_command.wait()
         .expect("failed to wait on child");
-
+    match status.code() {
+        Some(code) if code != 0 => {
+            std::process::exit(code);
+        }
+        None => {
+            //https://doc.rust-lang.org/std/process/struct.ExitStatus.html#method.code
+            #[cfg(unix)]
+            {
+                use std::os::unix::process::ExitStatusExt;
+                if let Some(signal) = status.signal() {
+                    std::process::exit(signal);
+                } else {
+                    std::process::exit(1);
+                }
+            }
+            #[cfg(not(unix))]
+            {
+                std::process::exit(1);
+            }
+        }
+        Some(_) => {}
+    }
     let target = if opt.release {
         "release"
     } else {
@@ -132,7 +153,7 @@ fn build(opt: BuildOpt) {
 
 }
 
-/// copy library to target directory 
+/// copy library to target directory
 fn copy_lib(out: String,target_type: &str) {
 
     let manifest_path = manifest_path();
@@ -198,7 +219,7 @@ fn lib_path(target: &Path,build_type: &str,target_name: &str) -> PathBuf {
     }else{
         panic!("Unsupported operating system.");
     }.replace("-","_");
-    
+
     target.join(target).join(build_type).join(file_name)
 }
 

nj-cli/src/main.rs

@@ -0,0 +1,51 @@
+     * [ ]  napi_create_arraybuffer - NOT HERE
+     * [ ]  napi_create_buffer
+     * [ ]  napi_create_buffer_copy
+     * [ ]  napi_create_date
+     * [ ]  napi_create_external
+     * [ ]  napi_create_external_arraybuffer
+     * [ ]  napi_create_external_buffer
+     * [ ]  napi_create_symbol
+     * [ ]  napi_create_typedarray
+     * [ ]  napi_create_dataview
+     * [ ]  napi_create_uint32
+     * [ ]  napi_create_bigint_int64
+     * [ ]  napi_create_bigint_uint64
+     * [ ]  napi_create_bigint_words
+     * [ ]  napi_create_string_latin1
+     * [ ]  napi_get_array_length
+     * [ ]  napi_get_arraybuffer_info
+     * [ ]  napi_get_buffer_info
+     * [ ]  napi_get_prototype
+     * [ ]  napi_get_typedarray_info
+     * [ ]  napi_get_dataview_info
+     * [ ]  napi_get_value_bool
+     * [ ]  napi_get_value_double
+     * [ ]  napi_get_value_bigint_int64
+     * [ ]  napi_get_value_bigint_uint64
+     * [ ]  napi_get_value_bigint_words
+     * [ ]  napi_get_value_external
+     * [ ]  napi_get_value_int32
+     * [ ]  napi_get_value_int64
+     * [ ]  napi_get_value_string_utf8
+     * [ ]  napi_get_value_uint32
+     * [ ]  napi_get_null
+     * [ ]  napi_coerce_to_bool
+     * [ ]  napi_coerce_to_number
+     * [ ]  napi_coerce_to_string
+     * [ ]  napi_instanceof
+     * [ ]  napi_is_dataview
+     * [ ]  napi_strict_equals
+     * [ ]  napi_detach_arraybuffer
+     * [ ]  napi_is_detached_arraybuffer
+     * [ ]  napi_get_property_names
+     * [ ]  napi_get_all_property_names
+     * [ ]  napi_delete_property
+     * [ ]  napi_set_named_property
+     * [ ]  napi_get_named_property
+     * [ ]  napi_has_named_property
+     * [ ]  napi_has_element
+     * [ ]  napi_delete_element
+     * [ ]  napi_create_function
+     * [ ]  napi_get_version
+     * [ ]  napi_get_uv_event_loop