feat: add Status iterator.

TBD

Author: Byron

gix/Cargo.toml

@@ -65,7 +65,7 @@ comfort = ["gix-features/progress-unit-bytes", "gix-features/progress-unit-human
 command = ["dep:gix-command"]
 
 ## Obtain information similar to `git status`.
-status = ["gix-status"]
+status = ["gix-status", "dirwalk"]
 
 ## Utilities for interrupting computations and cleaning up tempfiles.
 interrupt = ["dep:signal-hook", "gix-tempfile/signals"]
@@ -131,12 +131,12 @@ blocking-http-transport-curl-rustls = ["blocking-http-transport-curl", "dep:curl
 ## Stacks with `blocking-network-client` to provide support for HTTP/S using **reqwest**, and implies blocking networking as a whole, making the `https://` transport available.
 blocking-http-transport-reqwest = ["blocking-network-client", "gix-transport/http-client-reqwest"]
 ## Stacks with `blocking-http-transport-reqwest` and enables `https://` via the `rustls` crate.
-blocking-http-transport-reqwest-rust-tls = ["blocking-http-transport-reqwest", "reqwest-for-configuration-only/rustls-tls" ]
+blocking-http-transport-reqwest-rust-tls = ["blocking-http-transport-reqwest", "reqwest-for-configuration-only/rustls-tls"]
 ## Stacks with `blocking-http-transport-reqwest` and enables `https://` via the `rustls` crate.
 ## This also makes use of `trust-dns` to avoid `getaddrinfo`, but note it comes with its own problems.
 blocking-http-transport-reqwest-rust-tls-trust-dns = ["blocking-http-transport-reqwest", "reqwest-for-configuration-only/rustls-tls", "reqwest-for-configuration-only/trust-dns"]
 ## Stacks with `blocking-http-transport-reqwest` and enables `https://` via the `native-tls` crate.
-blocking-http-transport-reqwest-native-tls = ["blocking-http-transport-reqwest", "reqwest-for-configuration-only/default-tls" ]
+blocking-http-transport-reqwest-native-tls = ["blocking-http-transport-reqwest", "reqwest-for-configuration-only/default-tls"]
 
 
 #! #### Performance
@@ -186,11 +186,11 @@ pack-cache-lru-dynamic = ["gix-pack/pack-cache-lru-dynamic"]
 
 ## Activate other features that maximize performance, like usage of threads, `zlib-ng` and access to caching in object databases.
 ## Note that some platforms might suffer from compile failures, which is when `max-performance-safe` should be used.
-max-performance = [ "max-performance-safe", "zlib-ng", "fast-sha1" ]
+max-performance = ["max-performance-safe", "zlib-ng", "fast-sha1"]
 
 ## If enabled, use assembly versions of sha1 on supported platforms.
 ## This might cause compile failures as well which is why it can be turned off separately.
-fast-sha1 = [ "gix-features/fast-sha1" ]
+fast-sha1 = ["gix-features/fast-sha1"]
 
 ## Use the C-based zlib-ng backend, which can compress and decompress significantly faster.
 ## Note that this will cause duplicate symbol errors if the application also depends on `zlib` - use `zlib-ng-compat` in that case.
@@ -215,7 +215,7 @@ zlib-stock = ["gix-features/zlib-stock"]
 verbose-object-parsing-errors = ["gix-object/verbose-object-parsing-errors"]
 
 ## Data structures implement `serde::Serialize` and `serde::Deserialize`.
-serde = [   "dep:serde",
+serde = ["dep:serde",
     "gix-pack/serde",
     "gix-object/serde",
     "gix-protocol?/serde",
@@ -286,7 +286,7 @@ gix-hashtable = { version = "^0.5.1", path = "../gix-hashtable" }
 gix-commitgraph = { version = "^0.24.1", path = "../gix-commitgraph" }
 gix-pathspec = { version = "^0.7.0", path = "../gix-pathspec", optional = true }
 gix-submodule = { version = "^0.9.0", path = "../gix-submodule", optional = true }
-gix-status = { version = "^0.6.0", path = "../gix-status", optional = true }
+gix-status = { version = "^0.6.0", path = "../gix-status", optional = true, features = ["worktree-rewrites"] }
 gix-command = { version = "^0.3.5", path = "../gix-command", optional = true }
 
 gix-worktree-stream = { version = "^0.10.0", path = "../gix-worktree-stream", optional = true }
@@ -301,7 +301,7 @@ prodash = { workspace = true, optional = true, features = ["progress-tree"] }
 once_cell = "1.14.0"
 signal-hook = { version = "0.3.9", default-features = false, optional = true }
 thiserror = "1.0.26"
-serde = { version = "1.0.114", optional = true, default-features = false, features = ["derive"]}
+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/src/config/cache/access.rs

@@ -112,7 +112,7 @@ impl Cache {
     }
 
     #[cfg(feature = "blob-diff")]
-    pub(crate) fn diff_renames(&self) -> Result<Option<crate::diff::Rewrites>, crate::diff::new_rewrites::Error> {
+    pub(crate) fn diff_renames(&self) -> Result<Option<gix_diff::Rewrites>, crate::diff::new_rewrites::Error> {
         self.diff_renames
             .get_or_try_init(|| crate::diff::new_rewrites(&self.resolved, self.lenient_config))
             .copied()

gix/src/lib.rs

@@ -133,8 +133,6 @@ pub use gix_ref as refs;
 pub use gix_refspec as refspec;
 pub use gix_revwalk as revwalk;
 pub use gix_sec as sec;
-#[cfg(feature = "status")]
-pub use gix_status as status;
 pub use gix_tempfile as tempfile;
 pub use gix_trace as trace;
 pub use gix_traverse as traverse;
@@ -317,6 +315,10 @@ pub mod init;
 /// Not to be confused with 'status'.
 pub mod state;
 
+///
+#[cfg(feature = "status")]
+pub mod status;
+
 ///
 pub mod shallow;
 

gix/src/pathspec.rs

@@ -67,6 +67,13 @@ impl<'repo> Pathspec<'repo> {
             )?,
         )?;
         let cache = needs_cache.then(make_attributes).transpose()?;
+
+        gix_trace::debug!(
+            longest_prefix = ?search.longest_common_directory(),
+            prefix_dir = ?search.prefix_directory(),
+            patterns = ?search.patterns().map(gix_pathspec::Pattern::path).collect::<Vec<_>>()
+        );
+
         Ok(Self {
             repo,
             search,

gix/src/repository/dirwalk.rs

@@ -9,7 +9,7 @@ pub enum Error {
     #[error(transparent)]
     Walk(#[from] gix_dir::walk::Error),
     #[error("A working tree is required to perform a directory walk")]
-    MissinWorkDir,
+    MissingWorkDir,
     #[error(transparent)]
     Excludes(#[from] config::exclude_stack::Error),
     #[error(transparent)]
@@ -57,7 +57,7 @@ impl Repository {
         delegate: &mut dyn gix_dir::walk::Delegate,
     ) -> Result<Outcome<'_>, Error> {
         let _span = gix_trace::coarse!("gix::dirwalk");
-        let workdir = self.work_dir().ok_or(Error::MissinWorkDir)?;
+        let workdir = self.work_dir().ok_or(Error::MissingWorkDir)?;
         let mut excludes = self.excludes(
             index,
             None,
@@ -70,11 +70,6 @@ impl Repository {
             index,
             crate::worktree::stack::state::attributes::Source::WorktreeThenIdMapping,
         )?;
-        gix_trace::debug!(
-            longest_prefix = ?pathspec.search.longest_common_directory(),
-            prefix_dir = ?pathspec.search.prefix_directory(),
-            patterns = ?pathspec.search.patterns().map(gix_pathspec::Pattern::path).collect::<Vec<_>>()
-        );
 
         let git_dir_realpath =
             crate::path::realpath_opts(self.git_dir(), self.current_dir(), crate::path::realpath::MAX_SYMLINKS)?;