fixup! core: allow to install a branch

Author: vincenzopalazzo

coffee_core/src/coffee.rs

@@ -318,7 +318,10 @@ impl PluginManager for CoffeeManager {
                         plugin.exec_path = new_exec_path;
 
                         if let Some(branch) = branch {
-                            let _ = git_checkout(&plugin.root_path, &branch, verbose).await?;
+                            // FIXME: Where we store the date? how we manage it?
+                            let (commit, _) =
+                                git_checkout(&plugin.root_path, &branch, verbose).await?;
+                            plugin.commit = Some(commit);
                         }
 
                         log::debug!("plugin: {:?}", plugin);

coffee_github/src/utils.rs

@@ -53,20 +53,14 @@ pub async fn git_checkout(
     path: &str,
     branch: &str,
     verbose: bool,
-) -> Result<UpgradeStatus, CoffeeError> {
-    let repo = git2::Repository::open(path).map_err(|err| error!("{}", err.message()))?;
-    let (local_commit, _) = get_repo_info!(repo);
-
+) -> Result<(String, String), CoffeeError> {
     let mut cmd = format!("git fetch origin\n");
     cmd += &format!("git reset --hard\n");
     cmd += &format!("git checkout origin/{branch}");
     sh!(path, cmd, verbose);
 
+    let repo = git2::Repository::open(path).map_err(|err| error!("{}", err.message()))?;
     let (upstream_commit, date) = get_repo_info!(repo);
 
-    if local_commit == upstream_commit {
-        Ok(UpgradeStatus::UpToDate(upstream_commit, date))
-    } else {
-        Ok(UpgradeStatus::Updated(upstream_commit, date))
-    }
+    Ok((upstream_commit, date))
 }