chmod should throw on Windows (#4446)

1 files changed, 2 insertions, 1 deletions

diff --git a/cli/ops/fs.rs b/cli/ops/fs.rs
index 416ae210a..7e526d71e 100644
--- a/cli/ops/fs.rs
+++ b/cli/ops/fs.rs
@@ -353,14 +353,15 @@ fn op_chmod(
       use std::os::unix::fs::PermissionsExt;
       let permissions = PermissionsExt::from_mode(mode);
       std::fs::set_permissions(&path, permissions)?;
+      Ok(json!({}))
     }
     // TODO Implement chmod for Windows (#4357)
     #[cfg(not(unix))]
     {
       // Still check file/dir exists on Windows
       let _metadata = std::fs::metadata(&path)?;
+      return Err(OpError::not_implemented());
     }
-    Ok(json!({}))
   })
 }