better clone technique and unsupported OS messages

Author: david-crespo

dev-tools/downloader/src/lib.rs

@@ -257,15 +257,12 @@ impl<'a> Downloader<'a> {
         let temp_dir = tempfile::tempdir()?;
         let temp_path = Utf8PathBuf::try_from(temp_dir.path().to_path_buf())?;
 
-        // Shallow clone the repository
+        // Clone and checkout the specific commit
         info!(self.log, "Cloning {repo_url}");
         let mut clone_cmd = Command::new("git");
         clone_cmd
             .arg("clone")
-            .arg("--depth")
-            .arg("1")
-            .arg("--branch")
-            .arg(commit)
+            .arg("--filter=blob:none")
             .arg(repo_url)
             .arg(&temp_path);
 
@@ -275,6 +272,17 @@ impl<'a> Downloader<'a> {
             bail!("Failed to clone {repo_url}: {stderr}");
         }
 
+        // Checkout the specific commit
+        info!(self.log, "Checking out commit {commit}");
+        let mut checkout_cmd = Command::new("git");
+        checkout_cmd.arg("checkout").arg(commit).current_dir(&temp_path);
+
+        let checkout_output = checkout_cmd.output().await?;
+        if !checkout_output.status.success() {
+            let stderr = String::from_utf8_lossy(&checkout_output.stderr);
+            bail!("Failed to checkout {commit}: {stderr}");
+        }
+
         // Build each binary
         tokio::fs::create_dir_all(&cache_dir).await?;
         let mut result_paths = Vec::new();

tools/install_builder_prerequisites.sh

@@ -199,7 +199,7 @@ function install_packages {
     )
     confirm "Install (or update) [${packages[*]}]?" && brew install "${packages[@]}"
   else
-    echo "Unsupported OS: ${HOST_OS}"
+    echo "Skipping builder prereqs for unsupported OS: ${HOST_OS}"
     exit 1
   fi
 }

tools/install_runner_prerequisites.sh

@@ -153,7 +153,7 @@ function install_packages {
       confirm "Install (or update) [${packages[*]}]?" && $maybe_sudo apt-get install "${packages[@]}"
     fi
   else
-    echo "Unsupported OS: ${HOST_OS}"
+    echo "Skipping runner prereqs for unsupported OS: ${HOST_OS}"
     exit 1
   fi
 }