[PATCH v2 6/6] builtin/repo: add progress meter for stats
- From
Justin Tobler <jltobler@gmail.com>
- Date
- Sep 24, 2025, 21:24 UTC
- Message-ID
- <20250924212426.2930029-7-jltobler@gmail.com>
- In-Reply-To
- <20250924212426.2930029-1-jltobler@gmail.com>
When using the stats subcommand for git-repo(1), evaluating a repository may take some time depending on its shape. Add a progress meter to provide feedback to the user about what is happening. The progress meter is enabled by default when the command is executed from a tty. It can also be explicitly enabled/disabled via the --[no-]progress option.
Signed-off-by: Justin Tobler <jltobler@gmail.com> --- builtin/repo.c | 46 ++++++++++++++++++++++++++++++++++++++++------ 1 file changed, 40 insertions(+), 6 deletions(-)
diff --git a/builtin/repo.c b/builtin/repo.c index fe7d43f78e..fdc8af92dc 100644 --- a/builtin/repo.c +++ b/builtin/repo.c @@ -4,6 +4,7 @@ #include "environment.h" #include "parse-options.h" #include "path-walk.h" +#include "progress.h" #include "quote.h" #include "ref-filter.h" #include "refs.h" @@ -344,8 +345,14 @@ static void stats_keyvalue_print(struct repo_stats *stats, char key_delim, strbuf_release(&buf); } -static void stats_count_references(struct ref_stats *stats, struct ref_array *refs) +static void stats_count_references(struct ref_stats *stats, struct ref_array *refs, + struct repository *repo, int show_progress) { + struct progress *progress = NULL; + + if (show_progress) + progress = start_progress(repo, _("Counting references"), refs->nr); + for (int i = 0; i < refs->nr; i++) { struct ref_array_item *ref = refs->items[i]; @@ -365,13 +372,24 @@ static void stats_count_references(struct ref_stats *stats, struct ref_array *re default: BUG("unexpected reference type"); } + + display_progress(progress, i + 1); } + + stop_progress(&progress); } +struct count_objects_data { + struct object_stats *stats; + struct progress *progress; +}; + static int count_objects(const char *path UNUSED, struct oid_array *oids, enum object_type type, void *cb_data) { - struct object_stats *stats = cb_data; + struct count_objects_data *data = cb_data; + struct object_stats *stats = data->stats; + size_t object_count; switch (type) { case OBJ_TAG: @@ -390,17 +408,24 @@ static int count_objects(const char *path UNUSED, struct oid_array *oids, BUG("invalid object type"); } + object_count = stats->tags + stats->commits + stats->trees + stats->blobs; + display_progress(data->progress, object_count); + return 0; } static void stats_count_objects(struct object_stats *stats, - struct ref_array *refs, struct rev_info *revs) + struct ref_array *refs, struct rev_info *revs, + struct repository *repo, int show_progress) { struct path_walk_info info = PATH_WALK_INFO_INIT; + struct count_objects_data data = { + .stats = stats, + }; info.revs = revs; info.path_fn = count_objects; - info.path_fn_data = stats; + info.path_fn_data = &data; for (int i = 0; i < refs->nr; i++) { struct ref_array_item *ref = refs->items[i]; @@ -417,8 +442,12 @@ static void stats_count_objects(struct object_stats *stats, } } + if (show_progress) + data.progress = start_progress(repo, _("Counting Objects"), 0); + walk_objects_by_path(&info); path_walk_info_clear(&info); + stop_progress(&data.progress); } static int cmd_repo_stats(int argc, const char **argv, const char *prefix, @@ -432,10 +461,12 @@ static int cmd_repo_stats(int argc, const char **argv, const char *prefix, struct repo_stats stats = { 0 }; struct ref_array refs = { 0 }; struct rev_info revs; + int show_progress = -1; struct option options[] = { OPT_CALLBACK_F(0, "format", &format, N_("format"), N_("output format"), PARSE_OPT_NONEG, parse_format_cb), + OPT_BOOL(0, "progress", &show_progress, N_("show progress")), OPT_END() }; @@ -444,8 +475,11 @@ static int cmd_repo_stats(int argc, const char **argv, const char *prefix, if (filter_refs(&refs, &filter, FILTER_REFS_REGULAR)) die(_("unable to filter refs")); - stats_count_references(&stats.refs, &refs); - stats_count_objects(&stats.objects, &refs, &revs); + if (show_progress < 0) + show_progress = isatty(2); + + stats_count_references(&stats.refs, &refs, repo, show_progress); + stats_count_objects(&stats.objects, &refs, &revs, repo, show_progress); switch (format) { case FORMAT_TABLE:
-- 2.51.0.193.g4975ec3473b