From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751734Ab1KEEha (ORCPT ); Sat, 5 Nov 2011 00:37:30 -0400 Received: from b-pb-sasl-quonix.pobox.com ([208.72.237.35]:63655 "EHLO smtp.pobox.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750921Ab1KEEh2 (ORCPT ); Sat, 5 Nov 2011 00:37:28 -0400 DomainKey-Signature: a=rsa-sha1; c=nofws; d=pobox.com; h=from:to:cc :subject:references:date:in-reply-to:message-id:mime-version :content-type; q=dns; s=sasl; b=KN4+/kiXuoVTiQhIdLef6wOZK65R+rW4 1+3tpzn7Wlr0kXn456u9dQ21tiiHe8e7VwgPDJ2CIGO4W1YSM/ndIJuwRfb4NTiU z12U0EBxIEFK6/iZ34uQWbvnzOBttsgq/Qed2RToOUInUPTr4yotFVl9gVYyu4W7 2rar5KKuMFA= From: Junio C Hamano To: Linus Torvalds Cc: git@vger.kernel.org, Shawn Pearce , James Bottomley , Jeff Garzik , Andrew Morton , linux-ide@vger.kernel.org, LKML Subject: Re: [git patches] libata updates, GPG signed (but see admin notes) References: <20111026202235.GA20928@havoc.gtf.org> <1319969101.5215.20.camel@dabdike> <1320049150.8283.19.camel@dabdike> <7vy5w1ow90.fsf@alter.siamese.dyndns.org> <7vwrbjlj5r.fsf@alter.siamese.dyndns.org> <7vk47jld5s.fsf@alter.siamese.dyndns.org> <7vlirvbq47.fsf@alter.siamese.dyndns.org> <7v39e3bn1n.fsf@alter.siamese.dyndns.org> Date: Fri, 04 Nov 2011 21:37:25 -0700 In-Reply-To: (Linus Torvalds's message of "Fri, 4 Nov 2011 16:10:59 -0700") Message-ID: <7vipmz9oca.fsf@alter.siamese.dyndns.org> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/23.2 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii X-Pobox-Relay-ID: DD2407A0-0767-11E1-B6C1-9DB42E706CDE-77302942!b-pb-sasl-quonix.pobox.com Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Linus Torvalds writes: > However - exactly beause git apparently makes it do that "Merge commit > " message, I suspect we've peeled things too early and too much. We've > peeled it so early that once again something thinks it's a commit, not > a tag. And you are right. I am working on a larger series that should sit on top of 89587fa (Split GPG interface into its own helper library, 2011-09-07), which is the first commit in jc/signed-commit topic. -- >8 -- Subject: [PATCH] merge: notice local merging of tags and keep it unwrapped Signed-off-by: Junio C Hamano --- builtin/merge.c | 5 +++++ 1 files changed, 5 insertions(+), 0 deletions(-) diff --git a/builtin/merge.c b/builtin/merge.c index f61b367..ce3b4f8 100644 --- a/builtin/merge.c +++ b/builtin/merge.c @@ -428,6 +428,11 @@ static void merge_name(const char *remote, struct strbuf *msg) sha1_to_hex(branch_head), remote); goto cleanup; } + if (!prefixcmp(found_ref, "refs/tags/")) { + strbuf_addf(msg, "%s\t\ttag '%s' of .\n", + sha1_to_hex(branch_head), remote); + goto cleanup; + } if (!prefixcmp(found_ref, "refs/remotes/")) { strbuf_addf(msg, "%s\t\tremote-tracking branch '%s' of .\n", sha1_to_hex(branch_head), remote); -- 1.7.8.rc0.108.g71b5ec