Re: [PATCH v5 4/4] builtin/branch: strip refs/heads/ using skip_prefix

From: Kaartic Sivaraam <kaartic.sivaraam@gmail.com>
Date: Dec 8, 2017, 17:39 UTC
Message-ID: <b0d7a230-b03e-20a3-1e57-dc42b4cbdda7@gmail.com>
In-Reply-To: <xmqqd13qb079.fsf@gitster.mtv.corp.google.com>

On Friday 08 December 2017 04:44 AM, Junio C Hamano wrote:

Show 10 quoted lines

> Junio C Hamano <gitster@pobox.com> writes:
>>
>> Somehow this fell underneath my radar horizon.  I see v4 and v5 of
>> 4/4 but do not seem to find 1-3/4.  Is this meant to be a standalone
>> patch, or am I expected to already have 1-3 that we already are
>> committed to take?
> 
> Ah, I am guessing that this would apply on top of 1-3/4 in the
> thread with <20171118172648.17918-1-kaartic.sivaraam@gmail.com>
>

You guessed right; at the right time. I was about to ask why this got "out of your radar" in reply to your recent "What's cooking" email :-)

Show 5 quoted lines

> The base of the series seems to predate 16169285 ("Merge branch
> 'jc/branch-name-sanity'", 2017-11-28), so let me see how it looks by
> applying those three plus this one on top of 'master' before that
> point.
>

Let me know if this has terrible conflicts so that I can rebase the series on top of 'master'.

Thanks, Kaartic