feat: retry xcodegen at least three times.

Getting errors from objc NS side with Xcodegen being already created or it
can't be overwritten.

Author: kkharji

src/state/workspace.rs

@@ -111,25 +111,26 @@ impl Workspace {
     /// Update .compile commands
     #[cfg(feature = "xcodegen")]
     pub async fn update_xcodeproj(&mut self, update_config: bool) -> Result<()> {
-        match xcodegen::generate(&self.root).await {
-            Ok(msg) => {
-                tracing::info!("Updated {}.xcodeproj", self.name());
-                tracing::trace!("{:?}", msg);
-                if update_config {
-                    tracing::info!("Updated internal state.{}.project", self.name());
-                    self.project = Project::new_from_project_yml(
-                        self.root.clone(),
-                        xcodegen::config_path(self),
-                    )
-                    .await?;
+        tracing::info!("Updating {}.xcodeproj", self.name());
+
+        let retry_count = 0;
+        while retry_count < 3 {
+            if let Ok(code) = xcodegen::generate(&self.root).await {
+                if code.success() {
+                    if update_config {
+                        tracing::info!("Updating State.{}.Project", self.name());
+                        self.project = Project::new_from_project_yml(
+                            self.root.clone(),
+                            xcodegen::config_path(self),
+                        )
+                        .await?;
+                    }
+                    return Ok(());
                 }
-                Ok(())
-            }
-            Err(e) => {
-                tracing::error!("xcodegen generate: {:#?}", e);
-                Err(e)
             }
         }
+
+        anyhow::bail!("Fail to update_xcodeproj")
     }
 
     pub fn get_ignore_patterns(&self) -> Option<Vec<String>> {

src/xcode/actions.rs

@@ -18,7 +18,7 @@ where
     I: IntoIterator<Item = S>,
     S: AsRef<ffi::OsStr>,
 {
-    tracing::debug!("Building {:?}", root);
+    tracing::info!("Building {:?}", root);
     let output = Command::new("/usr/bin/xcodebuild")
         .arg("build")
         .args(args)
@@ -47,7 +47,7 @@ where
     I: IntoIterator<Item = S>,
     S: AsRef<ffi::OsStr>,
 {
-    tracing::debug!("Cleaning {:?}", root);
+    tracing::info!("Cleaning {:?}", root);
 
     Command::new("/usr/bin/xcodebuild")
         .arg("clean")
@@ -63,22 +63,20 @@ where
 
 #[cfg(feature = "serial")]
 pub async fn update_compiled_commands(root: &PathBuf, build_log: Vec<String>) -> Result<()> {
-    fs::write(
-        root.join(".compile"),
-        Compiliation::new(build_log).to_json()?,
-    )
-    .await?;
-    tracing::info!("Updated Compiled Commands");
+    let path = root.join(".compile");
+    tracing::info!("Updating {:?}", path);
+    fs::write(path, Compiliation::new(build_log).to_json()?).await?;
     Ok(())
 }
 
 pub async fn ensure_server_config_file(root: &PathBuf) -> Result<()> {
     let path = root.join("buildServer.json");
     if fs::File::open(&path).await.is_ok() {
-        tracing::debug!("buildServer.json exists.");
         return Ok(());
     }
 
+    tracing::info!("Creating {:?}", path);
+
     let mut file = fs::File::create(path).await?;
     let config = json! ({
         "name": "XcodeBase Server",
@@ -99,7 +97,5 @@ pub async fn ensure_server_config_file(root: &PathBuf) -> Result<()> {
     file.sync_all().await?;
     file.shutdown().await?;
 
-    tracing::debug!("buildServer.json created!");
-
     Ok(())
 }

src/xcodegen/mod.rs

@@ -1,7 +1,8 @@
 use std::fmt::Debug;
 use std::path::{Path, PathBuf};
+use std::process::ExitStatus;
 
-use anyhow::Result;
+use anyhow::{Context, Result};
 
 use crate::Workspace;
 
@@ -22,33 +23,17 @@ fn xcodgen() -> tokio::process::Command {
 }
 
 // Run xcodgen generate
-pub async fn generate<P: AsRef<Path> + Debug>(root: P) -> Result<Vec<String>> {
-    let output = xcodgen()
+pub async fn generate<P: AsRef<Path> + Debug>(root: P) -> Result<ExitStatus> {
+    xcodgen()
         .current_dir(root)
-        .stdout(std::process::Stdio::piped())
-        .stderr(std::process::Stdio::piped())
+        .stdout(std::process::Stdio::null())
+        .stderr(std::process::Stdio::null())
         .arg("generate")
         .arg("-c")
-        .spawn()
-        .expect("Failed to start xcodeGen.")
-        .wait_with_output()
+        .spawn()?
+        .wait()
         .await
-        .expect("Failed to run xcodeGen.");
-
-    if output.status.code().unwrap().ne(&0) {
-        anyhow::bail!(
-            "{:#?}",
-            String::from_utf8(output.stderr)?
-                .split("\n")
-                .map(|s| s.to_string())
-                .collect::<Vec<String>>()
-        )
-    } else {
-        Ok(String::from_utf8(output.stdout)?
-            .split("\n")
-            .map(|s| s.to_string())
-            .collect())
-    }
+        .context("xcodegen generate")
 }
 
 // NOTE: passing workspace in-case in the future we would allow configurability of project.yml path