From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!not-for-mail From: Dan Nicolaescu Newsgroups: gmane.emacs.devel Subject: Re: vc-dir default directory: repository root? Date: Tue, 20 Jan 2015 15:06:43 -0500 Message-ID: References: <54B0CC33.2050100@dancol.org> <54BC60A8.6040401@yandex.ru> <54BD9071.5070802@yandex.ru> NNTP-Posting-Host: plane.gmane.org Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii X-Trace: ger.gmane.org 1421784410 22729 80.91.229.3 (20 Jan 2015 20:06:50 GMT) X-Complaints-To: usenet@ger.gmane.org NNTP-Posting-Date: Tue, 20 Jan 2015 20:06:50 +0000 (UTC) Cc: Daniel Colascione , Emacs developers To: Dmitry Gutov Original-X-From: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Tue Jan 20 21:06:49 2015 Return-path: Envelope-to: ged-emacs-devel@m.gmane.org Original-Received: from lists.gnu.org ([208.118.235.17]) by plane.gmane.org with esmtp (Exim 4.69) (envelope-from ) id 1YDf4b-0002xq-38 for ged-emacs-devel@m.gmane.org; Tue, 20 Jan 2015 21:06:49 +0100 Original-Received: from localhost ([::1]:45472 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YDf4Z-0002Bh-Sd for ged-emacs-devel@m.gmane.org; Tue, 20 Jan 2015 15:06:47 -0500 Original-Received: from eggs.gnu.org ([2001:4830:134:3::10]:41152) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YDf4X-0002Bb-1m for emacs-devel@gnu.org; Tue, 20 Jan 2015 15:06:46 -0500 Original-Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1YDf4V-0005r1-OI for emacs-devel@gnu.org; Tue, 20 Jan 2015 15:06:44 -0500 Original-Received: from fencepost.gnu.org ([2001:4830:134:3::e]:36965) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YDf4V-0005qv-KS for emacs-devel@gnu.org; Tue, 20 Jan 2015 15:06:43 -0500 Original-Received: from dann by fencepost.gnu.org with local (Exim 4.71) (envelope-from ) id 1YDf4V-0000qc-6F; Tue, 20 Jan 2015 15:06:43 -0500 In-Reply-To: <54BD9071.5070802@yandex.ru> (Dmitry Gutov's message of "Tue, 20 Jan 2015 01:17:05 +0200") User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/23.3 (gnu/linux) X-detected-operating-system: by eggs.gnu.org: Error: Malformed IPv6 address (bad octet value). X-Received-From: 2001:4830:134:3::e X-BeenThere: emacs-devel@gnu.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: "Emacs development discussions." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Original-Sender: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Xref: news.gmane.org gmane.emacs.devel:181487 Archived-At: Dmitry Gutov writes: > On 01/19/2015 05:47 PM, Dan Nicolaescu wrote: > >> CVS does not work on the root level. >> Even in SVN, it's ok and desirable to just look at a subdirectory. > > Isn't that an implementation detail, as far as the current discussion > is concerned? Git can also show only the status of files in a > subdirectory, if you ask it. > >> vc-dir has used the current directory since the beginning, I haven't >> seen any request to change it until now, it doesn't seem like changing >> the default behavior after so many years when there were no requests to >> do it is a good idea just because someone sent a patch to change it. > > You've missed http://debbugs.gnu.org/12492, then. This patch also > constitutes one request, obviously. > > And you shouldn't forget that VC is not the most popular VCS package > among advanced Emacs users, Magit is. We're playing catch-up. > >> vc-hg displays the "Root" in the vc-dir header. The intention with that >> was to make it a button that would display vc-dir on the top level >> directory. >> All backends (especially vc-git) should implement that Root header and >> implement a button for it... > > That's an awkward proposal. To have to open vc-dir, then move my hand > to the mouse, or point to the button, then push it... Buttons can be pressed with the keyboard. That was just another way to get to the root... > Root is a good default, so vc-dir buffer for it should be easier to > create than for a subdirectory. It looks like root is a personal preference, there's still not much in terms of solid arguments of why to change a long time default, and the patch doesn't even give a way to get the current behavior, which most people like. You can make C-u C-x v d start vc-dir in the root directory. Or you can put the root directory in the history so that when C-x v d prompts for a directory it's just a M-p away. Or highlight the root in the path shown by C-x v d. These can be accomplished without removing the current behavior.