# [PATCH 1/2] branch: exit status now reflects if branch listing finds an error 3 messages from 2010-06-04 to 2010-06-04. Participants: Simo Melenius. Thread: https://gitlist.dev/t/24003 ## Simo Melenius, 2010-06-04 09:48 Subject: Re: [PATCH 2/2] branch: don't fail listing branches if one of the commits wasn't found Message-ID: URL: https://gitlist.dev/e/AANLkTilGOtScr7AyF_Dsm5ItTm2QBv_aPdBXRUDH29Ad%40mail.gmail.com In-Reply-To: <20100604024344.GB26311@progeny.tock> ``` On 4 June 2010 05:43, Jonathan Nieder wrote: > Except as pointed out by Sverre[1], >  Reviewed-by: Jonathan Nieder Thanks, I'll now resend the polished versions of these patches. Simo -- () Today is the car of the cdr of your life. /\ http://arc.pasp.de/ ``` ## Simo Melenius, 2010-06-04 09:50 Subject: [PATCH 1/2] branch: exit status now reflects if branch listing finds an error Message-ID: <1275645011-28795-1-git-send-email-simo.melenius@iki.fi> URL: https://gitlist.dev/e/1275645011-28795-1-git-send-email-simo.melenius%40iki.fi In-Reply-To: ``` If some refs could not be read when listing branches, this can now be observed in the exit status of the "git branch" command. Signed-off-by: Simo Melenius --- builtin/branch.c | 25 +++++++++++++++++++------ 1 files changed, 19 insertions(+), 6 deletions(-) diff --git a/builtin/branch.c b/builtin/branch.c index 6cf7e72..72a486c 100644 --- a/builtin/branch.c +++ b/builtin/branch.c @@ -257,9 +257,15 @@ static char *resolve_symref(const char *src, const char *prefix) return xstrdup(dst); } +struct append_ref_cb { + struct ref_list *ref_list; + int ret; +}; + static int append_ref(const char *refname, const unsigned char *sha1, int flags, void *cb_data) { - struct ref_list *ref_list = (struct ref_list*)(cb_data); + struct append_ref_cb *cb = (struct append_ref_cb*)(cb_data); + struct ref_list *ref_list = cb->ref_list; struct ref_item *newitem; struct commit *commit; int kind, i; @@ -293,8 +299,10 @@ static int append_ref(const char *refname, const unsigned char *sha1, int flags, commit = NULL; if (ref_list->verbose || ref_list->with_commit || merge_filter != NO_FILTER) { commit = lookup_commit_reference_gently(sha1, 1); - if (!commit) - return error("branch '%s' does not point at a commit", refname); + if (!commit) { + cb->ret = error("branch '%s' does not point at a commit", refname); + return cb->ret; + } /* Filter with with_commit if specified */ if (!is_descendant_of(commit, ref_list->with_commit)) @@ -484,9 +492,10 @@ static void show_detached(struct ref_list *ref_list) } } -static void print_ref_list(int kinds, int detached, int verbose, int abbrev, struct commit_list *with_commit) +static int print_ref_list(int kinds, int detached, int verbose, int abbrev, struct commit_list *with_commit) { int i; + struct append_ref_cb cb; struct ref_list ref_list; memset(&ref_list, 0, sizeof(ref_list)); @@ -496,7 +505,9 @@ static void print_ref_list(int kinds, int detached, int verbose, int abbrev, str ref_list.with_commit = with_commit; if (merge_filter != NO_FILTER) init_revisions(&ref_list.revs, NULL); - for_each_rawref(append_ref, &ref_list); + cb.ref_list = &ref_list; + cb.ret = 0; + for_each_rawref(append_ref, &cb); if (merge_filter != NO_FILTER) { struct commit *filter; filter = lookup_commit_reference_gently(merge_filter_ref, 0); @@ -527,6 +538,8 @@ static void print_ref_list(int kinds, int detached, int verbose, int abbrev, str } free_ref_list(&ref_list); + + return cb.ret; } static void rename_branch(const char *oldname, const char *newname, int force) @@ -679,7 +692,7 @@ int cmd_branch(int argc, const char **argv, const char *prefix) if (delete) return delete_branches(argc, argv, delete > 1, kinds); else if (argc == 0) - print_ref_list(kinds, detached, verbose, abbrev, with_commit); + return print_ref_list(kinds, detached, verbose, abbrev, with_commit); else if (rename && (argc == 1)) rename_branch(head, argv[0], rename > 1); else if (rename && (argc == 2)) -- 1.7.0.4 ``` ## Simo Melenius, 2010-06-04 09:50 Subject: [PATCH 2/2] branch: don't fail listing branches if one of the commits wasn't found Message-ID: <1275645011-28795-2-git-send-email-simo.melenius@iki.fi> URL: https://gitlist.dev/e/1275645011-28795-2-git-send-email-simo.melenius%40iki.fi In-Reply-To: ``` When listing branches with ref lookups, if one of the known raw refs doesn't point to a commit then "git branch" would return error(), terminating the whole for_each_rawref() iteration and possibly hiding any remaining refs. Signed-off-by: Simo Melenius --- builtin/branch.c | 5 ++++- 1 files changed, 4 insertions(+), 1 deletions(-) diff --git a/builtin/branch.c b/builtin/branch.c index 72a486c..2371ca0 100644 --- a/builtin/branch.c +++ b/builtin/branch.c @@ -301,7 +301,7 @@ static int append_ref(const char *refname, const unsigned char *sha1, int flags, commit = lookup_commit_reference_gently(sha1, 1); if (!commit) { cb->ret = error("branch '%s' does not point at a commit", refname); - return cb->ret; + return 0; } /* Filter with with_commit if specified */ @@ -539,6 +539,9 @@ static int print_ref_list(int kinds, int detached, int verbose, int abbrev, stru free_ref_list(&ref_list); + if (cb.ret) + error("some refs could not be read"); + return cb.ret; } -- 1.7.0.4 ```