From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 18990 invoked by alias); 8 Mar 2012 10:40:05 -0000 Received: (qmail 18979 invoked by uid 22791); 8 Mar 2012 10:40:03 -0000 X-SWARE-Spam-Status: No, hits=-6.8 required=5.0 tests=AWL,BAYES_00,RCVD_IN_DNSWL_HI,SPF_HELO_PASS,T_RP_MATCHES_RCVD X-Spam-Check-By: sourceware.org Received: from mx1.redhat.com (HELO mx1.redhat.com) (209.132.183.28) by sourceware.org (qpsmtpd/0.43rc1) with ESMTP; Thu, 08 Mar 2012 10:39:42 +0000 Received: from int-mx11.intmail.prod.int.phx2.redhat.com (int-mx11.intmail.prod.int.phx2.redhat.com [10.5.11.24]) by mx1.redhat.com (8.14.4/8.14.4) with ESMTP id q28AdfdD022555 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=OK) for ; Thu, 8 Mar 2012 05:39:42 -0500 Received: from [127.0.0.1] (ovpn01.gateway.prod.ext.phx2.redhat.com [10.5.9.1]) by int-mx11.intmail.prod.int.phx2.redhat.com (8.14.4/8.14.4) with ESMTP id q28AdeMc003106; Thu, 8 Mar 2012 05:39:41 -0500 Message-ID: <4F588C6C.7060602@redhat.com> Date: Thu, 08 Mar 2012 10:40:00 -0000 From: Pedro Alves User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:10.0.1) Gecko/20120216 Thunderbird/10.0.1 MIME-Version: 1.0 To: Jan Kratochvil CC: gdb-patches@sourceware.org Subject: Re: [patch 1/2] libunwind/ia64: Rename libunwind-frame.[ch] References: <20120304215728.GA2763@host2.jankratochvil.net> <4F55C2BB.7090205@redhat.com> <20120308091528.GA14321@host2.jankratochvil.net> In-Reply-To: <20120308091528.GA14321@host2.jankratochvil.net> Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Mailing-List: contact gdb-patches-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: gdb-patches-owner@sourceware.org X-SW-Source: 2012-03/txt/msg00257.txt.bz2 On 03/08/2012 09:15 AM, Jan Kratochvil wrote: > On Tue, 06 Mar 2012 08:54:35 +0100, Pedro Alves wrote: >> On 03/04/2012 09:57 PM, Jan Kratochvil wrote: >>> this is just a rename, it breaks the build, but it makes the changes >>> reviewable in [patch 2/2]. It would be checked-in as a single commit. >> >> BTW, there's a different way to split this so that you don't mix >> the rename with other changes to the same files, and so that >> git log sees through pure file renames without trouble. > 'git log' never see a rename while 'git log -M' sees the rename even if those > changes are in single commit as those changes are really very small: Thanks, but well, I said "git log", but I meant "git in general". > diff --git a/gdb/libunwind-frame.h b/gdb/ia64-libunwind-tdep.h > similarity index 93% > rename from gdb/libunwind-frame.h > rename to gdb/ia64-libunwind-tdep.h > index a6b3c34..221bbd2 100644 > --- a/gdb/libunwind-frame.h > +++ b/gdb/ia64-libunwind-tdep.h > @@ -1,4 +1,4 @@ > -/* Frame unwinder for frames with libunwind frame information. > +/* Frame unwinder for ia64 frames with libunwind frame information. > [...] > > So I do not find a need to use multiple commits into the (CVS) repository. Hmm, okay, that's smart. I do think rename-only in one commit/patch, and then changes as separate commits in the repository is anyway a good principle to aim for (making sure the build doesn't break in the process). But given that, I don't care so much. > I was posting it this way so that one can easily apply the series by 'patch' > while still being able to see the changes (in [patch 2/2]). There's nothing in my suggestion that'd prevent that, given the changes are always separate from the rename. > If one can assume every mailing list user has GIT anyway the repository > absolutely no longer makes sense in CVS. There's more to the cvs/git conversion than what mailing list users are using, so let's keep that out of the discussion. (That'd actually make my argument stronger. Making pure renames in separate commits increases the chances of tools other than git also understanding a rename.) Anyway, please go ahead as you prefer. -- Pedro Alves