Skip to content

Commit

Permalink
remote.c: add branch_get_push
Browse files Browse the repository at this point in the history
In a triangular workflow, the place you pull from and the
place you push to may be different. As we have
branch_get_upstream for the former, this patch adds
branch_get_push for the latter (and as the former implements
@{upstream}, so will this implement @{push} in a future
patch).

Note that the memory-handling for the return value bears
some explanation. Some code paths require allocating a new
string, and some let us return an existing string. We should
provide a consistent interface to the caller, so it knows
whether to free the result or not.

We could do so by xstrdup-ing any existing strings, and
having the caller always free. But that makes us
inconsistent with branch_get_upstream, so we would prefer to
simply take ownership of the resulting string. We do so by
storing it inside the "struct branch", just as we do with
the upstream refname (in that case we compute it when the
branch is created, but there's no reason not to just fill
it in lazily in this case).

Signed-off-by: Jeff King <peff@peff.net>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
  • Loading branch information
peff authored and gitster committed May 22, 2015
1 parent 979cb24 commit e291c75
Show file tree
Hide file tree
Showing 2 changed files with 95 additions and 0 deletions.
85 changes: 85 additions & 0 deletions remote.c
Expand Up @@ -1744,6 +1744,91 @@ const char *branch_get_upstream(struct branch *branch, struct strbuf *err)
return branch->merge[0]->dst;
}

static const char *tracking_for_push_dest(struct remote *remote,
const char *refname,
struct strbuf *err)
{
char *ret;

ret = apply_refspecs(remote->fetch, remote->fetch_refspec_nr, refname);
if (!ret)
return error_buf(err,
_("push destination '%s' on remote '%s' has no local tracking branch"),
refname, remote->name);
return ret;
}

static const char *branch_get_push_1(struct branch *branch, struct strbuf *err)
{
struct remote *remote;

if (!branch)
return error_buf(err, _("HEAD does not point to a branch"));

remote = remote_get(pushremote_for_branch(branch, NULL));
if (!remote)
return error_buf(err,
_("branch '%s' has no remote for pushing"),
branch->name);

if (remote->push_refspec_nr) {
char *dst;
const char *ret;

dst = apply_refspecs(remote->push, remote->push_refspec_nr,
branch->refname);
if (!dst)
return error_buf(err,
_("push refspecs for '%s' do not include '%s'"),
remote->name, branch->name);

ret = tracking_for_push_dest(remote, dst, err);
free(dst);
return ret;
}

if (remote->mirror)
return tracking_for_push_dest(remote, branch->refname, err);

switch (push_default) {
case PUSH_DEFAULT_NOTHING:
return error_buf(err, _("push has no destination (push.default is 'nothing')"));

case PUSH_DEFAULT_MATCHING:
case PUSH_DEFAULT_CURRENT:
return tracking_for_push_dest(remote, branch->refname, err);

case PUSH_DEFAULT_UPSTREAM:
return branch_get_upstream(branch, err);

case PUSH_DEFAULT_UNSPECIFIED:
case PUSH_DEFAULT_SIMPLE:
{
const char *up, *cur;

up = branch_get_upstream(branch, err);
if (!up)
return NULL;
cur = tracking_for_push_dest(remote, branch->refname, err);
if (!cur)
return NULL;
if (strcmp(cur, up))
return error_buf(err,
_("cannot resolve 'simple' push to a single destination"));
return cur;
}
}

die("BUG: unhandled push situation");
}

const char *branch_get_push(struct branch *branch, struct strbuf *err)
{
if (!branch->push_tracking_ref)
branch->push_tracking_ref = branch_get_push_1(branch, err);
return branch->push_tracking_ref;
}

static int ignore_symref_update(const char *refname)
{
unsigned char sha1[20];
Expand Down
10 changes: 10 additions & 0 deletions remote.h
Expand Up @@ -209,6 +209,8 @@ struct branch {
struct refspec **merge;
int merge_nr;
int merge_alloc;

const char *push_tracking_ref;
};

struct branch *branch_get(const char *name);
Expand All @@ -229,6 +231,14 @@ int branch_merge_matches(struct branch *, int n, const char *);
*/
const char *branch_get_upstream(struct branch *branch, struct strbuf *err);

/**
* Return the tracking branch that corresponds to the ref we would push to
* given a bare `git push` while `branch` is checked out.
*
* The return value and `err` conventions match those of `branch_get_upstream`.
*/
const char *branch_get_push(struct branch *branch, struct strbuf *err);

/* Flags to match_refs. */
enum match_refs_flags {
MATCH_REFS_NONE = 0,
Expand Down

0 comments on commit e291c75

Please sign in to comment.