From owner-freebsd-stable Mon Oct 14 9: 3:21 2002 Delivered-To: freebsd-stable@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 5CA8D37B401 for ; Mon, 14 Oct 2002 09:03:19 -0700 (PDT) Received: from apollo.backplane.com (apollo.backplane.com [216.240.41.2]) by mx1.FreeBSD.org (Postfix) with ESMTP id CB5F943EAA for ; Mon, 14 Oct 2002 09:03:18 -0700 (PDT) (envelope-from dillon@apollo.backplane.com) Received: from apollo.backplane.com (localhost [127.0.0.1]) by apollo.backplane.com (8.12.5/8.12.4) with ESMTP id g9EG33PQ061865; Mon, 14 Oct 2002 09:03:03 -0700 (PDT) (envelope-from dillon@apollo.backplane.com) Received: (from dillon@localhost) by apollo.backplane.com (8.12.5/8.12.4/Submit) id g9EG31Av061862; Mon, 14 Oct 2002 09:03:01 -0700 (PDT) (envelope-from dillon) Date: Mon, 14 Oct 2002 09:03:01 -0700 (PDT) From: Matthew Dillon Message-Id: <200210141603.g9EG31Av061862@apollo.backplane.com> To: Thomas Quinot Cc: "Crist J. Clark" , Trish Lynch , Adrian Wontroba , chris scott , freebsd-stable@FreeBSD.ORG Subject: Re: Patch #2 (Re: Ifconfig config of gif tunnels) References: <20021013172810.A19177@titus.hanley.stade.co.uk> <20021013130138.G523-100000@femme.sapphite.org> <20021013214642.GB90169@blossom.cjclark.org> <200210132154.g9DLsUmc057065@apollo.backplane.com> <20021013225509.GA90575@blossom.cjclark.org> <200210140228.g9E2SRlA058696@apollo.backplane.com> <20021014091111.GA66883@melusine.cuivre.fr.eu.org> Sender: owner-freebsd-stable@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.ORG :Should this be committed, it should be *prominently* documented that :there is a very significant change in the user interface: when :mergemaster does not drop you into a more(1) prompt: : - in the current situation, the user is guaranteed that he's seeing : the complete diff on his screen; : - with your patch he might see only a part of that. :So if the user wants to make is merging decision based on the diff, he :cannot do that anymore without additional manual intervention. : :Plus, for those who like/are used to the current behaviour, it would be :nice if your change could be tied to a command line switch (I don't mind :for the new behaviour to be the default, but I think the old one should :be left as an option.) : :Thomas. Well, I certainly agree with the prominent display. I don't think a switch is appropriate, though, since the whole idea of using mergemaster is to make things as easy as possible for the user. For the very first diff it does we could add a comment "Note that this and all initial diffs may be truncated. Use 'v' to view the whole diff". It should be noted that mergemaster already truncates diff sections when you use the 'm'erge command, so the truncation is not really all that new an idea. I do add a "..." at the end to make it more obvious. Also, it should be noted that 'manual intervention' is more like putting your eggs in one basket or another. The fact that you are sometimes left in the pager and sometimes not means that you are essentially doing a manual operation (i.e. you have to look at what state it is in to figure out what to type next), on every single diff. By removing that requirement we are introducing another potential manual operation... a person having to type 'v' to see the whole diff, but as I am sure most people know there are literallly only two or three files for which you might need to see the entire diff. So the tradeoff is well in the favor of having to do less manual intervention with these changes. -Matt Matthew Dillon To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-stable" in the body of the message