From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!not-for-mail From: Eli Zaretskii Newsgroups: gmane.emacs.devel Subject: Re: VC mode and git Date: Wed, 08 Apr 2015 11:15:21 +0300 Message-ID: <83k2xnghnq.fsf@gnu.org> References: <83twx2xoc8.fsf@gnu.org> <20150331085055.GA2871@acm.fritz.box> <87zj6tiko1.fsf@uwakimon.sk.tsukuba.ac.jp> <20150331104935.GB2871@acm.fritz.box> <87y4mdi7tj.fsf@uwakimon.sk.tsukuba.ac.jp> <20150331214347.GH2871@acm.fritz.box> <20150401103225.GA2633@acm.fritz.box> <87h9t080gx.fsf@javad.com> <83384jsx3o.fsf@gnu.org> <83pp7nrfdn.fsf@gnu.org> <83a8yqr226.fsf@gnu.org> <831tk2qvz5.fsf@gnu.org> <87384ii26v.fsf@uwakimon.sk.tsukuba.ac.jp> <83wq1tptvp.fsf@gnu.org> <87pp7lhc9h.fsf@uwakimon.sk.tsukuba.ac.jp> <83sichpqe9.fsf@gnu.org> <87ioddglu6.fsf@uwakimon.sk.tsukuba.ac.jp> <83a8yoq56m.fsf@gnu.org> <87384ghm1a.fsf@uwakimon.sk.tsukuba.ac.jp> <837ftspcis.fsf@gnu.org> <551FA115.5090400@gmx.at> <834mowp7cj.fsf@gnu.org> <55200A71.9040902@gmx.at> <83oan3onki.fsf@gnu.org> <55229F01.6030002@gmx.at> <837ftnj258.fsf@gnu.org> <5524D321.1080300@gmx> Reply-To: Eli Zaretskii NNTP-Posting-Host: plane.gmane.org X-Trace: ger.gmane.org 1428480933 30439 80.91.229.3 (8 Apr 2015 08:15:33 GMT) X-Complaints-To: usenet@ger.gmane.org NNTP-Posting-Date: Wed, 8 Apr 2015 08:15:33 +0000 (UTC) Cc: emacs-devel@gnu.org To: martin rudalics Original-X-From: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Wed Apr 08 10:15:24 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 1Yfl8t-0007bb-D8 for ged-emacs-devel@m.gmane.org; Wed, 08 Apr 2015 10:15:23 +0200 Original-Received: from localhost ([::1]:51410 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Yfl8s-0002XN-HW for ged-emacs-devel@m.gmane.org; Wed, 08 Apr 2015 04:15:22 -0400 Original-Received: from eggs.gnu.org ([2001:4830:134:3::10]:57830) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Yfl8o-0002XA-Mz for emacs-devel@gnu.org; Wed, 08 Apr 2015 04:15:19 -0400 Original-Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Yfl8k-0007XM-B6 for emacs-devel@gnu.org; Wed, 08 Apr 2015 04:15:18 -0400 Original-Received: from mtaout23.012.net.il ([80.179.55.175]:53079) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Yfl8k-0007Wm-34 for emacs-devel@gnu.org; Wed, 08 Apr 2015 04:15:14 -0400 Original-Received: from conversion-daemon.a-mtaout23.012.net.il by a-mtaout23.012.net.il (HyperSendmail v2007.08) id <0NMH00A00APQCB00@a-mtaout23.012.net.il> for emacs-devel@gnu.org; Wed, 08 Apr 2015 11:15:11 +0300 (IDT) Original-Received: from HOME-C4E4A596F7 ([87.69.4.28]) by a-mtaout23.012.net.il (HyperSendmail v2007.08) with ESMTPA id <0NMH00ASUAXA9V50@a-mtaout23.012.net.il>; Wed, 08 Apr 2015 11:15:11 +0300 (IDT) Sun-Java-System-SMTP-Warning: Lines longer than SMTP allows found and truncated. In-reply-to: <5524D321.1080300@gmx.at> X-012-Sender: halo1@inter.net.il X-detected-operating-system: by eggs.gnu.org: Solaris 10 X-Received-From: 80.179.55.175 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:185141 Archived-At: > Date: Wed, 08 Apr 2015 09:05:05 +0200 > From: martin rudalics > CC: emacs-devel@gnu.org > > >> It is a good idea to examine what you are about to push, before > >> actually doing so, because fixing mistakes before pushing is much > >> easier (see the next section). To do that, use the command git diff > >> origin/master. If you want to show your unpushed commits with their > >> commit log messages, use git show origin/master.. instead. If you only > >> have one local commit you want to push, just git show is enough. > >> > >> And here I would try to tell that the outputs of plain 'git diff' and > >> 'git status' are different from their outputs before the commit. > > > > In the new version, "git diff" is no longer mentioned, > > Hmm... I still see > > We recommend invoking git status and git diff to view the changes > which will be committed, before invoking git commit -a. Yes, but in that place it's TRT. This is only suggested before the commit, with the explicit purpose of showing what's changed and what will be committed, so why do we have to explain how the output after the commit is going to be different? > > and "git > > status" was never mentioned before, so do you still think we need to > > say something about that? > > Maybe it's only me. git adopts a principle epitomized by, for example, > > Typically you would want comparison with the latest commit, so if you > do not give , it defaults to HEAD. > > which is completely unintuitive IMO. Why should the output of two "git > diff"s differ just because I committed something in between? We no longer suggest using diff except _before_ a commit, precisely because of this issue. > > But the full text says this: > > > > This merge could fail due to conflicts between your changes and > > changes by others in the same portions of the same files. The > > conflicts could be in changes you have already committed locally, or > > in uncommitted changes. > > > > The second sentence refers to uncommitted changes. Is it really > > important to tell that in this case Git will not even start a merge? > > How will that help the reader/user when they are in this situation? > > Agreed. One nitpick still: > > Now you have conflicts due to local committed changes, described below. > > I would say > > Now you may have conflicts due to local committed changes, described below. > > instead. Done.