Make git_dir required in several git functions

It was always called with `Some`, so no need to complicate it with `Option`.

Author: Kobzol

src/bootstrap/src/core/config/config.rs

@@ -3181,13 +3181,8 @@ impl Config {
             .unwrap()
             .entry(paths.to_vec())
             .or_insert_with(|| {
-                check_path_modifications(
-                    Some(&self.src),
-                    &self.git_config(),
-                    paths,
-                    CiEnv::current(),
-                )
-                .unwrap()
+                check_path_modifications(&self.src, &self.git_config(), paths, CiEnv::current())
+                    .unwrap()
             })
             .clone()
     }

src/build_helper/src/git.rs

@@ -75,7 +75,7 @@ pub enum PathFreshness {
 /// PR made modifications to `target_paths`. If not, then we simply take the latest upstream
 /// commit, because on CI there is no need to avoid redownloading.
 pub fn check_path_modifications(
-    git_dir: Option<&Path>,
+    git_dir: &Path,
     config: &GitConfig<'_>,
     target_paths: &[&str],
     ci_env: CiEnv,
@@ -105,7 +105,7 @@ pub fn check_path_modifications(
         // Do not include HEAD, as it is never an upstream commit
         // If we do not find an upstream commit in CI, something is seriously wrong.
         Some(
-            get_closest_upstream_commit(git_dir, config, ci_env)?
+            get_closest_upstream_commit(Some(git_dir), config, ci_env)?
                 .expect("No upstream commit was found on CI"),
         )
     } else {
@@ -120,7 +120,7 @@ pub fn check_path_modifications(
         )?;
         match upstream_with_modifications {
             Some(sha) => Some(sha),
-            None => get_closest_upstream_commit(git_dir, config, ci_env)?,
+            None => get_closest_upstream_commit(Some(git_dir), config, ci_env)?,
         }
     };
 
@@ -141,12 +141,9 @@ pub fn check_path_modifications(
 }
 
 /// Returns true if any of the passed `paths` have changed since the `base` commit.
-pub fn has_changed_since(git_dir: Option<&Path>, base: &str, paths: &[&str]) -> bool {
+pub fn has_changed_since(git_dir: &Path, base: &str, paths: &[&str]) -> bool {
     let mut git = Command::new("git");
-
-    if let Some(git_dir) = git_dir {
-        git.current_dir(git_dir);
-    }
+    git.current_dir(git_dir);
 
     git.args(["diff-index", "--quiet", base, "--"]).args(paths);
 
@@ -158,15 +155,12 @@ pub fn has_changed_since(git_dir: Option<&Path>, base: &str, paths: &[&str]) ->
 /// Returns the latest commit that modified `target_paths`, or `None` if no such commit was found.
 /// If `author` is `Some`, only considers commits made by that author.
 fn get_latest_commit_that_modified_files(
-    git_dir: Option<&Path>,
+    git_dir: &Path,
     target_paths: &[&str],
     author: &str,
 ) -> Result<Option<String>, String> {
     let mut git = Command::new("git");
-
-    if let Some(git_dir) = git_dir {
-        git.current_dir(git_dir);
-    }
+    git.current_dir(git_dir);
 
     git.args(["rev-list", "-n1", "--first-parent", "HEAD", "--author", author]);
 

src/tools/compiletest/src/lib.rs

@@ -746,8 +746,7 @@ fn modified_tests(config: &Config, dir: &Path) -> Result<Vec<PathBuf>, String> {
         return Ok(vec![]);
     }
 
-    let files =
-        get_git_modified_files(&config.git_config(), Some(dir), &vec!["rs", "stderr", "fixed"])?;
+    let files = get_git_modified_files(&config.git_config(), dir, &vec!["rs", "stderr", "fixed"])?;
     // Add new test cases to the list, it will be convenient in daily development.
     let untracked_files = get_git_untracked_files(Some(dir))?.unwrap_or(vec![]);