Fix Windows by using dunce

Author: philipp-spiess

Cargo.lock

@@ -20,11 +20,12 @@ bench = false
 
 [dependencies]
 crossbeam-deque = "0.8.3"
-globset = { path = "../globset" }
+globset = "0.4.16"
 log = "0.4.20"
 memchr = "2.6.3"
 same-file = "1.0.6"
 walkdir = "2.4.0"
+dunce = "1.0.5"
 
 [dependencies.regex-automata]
 version = "0.4.0"

crates/ignore/Cargo.toml

@@ -176,7 +176,8 @@ impl Ignore {
         if !self.is_root() {
             panic!("Ignore::add_parents called on non-root matcher");
         }
-        let absolute_base = match path.as_ref().canonicalize() {
+        // CHANGED: Use `dunce::canonicalize` as we use it everywhere else.
+        let absolute_base = match dunce::canonicalize(path.as_ref()) {
             Ok(path) => Arc::new(path),
             Err(_) => {
                 // There's not much we can do here, so just return our
@@ -504,8 +505,6 @@ impl Ignore {
             &m_explicit,
         ];
 
-        dbg!(path.to_string_lossy(), order);
-
         for (idx, check) in order.into_iter().enumerate() {
             if check.is_none() {
                 continue;

crates/ignore/src/dir.rs

@@ -184,7 +184,6 @@ impl Gitignore {
     /// gitignore) is stripped. If there is no common suffix/prefix overlap,
     /// then `path` is assumed to be relative to this matcher.
     pub fn matched<P: AsRef<Path>>(&self, path: P, is_dir: bool) -> Match<&Glob> {
-        dbg!(path.as_ref(), &self.globs);
         if self.is_empty() {
             return Match::None;
         }
@@ -247,7 +246,6 @@ impl Gitignore {
         self.set.matches_candidate_into(&candidate, &mut *matches);
         for &i in matches.iter().rev() {
             let glob = &self.globs[i];
-            dbg!(&glob);
             if !glob.is_only_dir() || is_dir {
                 return if glob.is_whitelist() {
                     Match::Whitelist(glob)
@@ -278,7 +276,6 @@ impl Gitignore {
         // As an additional special case, if the root is just `.`, then we
         // shouldn't try to strip anything, e.g., when path begins with a `.`.
         if self.root != Path::new(".") && !is_file_name(path) {
-            dbg!(&self.root, &path);
             if let Some(p) = strip_prefix(&self.root, path) {
                 path = p;
                 // If we're left with a leading slash, get rid of it.

crates/ignore/src/gitignore.rs

@@ -100,8 +100,6 @@ impl Scanner {
     pub fn new(sources: Vec<PublicSourceEntry>) -> Self {
         let sources = Sources::new(public_source_entries_to_private_source_entries(sources));
 
-        dbg!(&sources);
-
         Self {
             sources: sources.clone(),
             walker: create_walker(sources),