remote.c: report specific errors from branch_get_upstream
When the previous commit introduced the branch_get_upstream helper, there was one call-site that could not be converted: the one in sha1_name.c, which gives detailed error messages for each possible failure. Let's teach the helper to optionally report these specific errors. This lets us convert another callsite, and means we can use the helper in other locations that want to give the same error messages. The logic and error messages come straight from sha1_name.c, with the exception that we start each error with a lowercase letter, as is our usual style (note that a few tests need updated as a result). Signed-off-by: Jeff King <peff@peff.net> Signed-off-by: Junio C Hamano <gitster@pobox.com>
This commit is contained in:
parent
a9f9f8cc1f
commit
3a429d0af3
@ -123,7 +123,7 @@ static int branch_merged(int kind, const char *name,
|
|||||||
|
|
||||||
if (kind == REF_LOCAL_BRANCH) {
|
if (kind == REF_LOCAL_BRANCH) {
|
||||||
struct branch *branch = branch_get(name);
|
struct branch *branch = branch_get(name);
|
||||||
const char *upstream = branch_get_upstream(branch);
|
const char *upstream = branch_get_upstream(branch, NULL);
|
||||||
unsigned char sha1[20];
|
unsigned char sha1[20];
|
||||||
|
|
||||||
if (upstream &&
|
if (upstream &&
|
||||||
|
@ -664,7 +664,7 @@ static void populate_value(struct refinfo *ref)
|
|||||||
continue;
|
continue;
|
||||||
branch = branch_get(ref->refname + 11);
|
branch = branch_get(ref->refname + 11);
|
||||||
|
|
||||||
refname = branch_get_upstream(branch);
|
refname = branch_get_upstream(branch, NULL);
|
||||||
if (!refname)
|
if (!refname)
|
||||||
continue;
|
continue;
|
||||||
} else if (starts_with(name, "color:")) {
|
} else if (starts_with(name, "color:")) {
|
||||||
|
@ -1632,7 +1632,7 @@ int cmd_cherry(int argc, const char **argv, const char *prefix)
|
|||||||
break;
|
break;
|
||||||
default:
|
default:
|
||||||
current_branch = branch_get(NULL);
|
current_branch = branch_get(NULL);
|
||||||
upstream = branch_get_upstream(current_branch);
|
upstream = branch_get_upstream(current_branch, NULL);
|
||||||
if (!upstream) {
|
if (!upstream) {
|
||||||
fprintf(stderr, _("Could not find a tracked"
|
fprintf(stderr, _("Could not find a tracked"
|
||||||
" remote branch, please"
|
" remote branch, please"
|
||||||
|
33
remote.c
33
remote.c
@ -1705,10 +1705,35 @@ int branch_merge_matches(struct branch *branch,
|
|||||||
return refname_match(branch->merge[i]->src, refname);
|
return refname_match(branch->merge[i]->src, refname);
|
||||||
}
|
}
|
||||||
|
|
||||||
const char *branch_get_upstream(struct branch *branch)
|
__attribute((format (printf,2,3)))
|
||||||
|
static const char *error_buf(struct strbuf *err, const char *fmt, ...)
|
||||||
{
|
{
|
||||||
if (!branch || !branch->merge || !branch->merge[0])
|
if (err) {
|
||||||
return NULL;
|
va_list ap;
|
||||||
|
va_start(ap, fmt);
|
||||||
|
strbuf_vaddf(err, fmt, ap);
|
||||||
|
va_end(ap);
|
||||||
|
}
|
||||||
|
return NULL;
|
||||||
|
}
|
||||||
|
|
||||||
|
const char *branch_get_upstream(struct branch *branch, struct strbuf *err)
|
||||||
|
{
|
||||||
|
if (!branch)
|
||||||
|
return error_buf(err, _("HEAD does not point to a branch"));
|
||||||
|
if (!branch->merge || !branch->merge[0] || !branch->merge[0]->dst) {
|
||||||
|
if (!ref_exists(branch->refname))
|
||||||
|
return error_buf(err, _("no such branch: '%s'"),
|
||||||
|
branch->name);
|
||||||
|
if (!branch->merge)
|
||||||
|
return error_buf(err,
|
||||||
|
_("no upstream configured for branch '%s'"),
|
||||||
|
branch->name);
|
||||||
|
return error_buf(err,
|
||||||
|
_("upstream branch '%s' not stored as a remote-tracking branch"),
|
||||||
|
branch->merge[0]->src);
|
||||||
|
}
|
||||||
|
|
||||||
return branch->merge[0]->dst;
|
return branch->merge[0]->dst;
|
||||||
}
|
}
|
||||||
|
|
||||||
@ -1921,7 +1946,7 @@ int stat_tracking_info(struct branch *branch, int *num_ours, int *num_theirs)
|
|||||||
int rev_argc;
|
int rev_argc;
|
||||||
|
|
||||||
/* Cannot stat unless we are marked to build on top of somebody else. */
|
/* Cannot stat unless we are marked to build on top of somebody else. */
|
||||||
base = branch_get_upstream(branch);
|
base = branch_get_upstream(branch, NULL);
|
||||||
if (!base)
|
if (!base)
|
||||||
return 0;
|
return 0;
|
||||||
|
|
||||||
|
6
remote.h
6
remote.h
@ -222,8 +222,12 @@ int branch_merge_matches(struct branch *, int n, const char *);
|
|||||||
* Return the fully-qualified refname of the tracking branch for `branch`.
|
* Return the fully-qualified refname of the tracking branch for `branch`.
|
||||||
* I.e., what "branch@{upstream}" would give you. Returns NULL if no
|
* I.e., what "branch@{upstream}" would give you. Returns NULL if no
|
||||||
* upstream is defined.
|
* upstream is defined.
|
||||||
|
*
|
||||||
|
* If `err` is not NULL and no upstream is defined, a more specific error
|
||||||
|
* message is recorded there (if the function does not return NULL, then
|
||||||
|
* `err` is not touched).
|
||||||
*/
|
*/
|
||||||
const char *branch_get_upstream(struct branch *branch);
|
const char *branch_get_upstream(struct branch *branch, struct strbuf *err);
|
||||||
|
|
||||||
/* Flags to match_refs. */
|
/* Flags to match_refs. */
|
||||||
enum match_refs_flags {
|
enum match_refs_flags {
|
||||||
|
25
sha1_name.c
25
sha1_name.c
@ -1059,27 +1059,16 @@ static const char *get_upstream_branch(const char *branch_buf, int len)
|
|||||||
{
|
{
|
||||||
char *branch = xstrndup(branch_buf, len);
|
char *branch = xstrndup(branch_buf, len);
|
||||||
struct branch *upstream = branch_get(*branch ? branch : NULL);
|
struct branch *upstream = branch_get(*branch ? branch : NULL);
|
||||||
|
struct strbuf err = STRBUF_INIT;
|
||||||
|
const char *ret;
|
||||||
|
|
||||||
/*
|
|
||||||
* Upstream can be NULL only if branch refers to HEAD and HEAD
|
|
||||||
* points to something different than a branch.
|
|
||||||
*/
|
|
||||||
if (!upstream)
|
|
||||||
die(_("HEAD does not point to a branch"));
|
|
||||||
if (!upstream->merge || !upstream->merge[0]->dst) {
|
|
||||||
if (!ref_exists(upstream->refname))
|
|
||||||
die(_("No such branch: '%s'"), branch);
|
|
||||||
if (!upstream->merge) {
|
|
||||||
die(_("No upstream configured for branch '%s'"),
|
|
||||||
upstream->name);
|
|
||||||
}
|
|
||||||
die(
|
|
||||||
_("Upstream branch '%s' not stored as a remote-tracking branch"),
|
|
||||||
upstream->merge[0]->src);
|
|
||||||
}
|
|
||||||
free(branch);
|
free(branch);
|
||||||
|
|
||||||
return upstream->merge[0]->dst;
|
ret = branch_get_upstream(upstream, &err);
|
||||||
|
if (!ret)
|
||||||
|
die("%s", err.buf);
|
||||||
|
|
||||||
|
return ret;
|
||||||
}
|
}
|
||||||
|
|
||||||
static int interpret_upstream_mark(const char *name, int namelen,
|
static int interpret_upstream_mark(const char *name, int namelen,
|
||||||
|
@ -150,7 +150,7 @@ test_expect_success 'branch@{u} works when tracking a local branch' '
|
|||||||
|
|
||||||
test_expect_success 'branch@{u} error message when no upstream' '
|
test_expect_success 'branch@{u} error message when no upstream' '
|
||||||
cat >expect <<-EOF &&
|
cat >expect <<-EOF &&
|
||||||
fatal: No upstream configured for branch ${sq}non-tracking${sq}
|
fatal: no upstream configured for branch ${sq}non-tracking${sq}
|
||||||
EOF
|
EOF
|
||||||
error_message non-tracking@{u} 2>actual &&
|
error_message non-tracking@{u} 2>actual &&
|
||||||
test_i18ncmp expect actual
|
test_i18ncmp expect actual
|
||||||
@ -158,7 +158,7 @@ test_expect_success 'branch@{u} error message when no upstream' '
|
|||||||
|
|
||||||
test_expect_success '@{u} error message when no upstream' '
|
test_expect_success '@{u} error message when no upstream' '
|
||||||
cat >expect <<-EOF &&
|
cat >expect <<-EOF &&
|
||||||
fatal: No upstream configured for branch ${sq}master${sq}
|
fatal: no upstream configured for branch ${sq}master${sq}
|
||||||
EOF
|
EOF
|
||||||
test_must_fail git rev-parse --verify @{u} 2>actual &&
|
test_must_fail git rev-parse --verify @{u} 2>actual &&
|
||||||
test_i18ncmp expect actual
|
test_i18ncmp expect actual
|
||||||
@ -166,7 +166,7 @@ test_expect_success '@{u} error message when no upstream' '
|
|||||||
|
|
||||||
test_expect_success 'branch@{u} error message with misspelt branch' '
|
test_expect_success 'branch@{u} error message with misspelt branch' '
|
||||||
cat >expect <<-EOF &&
|
cat >expect <<-EOF &&
|
||||||
fatal: No such branch: ${sq}no-such-branch${sq}
|
fatal: no such branch: ${sq}no-such-branch${sq}
|
||||||
EOF
|
EOF
|
||||||
error_message no-such-branch@{u} 2>actual &&
|
error_message no-such-branch@{u} 2>actual &&
|
||||||
test_i18ncmp expect actual
|
test_i18ncmp expect actual
|
||||||
@ -183,7 +183,7 @@ test_expect_success '@{u} error message when not on a branch' '
|
|||||||
|
|
||||||
test_expect_success 'branch@{u} error message if upstream branch not fetched' '
|
test_expect_success 'branch@{u} error message if upstream branch not fetched' '
|
||||||
cat >expect <<-EOF &&
|
cat >expect <<-EOF &&
|
||||||
fatal: Upstream branch ${sq}refs/heads/side${sq} not stored as a remote-tracking branch
|
fatal: upstream branch ${sq}refs/heads/side${sq} not stored as a remote-tracking branch
|
||||||
EOF
|
EOF
|
||||||
error_message bad-upstream@{u} 2>actual &&
|
error_message bad-upstream@{u} 2>actual &&
|
||||||
test_i18ncmp expect actual
|
test_i18ncmp expect actual
|
||||||
|
Loading…
Reference in New Issue
Block a user