fix: capture needed stdout for sever commands (#149)

* Fix several command fail because of stdout isn't captured

* Use read instead

.changes/fix-capture.md

@@ -0,0 +1,6 @@
+---
+"tauri-mobile": patch
+---
+
+Fix several commands fail because stdout isn't captured.
+

src/apple/deps/xcode_plugin.rs

@@ -60,12 +60,11 @@ pub fn xcode_user_dir() -> Result<PathBuf, Error> {
 
 pub fn xcode_developer_dir() -> Result<PathBuf, Error> {
     duct::cmd("xcode-select", ["-p"])
-        .run()
+        .read()
         .map(|output| {
-            let stdout = String::from_utf8_lossy(&output.stdout).to_string();
             // This output is expected to end with a newline, but we'll err on
             // the safe side and proceed gracefully if it doesn't.
-            PathBuf::from(stdout.strip_suffix('\n').unwrap_or(&stdout))
+            PathBuf::from(output.strip_suffix('\n').unwrap_or(&output))
         })
         .map_err(Error::XcodeSelectFailed)
 }

src/apple/teams.rs

@@ -12,6 +12,7 @@ pub fn get_pem_list(name_substr: &str) -> std::io::Result<std::process::Output>
         "security",
         ["find-certificate", "-p", "-a", "-c", name_substr],
     )
+    .stdout_capture()
     .run()
 }
 

src/util/git/repo.rs

@@ -89,10 +89,12 @@ impl Repo {
                 .map_err(Error::FetchFailed)?;
             let local = git
                 .command_parse("rev-parse HEAD")
+                .stdout_capture()
                 .run()
                 .map_err(Error::RevParseLocalFailed)?;
             let remote = git
                 .command_parse("rev-parse @{u}")
+                .stdout_capture()
                 .run()
                 .map_err(Error::RevParseRemoteFailed)?;
             if local.stdout != remote.stdout {

src/util/mod.rs

@@ -477,13 +477,9 @@ pub fn prepend_to_path(path: impl Display, base_path: impl Display) -> String {
 }
 
 pub fn command_present(name: &str) -> Result<bool, std::io::Error> {
-    command_path(name).map(|_path| true).or_else(|err| {
-        if err.raw_os_error().is_some() {
-            Ok(false)
-        } else {
-            Err(err)
-        }
-    })
+    command_path(name)
+        .map(|_path| true)
+        .or_else(|_err| Ok(false))
 }
 
 #[derive(Debug)]