From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933301Ab2AKDEC (ORCPT ); Tue, 10 Jan 2012 22:04:02 -0500 Received: from b-pb-sasl-quonix.pobox.com ([208.72.237.35]:61515 "EHLO smtp.pobox.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756535Ab2AKDEA (ORCPT ); Tue, 10 Jan 2012 22:04:00 -0500 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=NkNiQeYrb4Jn3RY9e06HnhISSE+eLMHe gCzoARFFTr2PsHRVhAtj/825uCGac7D99ldlgzwfQqwUq9X4+8ZGURNF32z7d/iU mvLuxNhzopw/7upGYHFmeSWOUa8hNGY/oIXnVFgROMtmamfTechtm7IqBn+dnKpe 3hvXhl12U+c= From: Junio C Hamano To: Linus Torvalds Cc: Mark Brown , Liam Girdwood , linux-kernel@vger.kernel.org, Git Mailing List Subject: Re: Regulator updates for 3.3 References: <20120109073727.GF22134@opensource.wolfsonmicro.com> <20120110184530.GE7164@opensource.wolfsonmicro.com> <20120110222711.GK7164@opensource.wolfsonmicro.com> <7vmx9v7z1r.fsf@alter.siamese.dyndns.org> Date: Tue, 10 Jan 2012 19:03:57 -0800 In-Reply-To: (Linus Torvalds's message of "Tue, 10 Jan 2012 18:47:55 -0800") Message-ID: <7vehv77xeq.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: E7DF31B0-3C00-11E1-A48E-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: > On Tue, Jan 10, 2012 at 6:28 PM, Junio C Hamano wrote: >> >> It is a non-starter to unconditionally start an editor. > > I really wonder. Because not being default will always lead to really > odd ways of saying "it should have been default, so we'll make up > these complex and arbitrary special rules" (like the ones you were > starting to outline). > > So I really suspect it would be easier and more straightforward to > instead just bite the bullet, and say: > > (a) start an editor by default if both stdin/stdout matched in fstat > and were istty(). > > (b) have some trivial way to disable that default behavior for people > who really want the legacy behavior. And by "trivial" I mean "set the > GIT_LEGACY_MERGE environment variable" or something. > > (c) have a "--no-editor" command line switch so that scripts and/or > users that want to make it explicit (rather than rely on the hacky > legacy workaround) can do so (and a explicit "--editor" switch to > enable people to use a GUI editor even if they aren't on a terminal - > think something IDE environment, whatever). Hrm. Lack of any quoted line other than the first line from my message, together with (c) above, makes me suspect that you did not read beyond the first line before composing this message you are responding to. > Yes, git has been very good about not breaking semantics. But it's > happened before too when it needed to happen. We've had much bigger > breaks (like the whole "git-xyz" to "git xyz" transition, for example, > which broke a lot of scripts). Yes, I am learning from the experience to be cautious ;-) I dunno. You just scrapped the plan for 1.7.10; it may have to be called 2.0 instead.