fix repo initialization

Author: ibraheemdev

crates/uv-git/src/git.rs

@@ -130,8 +130,6 @@ pub(crate) struct GitDatabase {
 
 /// A local checkout of a particular revision from a [`GitRepository`].
 pub(crate) struct GitCheckout {
-    /// Path to the root of the underlying Git repository on the local filesystem.
-    path: PathBuf,
     /// The git revision this checkout is for.
     revision: GitOid,
     /// Underlying Git repository instance for this checkout.
@@ -150,7 +148,7 @@ impl GitRepository {
         // Make sure there is a Git repository at the specified path.
         ProcessBuilder::new("git")
             .arg("rev-parse")
-            .arg("HEAD")
+            .cwd(path)
             .exec_with_streaming(&mut silent, &mut silent, true)?;
 
         Ok(GitRepository {
@@ -293,7 +291,9 @@ impl GitDatabase {
             .cwd(&self.repo.path)
             .exec_with_streaming(&mut silent, &mut silent, true)?;
 
-        Ok(String::from_utf8(output.stdout)?)
+        let mut result = String::from_utf8(output.stdout)?;
+        result.truncate(result.trim_end().len());
+        Ok(result)
     }
 
     /// Checks if the database contains the object of this `oid`.
@@ -343,11 +343,7 @@ impl GitCheckout {
     ///
     /// * The `repo` will be the checked out Git repository.
     fn new(revision: GitOid, repo: GitRepository) -> GitCheckout {
-        GitCheckout {
-            path: repo.path.clone(),
-            revision,
-            repo,
-        }
+        GitCheckout { revision, repo }
     }
 
     /// Clone a repo for a `revision` into a local path from a `database`.
@@ -384,7 +380,7 @@ impl GitCheckout {
         match self.repo.rev_parse("HEAD") {
             Ok(id) if id == self.revision => {
                 // See comments in reset() for why we check this
-                self.path.join(CHECKOUT_READY_LOCK).exists()
+                self.repo.path.join(CHECKOUT_READY_LOCK).exists()
             }
             _ => false,
         }
@@ -404,7 +400,7 @@ impl GitCheckout {
     ///
     /// [`.cargo-ok`]: CHECKOUT_READY_LOCK
     fn reset(&self) -> Result<()> {
-        let ok_file = self.path.join(CHECKOUT_READY_LOCK);
+        let ok_file = self.repo.path.join(CHECKOUT_READY_LOCK);
         let _ = paths::remove_file(&ok_file);
         debug!("reset {} to {}", self.repo.path.display(), self.revision);
 
@@ -419,7 +415,7 @@ impl GitCheckout {
             .arg("reset")
             .arg("--hard")
             .arg(self.revision.as_str())
-            .cwd(&self.path)
+            .cwd(&self.repo.path)
             .exec_with_streaming(&mut silent, &mut silent, true)?;
 
         paths::create(ok_file)?;
@@ -433,7 +429,7 @@ impl GitCheckout {
             .arg("update")
             .arg("--recursive")
             .arg("--init")
-            .cwd(&self.path)
+            .cwd(&self.repo.path)
             .exec_with_streaming(&mut silent, &mut silent, true)
             .map(drop)
     }