On Thu, Apr 4, 2013 at 10:50 AM, Junio C Hamano <gitster@xxxxxxxxx> wrote:
> Felipe Contreras <felipe.contreras@xxxxxxxxx> writes:
>
>> From: Dusty Phillips <dusty@xxxxxxxx>
>>
>> Signed-off-by: Felipe Contreras <felipe.contreras@xxxxxxxxx>
>> ---
>> contrib/remote-helpers/git-remote-hg | 4 ++++
>> 1 file changed, 4 insertions(+)
>>
>> diff --git a/contrib/remote-helpers/git-remote-hg b/contrib/remote-helpers/git-remote-hg
>> index 56b3641..d82eb2d 100755
>> --- a/contrib/remote-helpers/git-remote-hg
>> +++ b/contrib/remote-helpers/git-remote-hg
>> @@ -625,6 +625,10 @@ def parse_commit(parser):
>> if merge_mark:
>> get_merge_files(repo, p1, p2, files)
>>
>> + # Check if the ref is supposed to be a named branch
>> + if ref.startswith('refs/heads/branches/'):
>> + extra['branch'] = ref.rpartition('/')[2]
>> +
>
> Is this meant to cut everything after "refs/heads/branches/", or cut
> at the last slash? I know rpartition does the latter, but I was
> wondering if we see "refs/heads/branches/foo/bar" as its input here.
Good catch, it should be the former.
--
Felipe Contreras
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html