Merge pull request #1590 from kate-goldenring/remove-git-build-requir…

…ements

fix(build): Remove git worktree requirements for build to succeed

build.rs

@@ -14,6 +14,9 @@ const RUST_OUTBOUND_REDIS_INTEGRATION_TEST: &str = "tests/outbound-redis/http-ru
 const TIMER_TRIGGER_INTEGRATION_TEST: &str = "examples/spin-timer/app-example";
 
 fn main() {
+    // Extract environment information to be passed to plugins.
+    // Git information will be set to defaults if Spin is not
+    // built within a Git worktree.
     vergen::EmitBuilder::builder()
         .build_date()
         .build_timestamp()
@@ -23,7 +26,6 @@ fn main() {
         .git_commit_date()
         .git_commit_timestamp()
         .git_sha(true)
-        .fail_on_error()
         .emit()
         .expect("failed to extract build information");
 

sdk/rust/build.rs

@@ -8,17 +8,14 @@ fn main() {
         format!("-{pre}")
     };
 
-    let output = Command::new("git")
+    let commit = Command::new("git")
         .arg("rev-parse")
         .arg("HEAD")
         .output()
-        .expect("failed to execute `git`");
-
-    let commit = if output.status.success() {
-        String::from_utf8(output.stdout).unwrap()
-    } else {
-        panic!("`git` failed: {}", String::from_utf8_lossy(&output.stderr));
-    };
+        .ok()
+        .filter(|o| o.status.success())
+        .map(|o| String::from_utf8_lossy(&o.stdout).to_string())
+        .unwrap_or(String::from("unknown"));
 
     let commit = commit.trim();