From mboxrd@z Thu Jan 1 00:00:00 1970 From: Carl Worth Subject: Re: VCS comparison table Date: Tue, 17 Oct 2006 18:44:57 -0700 Message-ID: <87zmbucs86.wl%cworth@cworth.org> References: <9e4733910610140807p633f5660q49dd2d2111c9f5fe@mail.gmail.com> <20061018000026.GH20017@pasky.or.cz> <45357596.8050702@utoronto.ca> <200610180328.31234.jnareb@gmail.com> Mime-Version: 1.0 (generated by SEMI 1.14.6 - "Maruoka") Content-Type: multipart/signed; boundary="pgp-sign-Multipart_Tue_Oct_17_18:44:57_2006-1"; micalg=pgp-sha1; protocol="application/pgp-signature" Content-Transfer-Encoding: 7bit Cc: Aaron Bentley , Petr Baudis , Linus Torvalds , Andreas Ericsson , bazaar-ng@lists.canonical.com, git@vger.kernel.org, Matthieu Moy X-From: git-owner@vger.kernel.org Wed Oct 18 03:47:14 2006 Return-path: Envelope-to: gcvg-git@gmane.org Received: from vger.kernel.org ([209.132.176.167]) by ciao.gmane.org with esmtp (Exim 4.43) id 1Ga0UW-0002yn-GW for gcvg-git@gmane.org; Wed, 18 Oct 2006 03:45:12 +0200 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1750834AbWJRBpE (ORCPT ); Tue, 17 Oct 2006 21:45:04 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1751246AbWJRBpE (ORCPT ); Tue, 17 Oct 2006 21:45:04 -0400 Received: from cworth.org ([217.160.249.188]:22978 "EHLO theworths.org") by vger.kernel.org with ESMTP id S1750834AbWJRBpC (ORCPT ); Tue, 17 Oct 2006 21:45:02 -0400 Received: (qmail 26368 invoked from network); 17 Oct 2006 21:44:59 -0400 Received: from localhost (HELO raht.cworth.org) (127.0.0.1) by localhost with SMTP; 17 Oct 2006 21:44:59 -0400 To: Jakub Narebski In-Reply-To: <200610180328.31234.jnareb@gmail.com> User-Agent: Wanderlust/2.14.0 (Africa) Emacs/21.4 Mule/5.0 (SAKAKI) Sender: git-owner@vger.kernel.org Precedence: bulk X-Mailing-List: git@vger.kernel.org Archived-At: --pgp-sign-Multipart_Tue_Oct_17_18:44:57_2006-1 Content-Type: text/plain; charset=US-ASCII On Wed, 18 Oct 2006 03:28:30 +0200, Jakub Narebski wrote: > > Isn't it easier to review than "bundle", aka. mega-patch? There are even more important reasons to prefer a series of micro-commits over a mega-patch than just ease of merging. In the cairo project, I've often reviewed a single patch and said: "This all looks like perfectly good code and I'd be happy to have it all in the tree. But please rebuild this as a series of independent patches (perhaps along the lines of a, b, c, ...)" I do that not just to make the history "look nice" but because code history is something we _use_ a lot and separate commits for separate actions just make the history so much more usable. We have great tools like bisect to identify commits that introduce bugs. I know that I'd be delighted to see bisect comes back pointing at some minimal commit as causing a bug, (which would make finding the bug so much easier). But it's also been my experience that the largest commits are also the most likely to be the things returned by bisect. Big commits really do introduce bugs more frequently than small commits. Finally, if someone had gone through the useful work to create small, independent changes, (and likely finding and fixing bugs in the process), what a horrible shame it would be to throw away that work and merge it as a single patch, (welcome to the pain of CVS branch merging). Now, I do admit that it is often useful to take the overall view of a patch series being submitted. This is often the case when a patch series is in some sub-module of the code for which I don't have as much direct involvement. In cases like that I will often do review only of the diff between the tips of the mainline and the branch of interest, (or if I trust the maintainer enough, perhaps just the diffstat between the two). But I'm still very glad that what lands in the history is the series of independent changes, and not one mega commit. -Carl --pgp-sign-Multipart_Tue_Oct_17_18:44:57_2006-1 Content-Type: application/pgp-signature Content-Transfer-Encoding: 7bit -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.5 (GNU/Linux) iD8DBQBFNYcZ6JDdNq8qSWgRAuaKAJ9EVwWh2amn0eMNrAb9pGqKD+WCPwCfYFZG osCawWp8d65zxlyHLnh/ulw= =FUBx -----END PGP SIGNATURE----- --pgp-sign-Multipart_Tue_Oct_17_18:44:57_2006-1--