Merge pull request #15 from kilork/14_npm_windows

use cmd /c npm.cmd instead of just npm.cmd on windows

.github/workflows/rust.yml

@@ -5,7 +5,10 @@ on: [push]
 jobs:
   build:
 
-    runs-on: ubuntu-latest
+    runs-on: ${{ matrix.os }}
+    strategy:
+      matrix:
+        os: [ubuntu-latest, windows-latest, macos-latest]
 
     steps:
     - uses: actions/checkout@v1

src/impl.rs

@@ -554,7 +554,8 @@ impl NpmBuild {
 
     /// Executes `npm install`.
     pub fn install(self) -> io::Result<Self> {
-        if let Err(e) = Command::new(&self.executable)
+        if let Err(e) = self
+            .command()
             .arg("install")
             .current_dir(&self.package_json_dir)
             .status()
@@ -568,7 +569,8 @@ impl NpmBuild {
 
     /// Executes `npm run CMD`.
     pub fn run(self, cmd: &str) -> io::Result<Self> {
-        if let Err(e) = Command::new(&self.executable)
+        if let Err(e) = self
+            .command()
             .arg("run")
             .arg(cmd)
             .current_dir(&self.package_json_dir)
@@ -591,6 +593,20 @@ impl NpmBuild {
     pub fn to_resource_dir(self) -> ResourceDir {
         self.into()
     }
+
+    #[cfg(not(windows))]
+    fn command(&self) -> Command {
+        Command::new(&self.executable)
+    }
+
+    #[cfg(windows)]
+    fn command(&self) -> Command {
+        let mut cmd = Command::new("cmd");
+
+        cmd.arg("/c").arg(&self.executable);
+
+        cmd
+    }
 }
 
 impl From<NpmBuild> for ResourceDir {