diff options
author | Teddy Wing | 2020-10-17 15:27:55 +0200 |
---|---|---|
committer | Teddy Wing | 2020-10-17 15:27:55 +0200 |
commit | b6b137d1815a131372956fbdd3edba7d6bafc3e6 (patch) | |
tree | 7e5643b9554c262c3243dd2a2f49b3bfefa4730e /src | |
parent | a9608c229c49bc408855ba4c1e321e2e2327163a (diff) | |
download | git-todo-b6b137d1815a131372956fbdd3edba7d6bafc3e6.tar.bz2 |
write_since(): Output file paths relative to the current directory
Previously, the file paths of TODO lines in the output were relative to
the repository root. When you're in a subdirectory, though, this makes
them harder to work with, as they're relative paths, but aren't relative
to the current working directory.
Change the file path output so that paths are relative to the current
directory instead of the repo root.
Diffstat (limited to 'src')
-rw-r--r-- | src/lib.rs | 29 |
1 files changed, 28 insertions, 1 deletions
@@ -17,6 +17,7 @@ #![warn(rust_2018_idioms)] use std::io::Write; +use std::path::{Path, PathBuf}; use git2::{DiffOptions, Repository, Tree}; use thiserror::Error; @@ -71,7 +72,7 @@ impl Todos<'_> { write!( write_to, "{}:{}:{}", - path.display(), + relative_path(self.repo, path).display(), line_number, l, ).expect("write error"); @@ -95,3 +96,29 @@ impl Todos<'_> { Ok(master.get().peel_to_tree()?) } } + +/// Get `path` relative to the current directory. +/// +/// Attempt to remove the current directory prefix from `path` to turn it into a +/// relative path from the current directory. +fn relative_path(repo: &Repository, path: &Path) -> PathBuf { + let workdir = match repo.workdir() { + Some(d) => d, + None => return path.to_path_buf(), + }; + + let current_dir = match std::env::current_dir() { + Ok(d) => d, + Err(_) => return path.to_path_buf(), + }; + + let current_dir_relative = match current_dir.strip_prefix(workdir) { + Ok(d) => d, + Err(_) => return path.to_path_buf(), + }; + + match path.strip_prefix(current_dir_relative) { + Ok(d) => d.to_path_buf(), + Err(_) => path.to_path_buf(), + } +} |