feat: Add Windows support

.github/workflows/test.yml

@@ -3,7 +3,7 @@ on: [push, pull_request]
 name: ci
 
 jobs:
-  build_and_test:
+  test_coverage:
     name: cargo-run-bin
     runs-on: ubuntu-latest
     steps:
@@ -30,3 +30,24 @@ jobs:
         with:
           github-token: ${{ secrets.GITHUB_TOKEN }}
           path-to-lcov: ./lcov.info
+
+  test:
+    name: cargo-run-bin
+    strategy:
+      matrix:
+        os: [macos-latest, windows-latest]
+    runs-on: ${{ matrix.os }}
+    steps:
+      - uses: actions/checkout@v2
+      - uses: actions-rs/toolchain@v1
+        with:
+          toolchain: stable
+      - uses: Swatinem/rust-cache@v2
+        with:
+          cache-directories: ".bin"
+      - name: Install deps
+        run: |
+          cargo install --path .
+          cargo bin --build
+      - name: Test
+        run: cargo nextest run

Cargo.toml

@@ -16,22 +16,23 @@ name = "cargo-bin"
 path = "src/main.rs"
 
 [dependencies]
-anyhow = "1.0.40"
-clap = "4.3.19"
-owo-colors = "3.5.0"
-serde = { version = "1.0.149", features = ["derive"] }
-toml = "0.5.9"
-toml_edit = "0.19.14"
-version_check = "0.9.3"
-which = "4.4.0"
+anyhow = "=1.0.40"
+cfg-if = "=1.0.0"
+clap = "=4.3.19"
+owo-colors = "=3.5.0"
+serde = { version = "=1.0.149", features = ["derive"] }
+toml = "=0.5.9"
+toml_edit = "=0.19.14"
+version_check = "=0.9.3"
+which = "=4.4.0"
 
 [dev-dependencies]
-assert_cmd = "2.0.12"
-cargo-husky = { version = "1.5.0", default-features = false, features = ["user-hooks"] }
-insta = { version = "1.31.0", features = ["yaml"] }
+assert_cmd = "=2.0.12"
+cargo-husky = { version = "=1.5.0", default-features = false, features = ["user-hooks"] }
+insta = { version = "=1.31.0", features = ["yaml"] }
 
 [package.metadata.bin]
-cargo-binstall = { version = "1.4.4" }
+cargo-binstall = { version = "1.4.6" }
 cargo-cmd = { version = "0.3.1" }
 cargo-deny = { version = "0.13.5" }
 cargo-dist = { version = "0.1.0-prerelease.10" }

rust-toolchain.toml

@@ -1,3 +1,3 @@
 [toolchain]
-channel = "1.71.1"
+channel = "1.74.0"
 components = ["rustfmt", "clippy", "llvm-tools-preview"]

src/binary.rs

@@ -1,7 +1,6 @@
 use std::env;
 use std::fs;
 use std::io;
-use std::os::fd::AsFd;
 use std::path;
 use std::process;
 
@@ -22,11 +21,10 @@ pub fn cargo_install(
     binary_package: metadata::BinaryPackage,
     cache_path: path::PathBuf,
 ) -> Result<()> {
-    let stderr = io::stderr().as_fd().try_clone_to_owned()?;
     let mut cmd_prefix = process::Command::new("cargo");
 
     cmd_prefix
-        .stdout::<std::process::Stdio>(stderr.into())
+        .stdout(io::stderr())
         .stderr(process::Stdio::inherit())
         .arg("install")
         .arg("--root")
@@ -74,12 +72,10 @@ pub fn cargo_install(
 }
 
 pub fn binstall(binary_package: metadata::BinaryPackage, cache_path: path::PathBuf) -> Result<()> {
-    let stderr = io::stderr().as_fd().try_clone_to_owned()?;
-
     let mut cmd_prefix = process::Command::new("cargo");
 
     cmd_prefix
-        .stdout::<std::process::Stdio>(stderr.into())
+        .stdout(io::stderr())
         .stderr(process::Stdio::inherit())
         .arg("binstall")
         .arg("--no-confirm")
@@ -131,8 +127,11 @@ pub fn install(binary_package: metadata::BinaryPackage) -> Result<String> {
         .join(binary_package.package.clone())
         .join(binary_package.version.clone());
     let cache_bin_path = cache_path.join("bin").join(bin_name);
+    let mut cache_bin_path_win = cache_bin_path.clone();
+    cache_bin_path_win.set_extension("exe");
 
-    if !path::Path::new(&cache_bin_path).exists() {
+    if !path::Path::new(&cache_bin_path).exists() && !path::Path::new(&cache_bin_path_win).exists()
+    {
         fs::create_dir_all(&cache_path)?;
         if binary_package.bin_target.is_none()
             && binary_package.features.is_none()

src/cli.rs

@@ -122,7 +122,9 @@ pub fn run() -> Result<()> {
         app.print_long_help()?;
     } else {
         let mut args: Vec<_> = env::args().collect();
-        let start_index = args.iter().position(|e| return e.ends_with("/cargo-bin"));
+        let start_index = args
+            .iter()
+            .position(|e| return e.ends_with("/cargo-bin") || e.ends_with("cargo-bin.exe"));
         if start_index.is_none() || start_index.unwrap() == (args.len() + 1) {
             app.print_long_help()?;
             return Ok(());