From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.io!.POSTED.blaine.gmane.org!not-for-mail From: Dmitry Gutov Newsgroups: gmane.emacs.bugs Subject: bug#55871: Acknowledgement (27.1; vc-git.el log view 'a', 'f', 'd' do not work when following renames) Date: Fri, 15 Dec 2023 16:39:04 +0200 Message-ID: References: <78f97339-2aca-0dbd-4cb4-3532af78a895@yandex.ru> <1b16a642-8a85-e693-66fc-41c37df24908@yandex.ru> <36b48587-a723-73ae-fefb-886d69d04ec8@yandex.ru> <83y1dvobun.fsf@gnu.org> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Injection-Info: ciao.gmane.io; posting-host="blaine.gmane.org:116.202.254.214"; logging-data="39211"; mail-complaints-to="usenet@ciao.gmane.io" User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.13.0 Cc: 55871@debbugs.gnu.org, n.oje.bar@gmail.com To: Eli Zaretskii Original-X-From: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane-mx.org@gnu.org Fri Dec 15 15:40:27 2023 Return-path: Envelope-to: geb-bug-gnu-emacs@m.gmane-mx.org Original-Received: from lists.gnu.org ([209.51.188.17]) by ciao.gmane.io with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1rE9MX-0009xp-S9 for geb-bug-gnu-emacs@m.gmane-mx.org; Fri, 15 Dec 2023 15:40:26 +0100 Original-Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1rE9MG-0002jz-5L; Fri, 15 Dec 2023 09:40:08 -0500 Original-Received: from eggs.gnu.org ([2001:470:142:3::10]) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1rE9MB-0002is-RQ for bug-gnu-emacs@gnu.org; Fri, 15 Dec 2023 09:40:04 -0500 Original-Received: from debbugs.gnu.org ([2001:470:142:5::43]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1rE9MB-000127-Ig for bug-gnu-emacs@gnu.org; Fri, 15 Dec 2023 09:40:03 -0500 Original-Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1rE9MA-0001XA-8l for bug-gnu-emacs@gnu.org; Fri, 15 Dec 2023 09:40:02 -0500 X-Loop: help-debbugs@gnu.org Resent-From: Dmitry Gutov Original-Sender: "Debbugs-submit" Resent-CC: bug-gnu-emacs@gnu.org Resent-Date: Fri, 15 Dec 2023 14:40:02 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 55871 X-GNU-PR-Package: emacs X-GNU-PR-Keywords: patch Original-Received: via spool by 55871-submit@debbugs.gnu.org id=B55871.17026511605831 (code B ref 55871); Fri, 15 Dec 2023 14:40:02 +0000 Original-Received: (at 55871) by debbugs.gnu.org; 15 Dec 2023 14:39:20 +0000 Original-Received: from localhost ([127.0.0.1]:51814 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1rE9LT-0001Vz-T9 for submit@debbugs.gnu.org; Fri, 15 Dec 2023 09:39:20 -0500 Original-Received: from forward501c.mail.yandex.net ([178.154.239.209]:37914) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1rE9LP-0001Vk-Fx for 55871@debbugs.gnu.org; Fri, 15 Dec 2023 09:39:17 -0500 Original-Received: from mail-nwsmtp-smtp-production-main-78.myt.yp-c.yandex.net (mail-nwsmtp-smtp-production-main-78.myt.yp-c.yandex.net [IPv6:2a02:6b8:c12:5c92:0:640:faf9:0]) by forward501c.mail.yandex.net (Yandex) with ESMTP id BE2F760C7E; Fri, 15 Dec 2023 17:39:12 +0300 (MSK) Original-Received: by mail-nwsmtp-smtp-production-main-78.myt.yp-c.yandex.net (smtp/Yandex) with ESMTPSA id AdmNsL9JhW20-ldRCQCL9; Fri, 15 Dec 2023 17:39:12 +0300 X-Yandex-Fwd: 1 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=yandex.ru; s=mail; t=1702651152; bh=tRXm3Znf0RSjvtWqnoxU6isxttAQo624MrR5D7Salzg=; h=In-Reply-To:From:Subject:Message-ID:Cc:References:Date:To; b=j37ahjdaSztoA+V/Mu8Y15rCjmyKrRAscM57+HVTNlQIEm81Gyp6N8PhqH+mJKvPO Ej/vTuJ89FkHL6tG1zKfHS9Dc7JZHQ7ph2xf0s1FJlsY7YzIsQXJKHj6Xbn7Wcbe8E jPsEoGdXP5YryPiN/HSk6cl/VHxESa1nITbuBR4I= Authentication-Results: mail-nwsmtp-smtp-production-main-78.myt.yp-c.yandex.net; dkim=pass header.i=@yandex.ru Original-Received: from compute4.internal (compute4.nyi.internal [10.202.2.44]) by mailauth.nyi.internal (Postfix) with ESMTP id D21E627C0054; Fri, 15 Dec 2023 09:39:09 -0500 (EST) Original-Received: from mailfrontend1 ([10.202.2.162]) by compute4.internal (MEProxy); Fri, 15 Dec 2023 09:39:09 -0500 X-ME-Sender: X-ME-Received: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgedvkedrvddtvddgieejucetufdoteggodetrfdotf fvucfrrhhofhhilhgvmecuhfgrshhtofgrihhlpdfqfgfvpdfurfetoffkrfgpnffqhgen uceurghilhhouhhtmecufedttdenucesvcftvggtihhpihgvnhhtshculddquddttddmne cujfgurhepkfffgggfuffvvehfhfgjtgfgsehtjeertddtfeejnecuhfhrohhmpeffmhhi thhrhicuifhuthhovhcuoegughhuthhovheshigrnhguvgigrdhruheqnecuggftrfgrth htvghrnhepudfhfeeuudevfefgtdeigeetudetgfehleeugeefgfekhefhleejgfehffdt lefhnecuffhomhgrihhnpehgnhhurdhorhhgnecuvehluhhsthgvrhfuihiivgeptdenuc frrghrrghmpehmrghilhhfrhhomhepughguhhtohhvodhmvghsmhhtphgruhhthhhpvghr shhonhgrlhhithihqddufeeffeelleehhedvqddvleegjeejjeejiedqughguhhtohhvpe ephigrnhguvgigrdhruhesfhgrshhtmhgrihhlrdgtohhm X-ME-Proxy: Feedback-ID: ib1d9465d:Fastmail Original-Received: by mail.messagingengine.com (Postfix) with ESMTPA; Fri, 15 Dec 2023 09:39:08 -0500 (EST) Content-Language: en-US In-Reply-To: <83y1dvobun.fsf@gnu.org> X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list X-BeenThere: bug-gnu-emacs@gnu.org List-Id: "Bug reports for GNU Emacs, the Swiss army knife of text editors" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane-mx.org@gnu.org Original-Sender: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane-mx.org@gnu.org Xref: news.gmane.io gmane.emacs.bugs:276270 Archived-At: On 15/12/2023 15:05, Eli Zaretskii wrote: >> Cc: 55871@debbugs.gnu.org >> Date: Fri, 15 Dec 2023 04:01:59 +0200 >> From: Dmitry Gutov >> >> On 14/12/2023 03:23, Dmitry Gutov wrote: >>> On 14/12/2023 02:52, Dmitry Gutov wrote: >>>> but otherwise seems to function well (with potential for future >>>> additions) >>> >>> To clarify: this version only makes 'd' work (not 'f' or 'a'), but the >>> other two are fixed more easily. >> >> Attached is the new revision with combined fix. > > FYI: 'd' and 'f' work with bzr without any changes. To my understanding, Bazaar doesn't really exist in this day and age, so should we pay extra attention to it in this NEWS entry? We could say that the problem is relevant to Git and Hg, and the current solution only helps Git. I'm not sure what's the best phrasing, however, which won't bloat the NEWS entry too much. Though by the time 30 is released someone might implement the 'file-name-changes' handler for Hg too: it might be easy enough with the rest of the solution already in place. One will have to see whether Hg has any special caveats, though, like the one you noted for Bzr below. > 'a' doesn't work > (evidently, "bzr annotate -r REVISION FILE" doesn't work when FILE did > not exist in REVISION, but was renamed by a later revision, and one > needs to run "bzr status -Sr REVISION" and look for the "renamed" > report in the result, which will then provide the previous name). But when we're asking for 'annotate' for a file in some old revision (under old name), it won't be the same revision where it had been renamed, 99% of the time. > (FTR: I used src/unexcoff.c file to test this.) > >> +*** Support for viewing file change history across renames. >> +When a fileset's VC change history ends at a rename, we now print the >> +old name(s) and a button which jumps to their history. Only supported >> +with Git at the moment. > > I think this should at least tell that for files under Bazaar, the VC > change history will always include the renames. Looks like Mercurial > is in the same department as Git? More or less, yes, here's an even older bug report: https://debbugs.gnu.org/13004 > If so, I think the text should say > that this is not supported for Mercurial yet, and that Bazaar shows > the entire history, including renames, by default. Or something like > that. I don't want to make it a sticking point, but according to the wiki entry Monotone also tracks renames. We won't be mentioning it here, will we? I don't think we're going to persuade anybody new to use Bazaar here, and those who currently use it, probably don't pay this issue (the lack of it) much attention. > I also think this entry should mention the relevant VC commands > ("C-x v l" and what else?), since otherwise "VC change history" > is not concrete enough to tell users which command(s) is/are affected. Ok. Just 'C-x v l', with one or more files selected. The remaining 2 vc-print-* commands print the history for all files, so all renames are naturally included. One cannot use 'f' or 'a' on their output, though.