Re: [PATCH 7/9] remote.c: teach branch_get() to treat symrefs other than HEAD
- From
Ramkumar Ramachandra <artagnon@gmail.com>
- Date
- May 2, 2013, 15:43 UTC
- Message-ID
- <CALkWK0=eC2M6b5Dh9kR8TZEXRGYnYQtrpb5PUed4nDSStOCTpw@mail.gmail.com>
- In-Reply-To
- <CALkWK0=FbbCferG+q4qymvsm-t_rfSBRZfp7t3kO=z-EF5QYuA@mail.gmail.com>
Ramkumar Ramachandra wrote:
> So sorry about this, but this breaks some tests in t1507 > (rev-parse-upstream). I'm looking into this now. > > In the meantime, reviewers can focus on the commit message.
So, it turns out that some callers expect it to read_config().
If we're still adamant about not touching branch_get(), the alternative is to call resolve_ref_unsafe() before the branch_get() call in interpret_branch_name(). We'll still retain the H@{u}, but we'll lose the 'git branch -u ram/master M'.