feat: remove log dependency in favor of gix-trace

Author: Byron

Cargo.lock

@@ -20,14 +20,14 @@ serde = ["dep:serde", "bstr/serde", "gix-glob/serde", "kstring/serde"]
 gix-path = { version = "^0.9.0", path = "../gix-path" }
 gix-quote = { version = "^0.4.7", path = "../gix-quote" }
 gix-glob = { version = "^0.11.0", path = "../gix-glob" }
+gix-trace = { version = "^0.1.3", path = "../gix-trace" }
 
 bstr = { version = "1.3.0", default-features = false, features = ["std", "unicode"]}
 smallvec = "1.10.0"
 kstring = "2.0.0"
 unicode-bom = "2.0.2"
 thiserror = "1.0.26"
 serde = { version = "1.0.114", optional = true, default-features = false, features = ["derive"]}
-log = "0.4.17"
 
 document-features = { version = "0.2.1", optional = true }
 

gix-attributes/Cargo.toml

@@ -124,7 +124,7 @@ impl Search {
 impl Pattern for Attributes {
     type Value = Value;
 
-    fn bytes_to_patterns(bytes: &[u8], source: &std::path::Path) -> Vec<pattern::Mapping<Self::Value>> {
+    fn bytes_to_patterns(bytes: &[u8], _source: &std::path::Path) -> Vec<pattern::Mapping<Self::Value>> {
         fn into_owned_assignments<'a>(
             attrs: impl Iterator<Item = Result<crate::AssignmentRef<'a>, crate::name::Error>>,
         ) -> Option<Assignments> {
@@ -138,8 +138,8 @@ impl Pattern for Attributes {
                 .collect::<Result<Assignments, _>>();
             match res {
                 Ok(res) => Some(res),
-                Err(err) => {
-                    log::warn!("{}", err);
+                Err(_err) => {
+                    gix_trace::warn!("{}", _err);
                     None
                 }
             }
@@ -148,8 +148,8 @@ impl Pattern for Attributes {
         crate::parse(bytes)
             .filter_map(|res| match res {
                 Ok(pattern) => Some(pattern),
-                Err(err) => {
-                    log::warn!("{}: {}", source.display(), err);
+                Err(_err) => {
+                    gix_trace::warn!("{}: {}", _source.display(), _err);
                     None
                 }
             })

gix-attributes/src/search/attributes.rs

@@ -24,7 +24,6 @@ gix-sec = { version = "^0.9.0", path = "../gix-sec" }
 gix-ref = { version = "^0.35.0", path = "../gix-ref" }
 gix-glob = { version = "^0.11.0", path = "../gix-glob" }
 
-log = "0.4.17"
 winnow = { version = "0.5.14", features = ["simd"] }
 memchr = "2"
 thiserror = "1.0.26"

gix-config/Cargo.toml

@@ -92,7 +92,7 @@ impl File<'static> {
                     Err(err) => {
                         let err = Error::Io { source: err, path };
                         if options.ignore_io_errors {
-                            log::warn!("ignoring: {err:#?}");
+                            gix_features::trace::warn!("ignoring: {err:#?}");
                             continue;
                         } else {
                             return Err(err);
@@ -104,7 +104,7 @@ impl File<'static> {
                 Ok(_) => {}
                 Err(err) => {
                     if options.ignore_io_errors {
-                        log::warn!(
+                        gix_features::trace::warn!(
                             "ignoring: {:#?}",
                             Error::Io {
                                 source: err,

gix-config/src/file/init/from_paths.rs

@@ -246,7 +246,6 @@ prodash = { version = "26.2", optional = true, default-features = false, feature
 once_cell = "1.14.0"
 signal-hook = { version = "0.3.9", default-features = false, optional = true }
 thiserror = "1.0.26"
-log = "0.4.14"
 serde = { version = "1.0.114", optional = true, default-features = false, features = ["derive"]}
 smallvec = "1.9.0"
 async-std = { version = "1.12.0", optional = true }

gix/Cargo.toml

@@ -102,7 +102,7 @@ fn load_config(
                 path: config_path,
             };
             if lenient {
-                log::warn!("ignoring: {err:#?}");
+                gix_trace::warn!("ignoring: {err:#?}");
                 return Ok(gix_config::File::new(metadata));
             } else {
                 return Err(err);
@@ -117,7 +117,7 @@ fn load_config(
             path: config_path,
         };
         if lenient {
-            log::warn!("ignoring: {err:#?}");
+            gix_trace::warn!("ignoring: {err:#?}");
             buf.clear();
         } else {
             return Err(err);