Move git commands to function

1 files changed, 21 insertions, 16 deletions

diff --git a/build.rs b/build.rs
index 2b15f84..6efb53f 100644
--- a/build.rs
+++ b/build.rs
@@ -1,10 +1,10 @@
 //
 // Copyright (c) 2025 murilo ijanc' <murilo@ijanc.org>
-// 
+//
 // Permission to use, copy, modify, and distribute this software for any
 // purpose with or without fee is hereby granted, provided that the above
 // copyright notice and this permission notice appear in all copies.
-// 
+//
 // THE SOFTWARE IS PROVIDED "AS IS" AND THE AUTHOR DISCLAIMS ALL WARRANTIES
 // WITH REGARD TO THIS SOFTWARE INCLUDING ALL IMPLIED WARRANTIES OF
 // MERCHANTABILITY AND FITNESS. IN NO EVENT SHALL THE AUTHOR BE LIABLE FOR
@@ -14,24 +14,29 @@
 // OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
 //
 
-use std::process::Command;
-
 fn main() {
-    let git_hash = Command::new("git")
-        .args(["rev-parse", "--short=6", "HEAD"])
-        .output()
-        .map(|out| String::from_utf8_lossy(&out.stdout).trim().to_string())
-        .unwrap_or_else(|_| "unknown".into());
+    println!("cargo:rerun-if-changed=.git/HEAD");
+    println!("cargo:rerun-if-changed=.git/refs/heads");
 
-    let git_date = Command::new("git")
-        .args(["show", "-s", "--format=%cd", "--date=short", "HEAD"])
-        .output()
-        .map(|out| String::from_utf8_lossy(&out.stdout).trim().to_string())
-        .unwrap_or_else(|_| "unknown".into());
+    let git_hash = git_cmd(&["rev-parse", "--short=6", "HEAD"]);
+
+    let git_date =
+        git_cmd(&["show", "-s", "--format=%cd", "--date=short", "HEAD"]);
 
     println!("cargo:rustc-env=GIT_HASH={}", git_hash);
     println!("cargo:rustc-env=BUILD_DATE={}", git_date);
+}
 
-    println!("cargo:rerun-if-changed=.git/HEAD");
-    println!("cargo:rerun-if-changed=.git/refs/heads");
+fn git_cmd(args: &[&str]) -> String {
+    use std::process::Command;
+
+    Command::new("git")
+        .args(args)
+        .output()
+        .ok()
+        .filter(|output| output.status.success())
+        .and_then(|output| String::from_utf8(output.stdout).ok())
+        .map(|s| s.to_string())
+        .filter(|s| !s.is_empty())
+        .unwrap_or_else(|| "unknown".to_string())
 }