Refactor: Open git repository in command::build()

This commit refactors the git2::Repository::open() call to be in the
command::build() implementation, so we do not open the repository multiple
times.

Signed-off-by: Matthias Beyer <matthias.beyer@atos.net>

1 files changed, 5 insertions, 2 deletions

diff --git a/src/commands/build.rs b/src/commands/build.rs
index 1044896..4474d4e 100644
--- a/src/commands/build.rs
+++ b/src/commands/build.rs
@@ -61,7 +61,10 @@ pub async fn build(
 ) -> Result<()> {
     use crate::db::models::{EnvVar, GitHash, Image, Job, Package, Submit};
 
-    let _ = crate::ui::package_repo_cleanness_check(&repo_root)?;
+    let git_repo = git2::Repository::open(repo_path)
+        .with_context(|| anyhow!("Opening repository at {}", repo_path.display()))?;
+
+    let _ = crate::ui::package_repo_cleanness_check(&git_repo)?;
     let now = chrono::offset::Local::now().naive_local();
 
     let shebang = Shebang::from({
@@ -92,7 +95,7 @@ pub async fn build(
     }
 
     debug!("Getting repository HEAD");
-    let hash_str = crate::util::git::get_repo_head_commit_hash(repo_path)?;
+    let hash_str = crate::util::git::get_repo_head_commit_hash(&git_repo)?;
     trace!("Repository HEAD = {}", hash_str);
     let phases = config.available_phases();