Make ALTERNATIVE_LOCATIONS items PathBuf

Instead of directly OsString.

This is because these locations are paths. Representing them this
way has the further advantage of allowing some code to be slightly
simplified.

Author: EliahKagan

gix-path/src/env/git.rs

@@ -1,4 +1,3 @@
-use std::ffi::OsString;
 use std::path::{Path, PathBuf};
 use std::process::{Command, Stdio};
 
@@ -7,14 +6,14 @@ use once_cell::sync::Lazy;
 
 /// Other places to find Git in.
 #[cfg(windows)]
-pub(super) static ALTERNATIVE_LOCATIONS: Lazy<Vec<OsString>> = Lazy::new(|| {
+pub(super) static ALTERNATIVE_LOCATIONS: Lazy<Vec<PathBuf>> = Lazy::new(|| {
     vec![
         "C:/Program Files/Git/mingw64/bin".into(),
         "C:/Program Files (x86)/Git/mingw32/bin".into(),
     ]
 });
 #[cfg(not(windows))]
-pub(super) static ALTERNATIVE_LOCATIONS: Lazy<Vec<OsString>> = Lazy::new(|| vec![]);
+pub(super) static ALTERNATIVE_LOCATIONS: Lazy<Vec<PathBuf>> = Lazy::new(|| vec![]);
 
 #[cfg(windows)]
 pub(super) static EXE_NAME: &str = "git.exe";