Stefan Kangas <stefankangas@HIDDEN>
to control <at> debbugs.gnu.org.
Full text available.Received: (at 64531) by debbugs.gnu.org; 20 Aug 2023 10:26:20 +0000 From debbugs-submit-bounces <at> debbugs.gnu.org Sun Aug 20 06:26:20 2023 Received: from localhost ([127.0.0.1]:53148 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from <debbugs-submit-bounces <at> debbugs.gnu.org>) id 1qXfdT-0007NM-Ry for submit <at> debbugs.gnu.org; Sun, 20 Aug 2023 06:26:20 -0400 Received: from forward103b.mail.yandex.net ([2a02:6b8:c02:900:1:45:d181:d103]:39906) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from <dgutov@HIDDEN>) id 1qXfdR-0007N7-2Z for 64531 <at> debbugs.gnu.org; Sun, 20 Aug 2023 06:26:18 -0400 Received: from mail-nwsmtp-smtp-production-main-63.sas.yp-c.yandex.net (mail-nwsmtp-smtp-production-main-63.sas.yp-c.yandex.net [IPv6:2a02:6b8:c14:6e01:0:640:627f:0]) by forward103b.mail.yandex.net (Yandex) with ESMTP id 3B31960042; Sun, 20 Aug 2023 13:26:09 +0300 (MSK) Received: by mail-nwsmtp-smtp-production-main-63.sas.yp-c.yandex.net (smtp/Yandex) with ESMTPSA id 6QhH8lTDamI0-pztMNeNM; Sun, 20 Aug 2023 13:26:08 +0300 X-Yandex-Fwd: 1 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=yandex.ru; s=mail; t=1692527168; bh=JRu64OZzocL+/o6a6dMgFnk98ITNI52EDMxJHbjJRlo=; h=In-Reply-To:From:Subject:Message-ID:Cc:References:Date:To; b=doF8qFIjNHpOPQvaj7THAPFxhkn1yNAe2wLcQvsiHez7thPVjDyvZrCxsMXwFT3Ab bPKbcBpYzZJOSDqjpKe5ih5gBVnXvyFstyl2Q2SvUMU34m/CGlwSJYHTZmuI5uEiRv zhU1t8pmIGRB62lGCfW8ulHWrhuwXu09wpE/cW4s= Authentication-Results: mail-nwsmtp-smtp-production-main-63.sas.yp-c.yandex.net; dkim=pass header.i=@yandex.ru Received: from compute4.internal (compute4.nyi.internal [10.202.2.44]) by mailauth.nyi.internal (Postfix) with ESMTP id BB2D027C0054; Sun, 20 Aug 2023 06:26:06 -0400 (EDT) Received: from mailfrontend1 ([10.202.2.162]) by compute4.internal (MEProxy); Sun, 20 Aug 2023 06:26:06 -0400 X-ME-Sender: <xms:PurhZH7X5NFVPyldlD2QLKZdCeoetCQX5__5YXIcUpdeHYqIHxkS-A> <xme:PurhZM4QKSyf6DD2z_BJ17S0BshrJA17yAszFJM1T2u0-5EmtsaNF0b51gWzQGn3Y HxvMdJj9xgil6NwUXo> X-ME-Received: <xmr:PurhZOeT3cvijvFaMSzU5UVAqRHfLa8GBYm50UBOkkF89etHjrpbjdKo0YtXQ6M> X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgedviedruddujedgfedtucetufdoteggodetrfdotf fvucfrrhhofhhilhgvmecuhfgrshhtofgrihhlpdfqfgfvpdfurfetoffkrfgpnffqhgen uceurghilhhouhhtmecufedttdenucesvcftvggtihhpihgvnhhtshculddquddttddmne cujfgurhepkfffgggfuffvvehfhfgjtgfgsehtjeertddtfeejnecuhfhrohhmpeffmhhi thhrhicuifhuthhovhcuoegughhuthhovheshigrnhguvgigrdhruheqnecuggftrfgrth htvghrnheptdffgeegkeelteevtdekleethfeftdduvdegkedtkedujefhfedtveeftdff udevnecuvehluhhsthgvrhfuihiivgeptdenucfrrghrrghmpehmrghilhhfrhhomhepug hguhhtohhvodhmvghsmhhtphgruhhthhhpvghrshhonhgrlhhithihqddufeeffeelleeh hedvqddvleegjeejjeejiedqughguhhtohhvpeephigrnhguvgigrdhruhesfhgrshhtmh grihhlrdgtohhm X-ME-Proxy: <xmx:PurhZILvENCkd0LvxzLoy8UcSpANfVeGMq_MRANlfhx8TPrTUHXmIA> <xmx:PurhZLKKgJK0NlQb9azlT0ZuciTbk3ysvm7osRN5PJCxKVNY_m8Qeg> <xmx:PurhZByWVZbr1lP5xoBkXVn-ZfDvsBnPW5DtyM51_OoKHRNaHngbdQ> <xmx:PurhZBipEc7KQcu0XgjdrusW8ge6ni8wLF0VJ6VW_fcmBJh4z4mg1Q> Feedback-ID: ib1d9465d:Fastmail Received: by mail.messagingengine.com (Postfix) with ESMTPA; Sun, 20 Aug 2023 06:26:05 -0400 (EDT) Message-ID: <43944ccb-8c21-5158-4bb5-31020a10eec8@HIDDEN> Date: Sun, 20 Aug 2023 13:26:02 +0300 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.13.0 Subject: Re: bug#64531: [PATCH v2] Add new command vc-dir-copy-filename-as-kill Content-Language: en-US To: Eli Zaretskii <eliz@HIDDEN> References: <87bkgmq1ra.fsf@HIDDEN> <83jzvadabq.fsf@HIDDEN> <87bkfhy7fr.fsf_-_@HIDDEN> <83a5un1mn3.fsf@HIDDEN> <21b04273-e7e5-cbe5-cd95-9484a89a4b64@HIDDEN> <838ra6xm9n.fsf@HIDDEN> From: Dmitry Gutov <dgutov@HIDDEN> In-Reply-To: <838ra6xm9n.fsf@HIDDEN> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit X-Spam-Score: 1.8 (+) X-Spam-Report: Spam detection software, running on the system "debbugs.gnu.org", has NOT identified this incoming email as spam. The original message has been attached to this so you can view it or label similar future email. If you have any questions, see the administrator of that system for details. Content preview: On 20/08/2023 09:26, Eli Zaretskii wrote: >> Date: Sun, 20 Aug 2023 02:33:59 +0300 >> Cc: 64531 <at> debbugs.gnu.org >> From: Dmitry Gutov >> >> The difference from dired-copy-filename-as-kill in handing [...] Content analysis details: (1.8 points, 10.0 required) pts rule name description ---- ---------------------- -------------------------------------------------- -0.0 SPF_PASS SPF: sender matches SPF record 3.6 RCVD_IN_SBL_CSS RBL: Received via a relay in Spamhaus SBL-CSS [2a02:6b8:c02:900:1:45:d181:d103 listed in] [zen.spamhaus.org] -0.7 RCVD_IN_DNSWL_LOW RBL: Sender listed at https://www.dnswl.org/, low trust [2a02:6b8:c02:900:1:45:d181:d103 listed in] [list.dnswl.org] 0.0 FREEMAIL_FROM Sender email is commonly abused enduser mail provider (dgutov[at]yandex.ru) 0.0 SPF_HELO_NONE SPF: HELO does not publish an SPF Record -1.0 NICE_REPLY_A Looks like a legit reply (A) X-Debbugs-Envelope-To: 64531 Cc: ivan-p-sokolov@HIDDEN, 64531 <at> debbugs.gnu.org X-BeenThere: debbugs-submit <at> debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list List-Id: <debbugs-submit.debbugs.gnu.org> List-Unsubscribe: <https://debbugs.gnu.org/cgi-bin/mailman/options/debbugs-submit>, <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=unsubscribe> List-Archive: <https://debbugs.gnu.org/cgi-bin/mailman/private/debbugs-submit/> List-Post: <mailto:debbugs-submit <at> debbugs.gnu.org> List-Help: <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=help> List-Subscribe: <https://debbugs.gnu.org/cgi-bin/mailman/listinfo/debbugs-submit>, <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=subscribe> Errors-To: debbugs-submit-bounces <at> debbugs.gnu.org Sender: "Debbugs-submit" <debbugs-submit-bounces <at> debbugs.gnu.org> X-Spam-Score: 0.8 (/) On 20/08/2023 09:26, Eli Zaretskii wrote: >> Date: Sun, 20 Aug 2023 02:33:59 +0300 >> Cc: 64531 <at> debbugs.gnu.org >> From: Dmitry Gutov <dgutov@HIDDEN> >> >> The difference from dired-copy-filename-as-kill in handing of the prefix >> argument looks unfortunate (as a principle), but I'd rather leave it for >> you to judge. Perhaps changing the latter command's interface would be a >> better choice; I'm not sure why it works that way. > > What difference is that? I must be missing something, since it sounds > like both commands use the argument to select absolute or relative > file names? One of them mandates M-0 for that effect. >> Implementation difference: instead of going through (format "%S"), this >> one uses shell-quote-argument. Is that good enough for Windows systems? > > It depends on how the result will be used. But maybe we should do the > same as dired-copy-filename-as-kill, just for consistency of the UX? Maybe. I imagine it might be used in a shell, but indeed there are different options. >> And one more difference which we might want to see fixed: >> >> (When there's a single file, no quoting is done.) >> >> It seems like, at least, in some usage scenarios users would prefer to >> have copied filename not quoted. E.g. for subsequent use with find-file >> or... something similar. I don't really use this command all that much, >> so maybe I'm mistaken here, though. > > There will always be cases when quoting gets in the way, but hopefully > they are rare. The important thing, IMO, is that quoting is useful > for many/most uses of the file names in VC commands. Is it? When using 'M-!'? Maybe. I think consistency between 'w' commands is somewhat important, though.
bug-gnu-emacs@HIDDEN:bug#64531; Package emacs.
Full text available.Received: (at 64531) by debbugs.gnu.org; 20 Aug 2023 06:26:08 +0000 From debbugs-submit-bounces <at> debbugs.gnu.org Sun Aug 20 02:26:08 2023 Received: from localhost ([127.0.0.1]:52781 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from <debbugs-submit-bounces <at> debbugs.gnu.org>) id 1qXbt1-0000fI-SB for submit <at> debbugs.gnu.org; Sun, 20 Aug 2023 02:26:08 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:60180) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from <eliz@HIDDEN>) id 1qXbsz-0000ej-Jt for 64531 <at> debbugs.gnu.org; Sun, 20 Aug 2023 02:26:06 -0400 Received: from fencepost.gnu.org ([2001:470:142:3::e]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from <eliz@HIDDEN>) id 1qXbss-0004sa-SB; Sun, 20 Aug 2023 02:25:58 -0400 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=gnu.org; s=fencepost-gnu-org; h=References:Subject:In-Reply-To:To:From:Date: mime-version; bh=e17UqTMjl/xTmR4ds2KxkuZ6SqiKWwGXqulbXZotm4s=; b=H84EQ6/hffdk /O73Ots/U02Ble6fmV1+4xYegIf5OkU8F4v9Wq+PWhd9Iis1WDFVRghWgRfTqp9CTXjp4ESSoYgHE /k6V/uOHKHOQKWc+kK+IQSkbcgtDUwpJQN4JhHEnE71OJUVi86TSAVAIehGaknnUA6xIZgNPA4GFq aTZc/O7+lmP+atRu/4dxZlIN5A564kaca3cop3h2EF/9+QKGiDP1H7HjN5eZafCOANoHEOJhs41Lq ElVFoxSBdwXeafsOWUZOQt/5mZ2wy65i9ECDf8a15VuPcG4UkxJSCt0GW/zjFT5rQsJHyss9aUqWL rpZJtap9PV9iBXhZxFQ1kA==; Date: Sun, 20 Aug 2023 09:26:12 +0300 Message-Id: <838ra6xm9n.fsf@HIDDEN> From: Eli Zaretskii <eliz@HIDDEN> To: Dmitry Gutov <dgutov@HIDDEN> In-Reply-To: <21b04273-e7e5-cbe5-cd95-9484a89a4b64@HIDDEN> (message from Dmitry Gutov on Sun, 20 Aug 2023 02:33:59 +0300) Subject: Re: bug#64531: [PATCH v2] Add new command vc-dir-copy-filename-as-kill References: <87bkgmq1ra.fsf@HIDDEN> <83jzvadabq.fsf@HIDDEN> <87bkfhy7fr.fsf_-_@HIDDEN> <83a5un1mn3.fsf@HIDDEN> <21b04273-e7e5-cbe5-cd95-9484a89a4b64@HIDDEN> X-Spam-Score: -2.3 (--) X-Debbugs-Envelope-To: 64531 Cc: 64531 <at> debbugs.gnu.org, ivan-p-sokolov@HIDDEN X-BeenThere: debbugs-submit <at> debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list List-Id: <debbugs-submit.debbugs.gnu.org> List-Unsubscribe: <https://debbugs.gnu.org/cgi-bin/mailman/options/debbugs-submit>, <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=unsubscribe> List-Archive: <https://debbugs.gnu.org/cgi-bin/mailman/private/debbugs-submit/> List-Post: <mailto:debbugs-submit <at> debbugs.gnu.org> List-Help: <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=help> List-Subscribe: <https://debbugs.gnu.org/cgi-bin/mailman/listinfo/debbugs-submit>, <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=subscribe> Errors-To: debbugs-submit-bounces <at> debbugs.gnu.org Sender: "Debbugs-submit" <debbugs-submit-bounces <at> debbugs.gnu.org> X-Spam-Score: -3.3 (---) > Date: Sun, 20 Aug 2023 02:33:59 +0300 > Cc: 64531 <at> debbugs.gnu.org > From: Dmitry Gutov <dgutov@HIDDEN> > > The difference from dired-copy-filename-as-kill in handing of the prefix > argument looks unfortunate (as a principle), but I'd rather leave it for > you to judge. Perhaps changing the latter command's interface would be a > better choice; I'm not sure why it works that way. What difference is that? I must be missing something, since it sounds like both commands use the argument to select absolute or relative file names? > Implementation difference: instead of going through (format "%S"), this > one uses shell-quote-argument. Is that good enough for Windows systems? It depends on how the result will be used. But maybe we should do the same as dired-copy-filename-as-kill, just for consistency of the UX? > And one more difference which we might want to see fixed: > > (When there's a single file, no quoting is done.) > > It seems like, at least, in some usage scenarios users would prefer to > have copied filename not quoted. E.g. for subsequent use with find-file > or... something similar. I don't really use this command all that much, > so maybe I'm mistaken here, though. There will always be cases when quoting gets in the way, but hopefully they are rare. The important thing, IMO, is that quoting is useful for many/most uses of the file names in VC commands. Is it?
bug-gnu-emacs@HIDDEN:bug#64531; Package emacs.
Full text available.Received: (at 64531) by debbugs.gnu.org; 20 Aug 2023 06:19:57 +0000 From debbugs-submit-bounces <at> debbugs.gnu.org Sun Aug 20 02:19:57 2023 Received: from localhost ([127.0.0.1]:52771 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from <debbugs-submit-bounces <at> debbugs.gnu.org>) id 1qXbn2-0000Tg-Kz for submit <at> debbugs.gnu.org; Sun, 20 Aug 2023 02:19:56 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:42202) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from <eliz@HIDDEN>) id 1qXbn0-0000TS-6q for 64531 <at> debbugs.gnu.org; Sun, 20 Aug 2023 02:19:54 -0400 Received: from fencepost.gnu.org ([2001:470:142:3::e]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from <eliz@HIDDEN>) id 1qXbms-0003hR-IA; Sun, 20 Aug 2023 02:19:46 -0400 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=gnu.org; s=fencepost-gnu-org; h=References:Subject:In-Reply-To:To:From:Date: mime-version; bh=7vbldJsbH8yK0WH8F4G2LKnVZlIRXQqx4RI0zaveRwQ=; b=SoLGhDgoDz8d 6DeNx1zqiVqfx7QmmGKiIrJ6W/Rajj2wFufiZYS9cNWcG02IQyBVa3t0l4D66Ef6xiTLVaPBxplgL 2GJb51quQPv8Wqsib7tKUIYorh8pxQmG3LEaWs1CYod/0C+mlRpSmccQ+eXC1aSo+iaYjrQi9egXg fizSr49H21fkA5dfqR+mZUJuqHDf0hWYH7+YAHKmnuAA+9X3bmAx4z8ik+V9PcyEckZvudEcVvT4A 1s0unTLu2PE4DBJnd/TOkvWiWhcRoIstUkJBqrMKwRyRl3C1mjed1NpqAv8MT58Rq9Nxub6Qvz6vT ZfBKT5bhO0TheF39r1BRJA==; Date: Sun, 20 Aug 2023 09:20:01 +0300 Message-Id: <83bkf2xmjy.fsf@HIDDEN> From: Eli Zaretskii <eliz@HIDDEN> To: Ivan Sokolov <ivan-p-sokolov@HIDDEN> In-Reply-To: <87sf8eeie7.fsf@HIDDEN> (message from Ivan Sokolov on Sun, 20 Aug 2023 02:12:32 +0300) Subject: Re: bug#64531: [PATCH v2] Add new command vc-dir-copy-filename-as-kill References: <87bkgmq1ra.fsf@HIDDEN> <83jzvadabq.fsf@HIDDEN> <87bkfhy7fr.fsf_-_@HIDDEN> <83a5un1mn3.fsf@HIDDEN> <87sf8eeie7.fsf@HIDDEN> X-Spam-Score: -2.3 (--) X-Debbugs-Envelope-To: 64531 Cc: 64531 <at> debbugs.gnu.org, dgutov@HIDDEN X-BeenThere: debbugs-submit <at> debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list List-Id: <debbugs-submit.debbugs.gnu.org> List-Unsubscribe: <https://debbugs.gnu.org/cgi-bin/mailman/options/debbugs-submit>, <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=unsubscribe> List-Archive: <https://debbugs.gnu.org/cgi-bin/mailman/private/debbugs-submit/> List-Post: <mailto:debbugs-submit <at> debbugs.gnu.org> List-Help: <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=help> List-Subscribe: <https://debbugs.gnu.org/cgi-bin/mailman/listinfo/debbugs-submit>, <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=subscribe> Errors-To: debbugs-submit-bounces <at> debbugs.gnu.org Sender: "Debbugs-submit" <debbugs-submit-bounces <at> debbugs.gnu.org> X-Spam-Score: -3.3 (---) > From: Ivan Sokolov <ivan-p-sokolov@HIDDEN> > Cc: Dmitry Gutov <dgutov@HIDDEN>, 64531 <at> debbugs.gnu.org > Date: Sun, 20 Aug 2023 02:12:32 +0300 > > Eli Zaretskii <eliz@HIDDEN> writes: > > >> From: Ivan Sokolov <ivan-p-sokolov@HIDDEN> > >> Cc: 64531 <at> debbugs.gnu.org > >> Date: Tue, 08 Aug 2023 16:44:56 +0300 > >> > >> Eli Zaretskii <eliz@HIDDEN> writes: > >> > >> > However, new commands should be called out in NEWS, and commands that > >> > are important enough should also be mentioned in the user manual. > >> > >> Added entry in NEWS, but not in the user manual, there is no such for > >> vc-dir afaik. > >> > >> >> +(defun vc-dir-copy-filename-as-kill (&optional absolutep) > >> >> + "Copy names of marked files (or file under cursor) into the kill ring. > >> > > >> > This sentence is too general: it doesn't mention VC Dir, although > >> > AFAIU it is specific to vc-dir. > >> > >> Fixed. > > > > Dmitry, is this good to go in? > > > > I still think it should be mention in the Emacs user manual, where we > > document vc-dir. > > Hi, Eli! > > I am working on the 3rd version of the patch with all the necessary > documentation and changes to the command which I will explain in the > cover letter. Not sure when I'll finish it though. Good to hear, and TIA. There's no rush, so take your time. (I will ping again if it takes too long IMO ;-)
bug-gnu-emacs@HIDDEN:bug#64531; Package emacs.
Full text available.Received: (at 64531) by debbugs.gnu.org; 19 Aug 2023 23:34:16 +0000 From debbugs-submit-bounces <at> debbugs.gnu.org Sat Aug 19 19:34:16 2023 Received: from localhost ([127.0.0.1]:52451 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from <debbugs-submit-bounces <at> debbugs.gnu.org>) id 1qXVSS-0006Kz-Ax for submit <at> debbugs.gnu.org; Sat, 19 Aug 2023 19:34:16 -0400 Received: from forward102c.mail.yandex.net ([2a02:6b8:c03:500:1:45:d181:d102]:46304) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from <dgutov@HIDDEN>) id 1qXVSQ-0006Km-Iv for 64531 <at> debbugs.gnu.org; Sat, 19 Aug 2023 19:34:15 -0400 Received: from mail-nwsmtp-smtp-production-main-81.myt.yp-c.yandex.net (mail-nwsmtp-smtp-production-main-81.myt.yp-c.yandex.net [IPv6:2a02:6b8:c12:530c:0:640:7b0d:0]) by forward102c.mail.yandex.net (Yandex) with ESMTP id A720860014; Sun, 20 Aug 2023 02:34:06 +0300 (MSK) Received: by mail-nwsmtp-smtp-production-main-81.myt.yp-c.yandex.net (smtp/Yandex) with ESMTPSA id 4YtrXPPDeGk0-z7cdEueC; Sun, 20 Aug 2023 02:34:05 +0300 X-Yandex-Fwd: 1 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=yandex.ru; s=mail; t=1692488046; bh=HjztpwqWjDfPiCpqRlVFxkMPyeI0d/UbjQMpt5TFZU0=; h=In-Reply-To:From:Subject:Message-ID:Cc:References:Date:To; b=hjolQarVmfGCyiVqx38cj25ATE70BCr6GPN9DFaxz6AqPSy08sV91ZEQ8/kWO2vOa Bk0gHvPl3SVJYNdEnq8bzeyThqU4AO+NqkuCIdAqPnffMdxdMgdPkbJwoPGvCmbxrk SUsObQEKNtHgXModwCsD5fYqohgh+jry2rp6Oymc= Authentication-Results: mail-nwsmtp-smtp-production-main-81.myt.yp-c.yandex.net; dkim=pass header.i=@yandex.ru Received: from compute2.internal (compute2.nyi.internal [10.202.2.46]) by mailauth.nyi.internal (Postfix) with ESMTP id CD51227C0054; Sat, 19 Aug 2023 19:34:03 -0400 (EDT) Received: from mailfrontend1 ([10.202.2.162]) by compute2.internal (MEProxy); Sat, 19 Aug 2023 19:34:03 -0400 X-ME-Sender: <xms:alHhZIR_m5C0hZGc7kfxAH5yjJkpPtZlP8Desqt9zI6hBHCihGvt_A> <xme:alHhZFz_a5Sd0R82pV7UvfANwMOfHP8sezAYnbDgsCkUCw833UEHgsGhWeqtzgTKr T_js7vWCV262v-Es-c> X-ME-Received: <xmr:alHhZF26mGEBqC3BovGk8Iu0FXXA1iJu31o2kNLRzKh6d9EX-VnEjOm_KV5aMN8> X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgedviedrudduiedgvdegucetufdoteggodetrfdotf fvucfrrhhofhhilhgvmecuhfgrshhtofgrihhlpdfqfgfvpdfurfetoffkrfgpnffqhgen uceurghilhhouhhtmecufedttdenucesvcftvggtihhpihgvnhhtshculddquddttddmne cujfgurhepkfffgggfuffvvehfhfgjtgfgsehtjeertddtfeejnecuhfhrohhmpeffmhhi thhrhicuifhuthhovhcuoegughhuthhovheshigrnhguvgigrdhruheqnecuggftrfgrth htvghrnheptdffgeegkeelteevtdekleethfeftdduvdegkedtkedujefhfedtveeftdff udevnecuvehluhhsthgvrhfuihiivgeptdenucfrrghrrghmpehmrghilhhfrhhomhepug hguhhtohhvodhmvghsmhhtphgruhhthhhpvghrshhonhgrlhhithihqddufeeffeelleeh hedvqddvleegjeejjeejiedqughguhhtohhvpeephigrnhguvgigrdhruhesfhgrshhtmh grihhlrdgtohhm X-ME-Proxy: <xmx:alHhZMBTFn5vnJVft1FyXJzT5-2Nhnpsg1vYabMnfNz94vrFj84hVw> <xmx:alHhZBjtA7CPytZ8ebExquGqFKjiBgfrdKSiv0i7l3OSnR6RMfs5Fg> <xmx:alHhZIoftltZPR0uPpDA__2qcNVjkSuyz2_c2PSE5kW5ai1nxXqhfA> <xmx:a1HhZBYX_JRgylwFyhIGrjEoC9c5fPoNDAIHcs_H4PUqeTJPv8az8w> Feedback-ID: ib1d9465d:Fastmail Received: by mail.messagingengine.com (Postfix) with ESMTPA; Sat, 19 Aug 2023 19:34:01 -0400 (EDT) Message-ID: <21b04273-e7e5-cbe5-cd95-9484a89a4b64@HIDDEN> Date: Sun, 20 Aug 2023 02:33:59 +0300 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.13.0 Subject: Re: bug#64531: [PATCH v2] Add new command vc-dir-copy-filename-as-kill Content-Language: en-US To: Eli Zaretskii <eliz@HIDDEN>, Ivan Sokolov <ivan-p-sokolov@HIDDEN> References: <87bkgmq1ra.fsf@HIDDEN> <83jzvadabq.fsf@HIDDEN> <87bkfhy7fr.fsf_-_@HIDDEN> <83a5un1mn3.fsf@HIDDEN> From: Dmitry Gutov <dgutov@HIDDEN> In-Reply-To: <83a5un1mn3.fsf@HIDDEN> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit X-Spam-Score: 1.8 (+) X-Spam-Report: Spam detection software, running on the system "debbugs.gnu.org", has NOT identified this incoming email as spam. The original message has been attached to this so you can view it or label similar future email. If you have any questions, see the administrator of that system for details. Content preview: Hi Eli, On 19/08/2023 11:06, Eli Zaretskii wrote: > Dmitry, is this good to go in? I'm good with the binding and the general functionality. Content analysis details: (1.8 points, 10.0 required) pts rule name description ---- ---------------------- -------------------------------------------------- 3.6 RCVD_IN_SBL_CSS RBL: Received via a relay in Spamhaus SBL-CSS [2a02:6b8:c03:500:1:45:d181:d102 listed in] [zen.spamhaus.org] -0.0 SPF_PASS SPF: sender matches SPF record -0.7 RCVD_IN_DNSWL_LOW RBL: Sender listed at https://www.dnswl.org/, low trust [2a02:6b8:c03:500:1:45:d181:d102 listed in] [list.dnswl.org] 0.0 FREEMAIL_FROM Sender email is commonly abused enduser mail provider (dgutov[at]yandex.ru) 0.0 SPF_HELO_NONE SPF: HELO does not publish an SPF Record -1.0 NICE_REPLY_A Looks like a legit reply (A) X-Debbugs-Envelope-To: 64531 Cc: 64531 <at> debbugs.gnu.org X-BeenThere: debbugs-submit <at> debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list List-Id: <debbugs-submit.debbugs.gnu.org> List-Unsubscribe: <https://debbugs.gnu.org/cgi-bin/mailman/options/debbugs-submit>, <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=unsubscribe> List-Archive: <https://debbugs.gnu.org/cgi-bin/mailman/private/debbugs-submit/> List-Post: <mailto:debbugs-submit <at> debbugs.gnu.org> List-Help: <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=help> List-Subscribe: <https://debbugs.gnu.org/cgi-bin/mailman/listinfo/debbugs-submit>, <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=subscribe> Errors-To: debbugs-submit-bounces <at> debbugs.gnu.org Sender: "Debbugs-submit" <debbugs-submit-bounces <at> debbugs.gnu.org> X-Spam-Score: 0.8 (/) Hi Eli, On 19/08/2023 11:06, Eli Zaretskii wrote: > Dmitry, is this good to go in? I'm good with the binding and the general functionality. The difference from dired-copy-filename-as-kill in handing of the prefix argument looks unfortunate (as a principle), but I'd rather leave it for you to judge. Perhaps changing the latter command's interface would be a better choice; I'm not sure why it works that way. Implementation difference: instead of going through (format "%S"), this one uses shell-quote-argument. Is that good enough for Windows systems? And one more difference which we might want to see fixed: (When there's a single file, no quoting is done.) It seems like, at least, in some usage scenarios users would prefer to have copied filename not quoted. E.g. for subsequent use with find-file or... something similar. I don't really use this command all that much, so maybe I'm mistaken here, though.
bug-gnu-emacs@HIDDEN:bug#64531; Package emacs.
Full text available.Received: (at 64531) by debbugs.gnu.org; 19 Aug 2023 23:13:03 +0000 From debbugs-submit-bounces <at> debbugs.gnu.org Sat Aug 19 19:13:03 2023 Received: from localhost ([127.0.0.1]:52440 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from <debbugs-submit-bounces <at> debbugs.gnu.org>) id 1qXV7u-0005pp-Pi for submit <at> debbugs.gnu.org; Sat, 19 Aug 2023 19:13:03 -0400 Received: from forward100b.mail.yandex.net ([178.154.239.147]:42086) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from <ivan-p-sokolov@HIDDEN>) id 1qXV7r-0005pM-6a for 64531 <at> debbugs.gnu.org; Sat, 19 Aug 2023 19:13:01 -0400 Received: from mail-nwsmtp-smtp-production-main-45.sas.yp-c.yandex.net (mail-nwsmtp-smtp-production-main-45.sas.yp-c.yandex.net [IPv6:2a02:6b8:c14:c83:0:640:84f9:0]) by forward100b.mail.yandex.net (Yandex) with ESMTP id AC6DE60031; Sun, 20 Aug 2023 02:12:55 +0300 (MSK) Received: by mail-nwsmtp-smtp-production-main-45.sas.yp-c.yandex.net (smtp/Yandex) with ESMTPSA id sCtbGSPWnmI0-85Bl4iUl; Sun, 20 Aug 2023 02:12:55 +0300 X-Yandex-Fwd: 1 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ya.ru; s=mail; t=1692486775; bh=8qn5yJDgVO4GIr2/x4v6Inuw6iKehAvGMUCQfGjHBb4=; h=Message-ID:Date:Cc:In-Reply-To:Subject:References:To:From; b=Lf5kmbKt73Rj+AT4L/nvR10HHjMQBF9ogn7MV+PpBpVvQq8ZOUEa4+PdvVdeQVkXm 9fHqYkn1VzhyFf3pRHQd97oBJHJH6r/uC+bnwXR9Nf62+2Z+0K7gEGdlg6yGSeZ8mZ +YNgdWYHgkouyRcaGKh+PVNqGeYxoQu7c5DIw3o4= Authentication-Results: mail-nwsmtp-smtp-production-main-45.sas.yp-c.yandex.net; dkim=pass header.i=@ya.ru From: Ivan Sokolov <ivan-p-sokolov@HIDDEN> To: Eli Zaretskii <eliz@HIDDEN> Subject: Re: bug#64531: [PATCH v2] Add new command vc-dir-copy-filename-as-kill In-Reply-To: <83a5un1mn3.fsf@HIDDEN> (Eli Zaretskii's message of "Sat, 19 Aug 2023 11:06:56 +0300") References: <87bkgmq1ra.fsf@HIDDEN> <83jzvadabq.fsf@HIDDEN> <87bkfhy7fr.fsf_-_@HIDDEN> <83a5un1mn3.fsf@HIDDEN> Date: Sun, 20 Aug 2023 02:12:32 +0300 Message-ID: <87sf8eeie7.fsf@HIDDEN> User-Agent: Gnus/5.13 (Gnus v5.13) MIME-Version: 1.0 Content-Type: text/plain X-Spam-Score: 0.0 (/) X-Debbugs-Envelope-To: 64531 Cc: 64531 <at> debbugs.gnu.org, Dmitry Gutov <dgutov@HIDDEN> X-BeenThere: debbugs-submit <at> debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list List-Id: <debbugs-submit.debbugs.gnu.org> List-Unsubscribe: <https://debbugs.gnu.org/cgi-bin/mailman/options/debbugs-submit>, <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=unsubscribe> List-Archive: <https://debbugs.gnu.org/cgi-bin/mailman/private/debbugs-submit/> List-Post: <mailto:debbugs-submit <at> debbugs.gnu.org> List-Help: <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=help> List-Subscribe: <https://debbugs.gnu.org/cgi-bin/mailman/listinfo/debbugs-submit>, <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=subscribe> Errors-To: debbugs-submit-bounces <at> debbugs.gnu.org Sender: "Debbugs-submit" <debbugs-submit-bounces <at> debbugs.gnu.org> X-Spam-Score: -1.0 (-) Eli Zaretskii <eliz@HIDDEN> writes: >> From: Ivan Sokolov <ivan-p-sokolov@HIDDEN> >> Cc: 64531 <at> debbugs.gnu.org >> Date: Tue, 08 Aug 2023 16:44:56 +0300 >> >> Eli Zaretskii <eliz@HIDDEN> writes: >> >> > However, new commands should be called out in NEWS, and commands that >> > are important enough should also be mentioned in the user manual. >> >> Added entry in NEWS, but not in the user manual, there is no such for >> vc-dir afaik. >> >> >> +(defun vc-dir-copy-filename-as-kill (&optional absolutep) >> >> + "Copy names of marked files (or file under cursor) into the kill ring. >> > >> > This sentence is too general: it doesn't mention VC Dir, although >> > AFAIU it is specific to vc-dir. >> >> Fixed. > > Dmitry, is this good to go in? > > I still think it should be mention in the Emacs user manual, where we > document vc-dir. Hi, Eli! I am working on the 3rd version of the patch with all the necessary documentation and changes to the command which I will explain in the cover letter. Not sure when I'll finish it though.
bug-gnu-emacs@HIDDEN:bug#64531; Package emacs.
Full text available.Received: (at 64531) by debbugs.gnu.org; 19 Aug 2023 08:06:54 +0000 From debbugs-submit-bounces <at> debbugs.gnu.org Sat Aug 19 04:06:54 2023 Received: from localhost ([127.0.0.1]:49312 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from <debbugs-submit-bounces <at> debbugs.gnu.org>) id 1qXGz0-00045v-Gx for submit <at> debbugs.gnu.org; Sat, 19 Aug 2023 04:06:54 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]:37192) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from <eliz@HIDDEN>) id 1qXGyz-00045g-2O for 64531 <at> debbugs.gnu.org; Sat, 19 Aug 2023 04:06:53 -0400 Received: from fencepost.gnu.org ([2001:470:142:3::e]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from <eliz@HIDDEN>) id 1qXGyr-00025U-QC; Sat, 19 Aug 2023 04:06:45 -0400 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=gnu.org; s=fencepost-gnu-org; h=References:Subject:In-Reply-To:To:From:Date: mime-version; bh=qx+4YZ53VEf7gcXBgiZJKLibjWxE5kVZjtNjypvJXWA=; b=hpwHPdw6wFRI feGDLlMqZ7X//9blwWT08OQ8SGrfR58zHRqKU2uSgF70zVv1TDTBAueh2wuJ8N/QwOWXCiPML/x3+ wrH0vSwdDYtlul59KRsYmZhIk8y5eZn5sRsAYDr8aOTXyJjNU85GP5DrWqCFtp/VYvRUUZc1nIu8W 1XaEcq54q9lEJzQeDAAwWUv4fnY55/yticO5e2fZg0lYLs5vt4RpVJwSeLG0XaYnwWgjrfSvVerib Tg7AYF+gIWMQpTlDQLnQ7BefFuKdRemIEVT394nX8S9q4ejNGa+JVUDxLwTAE3gKFFOwoV/FOK4oP 6Gg8Sun0r6jy81I0JzhQkA==; Date: Sat, 19 Aug 2023 11:06:56 +0300 Message-Id: <83a5un1mn3.fsf@HIDDEN> From: Eli Zaretskii <eliz@HIDDEN> To: Ivan Sokolov <ivan-p-sokolov@HIDDEN>, Dmitry Gutov <dgutov@HIDDEN> In-Reply-To: <87bkfhy7fr.fsf_-_@HIDDEN> (message from Ivan Sokolov on Tue, 08 Aug 2023 16:44:56 +0300) Subject: Re: bug#64531: [PATCH v2] Add new command vc-dir-copy-filename-as-kill References: <87bkgmq1ra.fsf@HIDDEN> <83jzvadabq.fsf@HIDDEN> <87bkfhy7fr.fsf_-_@HIDDEN> X-Spam-Score: -2.3 (--) X-Debbugs-Envelope-To: 64531 Cc: 64531 <at> debbugs.gnu.org X-BeenThere: debbugs-submit <at> debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list List-Id: <debbugs-submit.debbugs.gnu.org> List-Unsubscribe: <https://debbugs.gnu.org/cgi-bin/mailman/options/debbugs-submit>, <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=unsubscribe> List-Archive: <https://debbugs.gnu.org/cgi-bin/mailman/private/debbugs-submit/> List-Post: <mailto:debbugs-submit <at> debbugs.gnu.org> List-Help: <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=help> List-Subscribe: <https://debbugs.gnu.org/cgi-bin/mailman/listinfo/debbugs-submit>, <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=subscribe> Errors-To: debbugs-submit-bounces <at> debbugs.gnu.org Sender: "Debbugs-submit" <debbugs-submit-bounces <at> debbugs.gnu.org> X-Spam-Score: -3.3 (---) > From: Ivan Sokolov <ivan-p-sokolov@HIDDEN> > Cc: 64531 <at> debbugs.gnu.org > Date: Tue, 08 Aug 2023 16:44:56 +0300 > > Eli Zaretskii <eliz@HIDDEN> writes: > > > However, new commands should be called out in NEWS, and commands that > > are important enough should also be mentioned in the user manual. > > Added entry in NEWS, but not in the user manual, there is no such for > vc-dir afaik. > > >> +(defun vc-dir-copy-filename-as-kill (&optional absolutep) > >> + "Copy names of marked files (or file under cursor) into the kill ring. > > > > This sentence is too general: it doesn't mention VC Dir, although > > AFAIU it is specific to vc-dir. > > Fixed. Dmitry, is this good to go in? I still think it should be mention in the Emacs user manual, where we document vc-dir.
bug-gnu-emacs@HIDDEN:bug#64531; Package emacs.
Full text available.
Received: (at 64531) by debbugs.gnu.org; 8 Aug 2023 13:56:13 +0000
From debbugs-submit-bounces <at> debbugs.gnu.org Tue Aug 08 09:56:13 2023
Received: from localhost ([127.0.0.1]:37396 helo=debbugs.gnu.org)
by debbugs.gnu.org with esmtp (Exim 4.84_2)
(envelope-from <debbugs-submit-bounces <at> debbugs.gnu.org>)
id 1qTNC1-0004BX-4o
for submit <at> debbugs.gnu.org; Tue, 08 Aug 2023 09:56:13 -0400
Received: from eggs.gnu.org ([2001:470:142:3::10]:46710)
by debbugs.gnu.org with esmtp (Exim 4.84_2)
(envelope-from <eliz@HIDDEN>) id 1qTNBy-0004BP-B0
for 64531 <at> debbugs.gnu.org; Tue, 08 Aug 2023 09:56:11 -0400
Received: from fencepost.gnu.org ([2001:470:142:3::e])
by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256)
(Exim 4.90_1) (envelope-from <eliz@HIDDEN>)
id 1qTNBs-0001PM-Ly; Tue, 08 Aug 2023 09:56:04 -0400
DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=gnu.org;
s=fencepost-gnu-org; h=References:Subject:In-Reply-To:To:From:Date:
mime-version; bh=OxWByizXdTCO6r7OvZb41isJUvLH4oTUf5oh/gerP+4=; b=fkqVzREJvwl0
G68IvRDOZPNdVHn36iRtyfHvykHTIvdAxd1crSogFx4sZaq1GsYWu8KtRRjRuA17O2Ua9bRvX2Ect
wvZrPOlEv4LKRr6092hi/9DaE59IcPE7+eyg0Wueja1Wz/FCrHMGcXt4jVvuZ72o77o8CcKxPiHqt
XOSyTzeyDqwaK8Nc355obyTBlLs3xN0AQ8d9ScMt8YmVUppk1jQ9UBvdtQaURacil/wwFUtty3g3m
RIeNgq4uYbvnlKVUmCqxuyRUK3Yao4FREOIe53xtHB24vRgCTClobaAkllNGuYEjOtuqB9LkM7k9S
u9AFlt8U70ko98s1iEdJMg==;
Date: Tue, 08 Aug 2023 16:56:26 +0300
Message-Id: <83r0odmyd1.fsf@HIDDEN>
From: Eli Zaretskii <eliz@HIDDEN>
To: Ivan Sokolov <ivan-p-sokolov@HIDDEN>
In-Reply-To: <87bkfhy7fr.fsf_-_@HIDDEN> (message from Ivan Sokolov on Tue, 08
Aug 2023 16:44:56 +0300)
Subject: Re: bug#64531: [PATCH v2] Add new command vc-dir-copy-filename-as-kill
References: <87bkgmq1ra.fsf@HIDDEN> <83jzvadabq.fsf@HIDDEN>
<87bkfhy7fr.fsf_-_@HIDDEN>
X-Spam-Score: -2.3 (--)
X-Debbugs-Envelope-To: 64531
Cc: 64531 <at> debbugs.gnu.org
X-BeenThere: debbugs-submit <at> debbugs.gnu.org
X-Mailman-Version: 2.1.18
Precedence: list
List-Id: <debbugs-submit.debbugs.gnu.org>
List-Unsubscribe: <https://debbugs.gnu.org/cgi-bin/mailman/options/debbugs-submit>,
<mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=unsubscribe>
List-Archive: <https://debbugs.gnu.org/cgi-bin/mailman/private/debbugs-submit/>
List-Post: <mailto:debbugs-submit <at> debbugs.gnu.org>
List-Help: <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=help>
List-Subscribe: <https://debbugs.gnu.org/cgi-bin/mailman/listinfo/debbugs-submit>,
<mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=subscribe>
Errors-To: debbugs-submit-bounces <at> debbugs.gnu.org
Sender: "Debbugs-submit" <debbugs-submit-bounces <at> debbugs.gnu.org>
X-Spam-Score: -3.3 (---)
> From: Ivan Sokolov <ivan-p-sokolov@HIDDEN>
> Cc: 64531 <at> debbugs.gnu.org
> Date: Tue, 08 Aug 2023 16:44:56 +0300
>
> Eli Zaretskii <eliz@HIDDEN> writes:
>
> > However, new commands should be called out in NEWS, and commands that
> > are important enough should also be mentioned in the user manual.
>
> Added entry in NEWS, but not in the user manual, there is no such for
> vc-dir afaik.
vc-dir is described in the subsection "VC Directory Mode" and its
sub-subsections.
> +(defun vc-dir-copy-filename-as-kill (&optional absolutep)
> + "Copy filenames from the `vc-dir' buffer into the kill ring.
> +If there are marked files, copy their names, otherwise copy the
> +file at point. Names will be separated by a space, each will be
> +quoted using `shell-quote-argument'. If ABSOLUTEP use absolute
^
A comma is missing there.
bug-gnu-emacs@HIDDEN:bug#64531; Package emacs.
Full text available.
Received: (at 64531) by debbugs.gnu.org; 8 Aug 2023 13:45:23 +0000
From debbugs-submit-bounces <at> debbugs.gnu.org Tue Aug 08 09:45:23 2023
Received: from localhost ([127.0.0.1]:35684 helo=debbugs.gnu.org)
by debbugs.gnu.org with esmtp (Exim 4.84_2)
(envelope-from <debbugs-submit-bounces <at> debbugs.gnu.org>)
id 1qTN1W-0003Nw-SU
for submit <at> debbugs.gnu.org; Tue, 08 Aug 2023 09:45:23 -0400
Received: from forward500b.mail.yandex.net ([178.154.239.144]:60254)
by debbugs.gnu.org with esmtp (Exim 4.84_2)
(envelope-from <ivan-p-sokolov@HIDDEN>) id 1qTN1T-0003Nk-KP
for 64531 <at> debbugs.gnu.org; Tue, 08 Aug 2023 09:45:21 -0400
Received: from mail-nwsmtp-smtp-production-main-17.iva.yp-c.yandex.net
(mail-nwsmtp-smtp-production-main-17.iva.yp-c.yandex.net
[IPv6:2a02:6b8:c0c:c20c:0:640:509d:0])
by forward500b.mail.yandex.net (Yandex) with ESMTP id 796735E5FD;
Tue, 8 Aug 2023 16:45:17 +0300 (MSK)
Received: by mail-nwsmtp-smtp-production-main-17.iva.yp-c.yandex.net
(smtp/Yandex) with ESMTPSA id GjY0Cb3WsqM0-E2w7NDOO;
Tue, 08 Aug 2023 16:45:17 +0300
X-Yandex-Fwd: 1
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ya.ru; s=mail;
t=1691502317; bh=uPOYi6PsbKwx4ghBnqhfBzSqtTUb+t+45Epp3bXVhRE=;
h=Message-ID:Date:Cc:In-Reply-To:Subject:References:To:From;
b=rEYqzsQNKJTqLhhh9Qm5hIrmF6x0LVRSu+7eLeY+LgidJijHGblLrWkQzwjvobPHe
dlYPtW4gadD1Qvx7NXnnt4zXEUAC0r70FrNluubcc+cXXb8yl5iSYWqETFX22yhUGW
fPeDM7LbFGh6Zt/Y2cicpT9rfqpGprkdn4lK/GnU=
Authentication-Results: mail-nwsmtp-smtp-production-main-17.iva.yp-c.yandex.net;
dkim=pass header.i=@ya.ru
From: Ivan Sokolov <ivan-p-sokolov@HIDDEN>
To: Eli Zaretskii <eliz@HIDDEN>
Subject: bug#64531: [PATCH v2] Add new command vc-dir-copy-filename-as-kill
In-Reply-To: <83jzvadabq.fsf@HIDDEN> (Eli Zaretskii's message of "Sat, 08 Jul
2023 12:25:13 +0300")
References: <87bkgmq1ra.fsf@HIDDEN> <83jzvadabq.fsf@HIDDEN>
Date: Tue, 08 Aug 2023 16:44:56 +0300
Message-ID: <87bkfhy7fr.fsf_-_@HIDDEN>
User-Agent: Gnus/5.13 (Gnus v5.13)
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="=-=-="
X-Spam-Score: -0.0 (/)
X-Debbugs-Envelope-To: 64531
Cc: 64531 <at> debbugs.gnu.org
X-BeenThere: debbugs-submit <at> debbugs.gnu.org
X-Mailman-Version: 2.1.18
Precedence: list
List-Id: <debbugs-submit.debbugs.gnu.org>
List-Unsubscribe: <https://debbugs.gnu.org/cgi-bin/mailman/options/debbugs-submit>,
<mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=unsubscribe>
List-Archive: <https://debbugs.gnu.org/cgi-bin/mailman/private/debbugs-submit/>
List-Post: <mailto:debbugs-submit <at> debbugs.gnu.org>
List-Help: <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=help>
List-Subscribe: <https://debbugs.gnu.org/cgi-bin/mailman/listinfo/debbugs-submit>,
<mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=subscribe>
Errors-To: debbugs-submit-bounces <at> debbugs.gnu.org
Sender: "Debbugs-submit" <debbugs-submit-bounces <at> debbugs.gnu.org>
X-Spam-Score: -1.0 (-)
--=-=-=
Content-Type: text/plain
Eli Zaretskii <eliz@HIDDEN> writes:
> However, new commands should be called out in NEWS, and commands that
> are important enough should also be mentioned in the user manual.
Added entry in NEWS, but not in the user manual, there is no such for
vc-dir afaik.
>> +(defun vc-dir-copy-filename-as-kill (&optional absolutep)
>> + "Copy names of marked files (or file under cursor) into the kill ring.
>
> This sentence is too general: it doesn't mention VC Dir, although
> AFAIU it is specific to vc-dir.
Fixed.
--=-=-=
Content-Type: text/x-patch
Content-Disposition: attachment;
filename=v2-0001-Add-new-command-vc-dir-copy-filename-as-kill.patch
From 88cb2e3407adf0245bc15a88896e1cd611d559c8 Mon Sep 17 00:00:00 2001
From: Ivan Sokolov <ivan-p-sokolov@HIDDEN>
Date: Tue, 11 Jul 2023 18:19:57 +0300
Subject: [PATCH v2] Add new command vc-dir-copy-filename-as-kill
* lisp/vc/vc-dir.el (vc-dir-copy-filename-as-kill): New command.
(vc-dir-mode-map): Bind vc-dir-copy-filename-as-kill to `w'.
---
etc/NEWS | 3 +++
lisp/vc/vc-dir.el | 18 ++++++++++++++++++
2 files changed, 21 insertions(+)
diff --git a/etc/NEWS b/etc/NEWS
index 1a86c9e55e2..da3672e068b 100644
--- a/etc/NEWS
+++ b/etc/NEWS
@@ -154,6 +154,9 @@ This is a string or a list of strings that specifies the Git log
switches for shortlogs, such as the one produced by 'C-x v L'.
'vc-git-log-switches' is no longer used for shortlogs.
+*** New command 'vc-dir-copy-filename-as-kill'.
+This works like 'dired-copy-filename-as-kill' but in vc-dir buffers.
+
** Diff Mode
+++
diff --git a/lisp/vc/vc-dir.el b/lisp/vc/vc-dir.el
index 53d58870b32..dc25467a3e9 100644
--- a/lisp/vc/vc-dir.el
+++ b/lisp/vc/vc-dir.el
@@ -353,6 +353,7 @@ See `run-hooks'."
(define-key map (kbd "M-s a C-s") #'vc-dir-isearch)
(define-key map (kbd "M-s a M-C-s") #'vc-dir-isearch-regexp)
(define-key map "G" #'vc-dir-ignore)
+ (define-key map "w" #'vc-dir-copy-filename-as-kill)
(let ((branch-map (make-sparse-keymap)))
(define-key map "b" branch-map)
@@ -930,6 +931,23 @@ system."
(interactive)
(view-file (vc-dir-current-file)))
+(defun vc-dir-copy-filename-as-kill (&optional absolutep)
+ "Copy filenames from the `vc-dir' buffer into the kill ring.
+If there are marked files, copy their names, otherwise copy the
+file at point. Names will be separated by a space, each will be
+quoted using `shell-quote-argument'. If ABSOLUTEP use absolute
+names, otherwise names are relative to the `default-directory'."
+ (interactive "P")
+ (let ((files (or (vc-dir-marked-files)
+ (list (vc-dir-current-file)))))
+ (unless absolutep
+ (setq files (mapcar #'file-relative-name files)))
+ (let ((string (mapconcat #'shell-quote-argument files " ")))
+ (if (eq last-command 'kill-region)
+ (kill-append string nil)
+ (kill-new string))
+ (message "%s" string))))
+
(defun vc-dir-isearch ()
"Search for a string through all marked buffers using Isearch."
(interactive)
--
2.41.0
--=-=-=--
bug-gnu-emacs@HIDDEN:bug#64531; Package emacs.
Full text available.Received: (at 64531) by debbugs.gnu.org; 8 Jul 2023 09:25:26 +0000 From debbugs-submit-bounces <at> debbugs.gnu.org Sat Jul 08 05:25:26 2023 Received: from localhost ([127.0.0.1]:44290 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from <debbugs-submit-bounces <at> debbugs.gnu.org>) id 1qI4By-0000bn-H1 for submit <at> debbugs.gnu.org; Sat, 08 Jul 2023 05:25:26 -0400 Received: from eggs.gnu.org ([209.51.188.92]:57936) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from <eliz@HIDDEN>) id 1qI4Bv-0000bX-8u for 64531 <at> debbugs.gnu.org; Sat, 08 Jul 2023 05:25:25 -0400 Received: from fencepost.gnu.org ([2001:470:142:3::e]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from <eliz@HIDDEN>) id 1qI4Bp-0000tb-Co; Sat, 08 Jul 2023 05:25:17 -0400 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=gnu.org; s=fencepost-gnu-org; h=References:Subject:In-Reply-To:To:From:Date: mime-version; bh=cdVPtK+l5baK0S7C3TiSqRA3OHv8Vic5hA7385tF7eQ=; b=EiVhkh3KYSp8 GMEMAuztPzSWr9Je3e6ZdUbD2Arlgc8iwDakHhSYQtspBScjii74f/bAWpyA1vEhrVbi7m8TbrUhr AJp3V5zT8VKAHmoub6UFQmgxCP0aKNf/3OLAqj2sD2jY51ieES0VHEBU0F+4doK06zn7n6icTpMWQ cA023YJC3j39jTmajgzXjYGbtnG6HaDmuhHyM7i+FWPK/apa7B3FIsjbeAjLVFOaT7mq1UnxAaTJu zM4TzLN7E9//aA2yN6WUrErnJW/ITSBUXiz8pGwt3tQ7J3ZNzb39649/o6x6faPGZHfhjJiVXl9Fe coAvIafkk+jqg3llMyd5oA==; Received: from [87.69.77.57] (helo=home-c4e4a596f7) by fencepost.gnu.org with esmtpsa (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from <eliz@HIDDEN>) id 1qI4Bi-0004el-Ht; Sat, 08 Jul 2023 05:25:11 -0400 Date: Sat, 08 Jul 2023 12:25:13 +0300 Message-Id: <83jzvadabq.fsf@HIDDEN> From: Eli Zaretskii <eliz@HIDDEN> To: Ivan Sokolov <ivan-p-sokolov@HIDDEN> In-Reply-To: <87bkgmq1ra.fsf@HIDDEN> (message from Ivan Sokolov on Sat, 08 Jul 2023 10:51:53 +0300) Subject: Re: bug#64531: [PATCH] Add new command vc-dir-copy-filename-as-kill References: <87bkgmq1ra.fsf@HIDDEN> X-Spam-Score: -2.3 (--) X-Debbugs-Envelope-To: 64531 Cc: 64531 <at> debbugs.gnu.org X-BeenThere: debbugs-submit <at> debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list List-Id: <debbugs-submit.debbugs.gnu.org> List-Unsubscribe: <https://debbugs.gnu.org/cgi-bin/mailman/options/debbugs-submit>, <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=unsubscribe> List-Archive: <https://debbugs.gnu.org/cgi-bin/mailman/private/debbugs-submit/> List-Post: <mailto:debbugs-submit <at> debbugs.gnu.org> List-Help: <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=help> List-Subscribe: <https://debbugs.gnu.org/cgi-bin/mailman/listinfo/debbugs-submit>, <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=subscribe> Errors-To: debbugs-submit-bounces <at> debbugs.gnu.org Sender: "Debbugs-submit" <debbugs-submit-bounces <at> debbugs.gnu.org> X-Spam-Score: -3.3 (---) > From: Ivan Sokolov <ivan-p-sokolov@HIDDEN> > Date: Sat, 08 Jul 2023 10:51:53 +0300 > > This new command is inspired by `dired-copy-filename-as-kill', but has a > few differences, that can be easily reversed if there is such need. > > Changes: > 1. even single filename is quoted; > 2. filenames are relative by default, prefix makes them absolute; > 3. value of prefix argument does not matter, in dired it has to be 0. > > I bound it to the `w' key because `w' is currently unbound in > vc-dir-mode-map and it is what is used in dired. Thanks. However, new commands should be called out in NEWS, and commands that are important enough should also be mentioned in the user manual. > +(defun vc-dir-copy-filename-as-kill (&optional absolutep) > + "Copy names of marked files (or file under cursor) into the kill ring. This sentence is too general: it doesn't mention VC Dir, although AFAIU it is specific to vc-dir.
bug-gnu-emacs@HIDDEN:bug#64531; Package emacs.
Full text available.
Received: (at submit) by debbugs.gnu.org; 8 Jul 2023 07:52:31 +0000
From debbugs-submit-bounces <at> debbugs.gnu.org Sat Jul 08 03:52:31 2023
Received: from localhost ([127.0.0.1]:44239 helo=debbugs.gnu.org)
by debbugs.gnu.org with esmtp (Exim 4.84_2)
(envelope-from <debbugs-submit-bounces <at> debbugs.gnu.org>)
id 1qI2k3-0006iV-Ah
for submit <at> debbugs.gnu.org; Sat, 08 Jul 2023 03:52:31 -0400
Received: from lists.gnu.org ([209.51.188.17]:42078)
by debbugs.gnu.org with esmtp (Exim 4.84_2)
(envelope-from <ivan-p-sokolov@HIDDEN>) id 1qI2k0-0006iK-Sh
for submit <at> debbugs.gnu.org; Sat, 08 Jul 2023 03:52:30 -0400
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 <ivan-p-sokolov@HIDDEN>)
id 1qI2k0-0001kq-M3
for bug-gnu-emacs@HIDDEN; Sat, 08 Jul 2023 03:52:28 -0400
Received: from forward102b.mail.yandex.net ([2a02:6b8:c02:900:1:45:d181:d102])
by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256)
(Exim 4.90_1) (envelope-from <ivan-p-sokolov@HIDDEN>)
id 1qI2jw-0003Ze-U1
for bug-gnu-emacs@HIDDEN; Sat, 08 Jul 2023 03:52:28 -0400
Received: from mail-nwsmtp-smtp-production-main-29.myt.yp-c.yandex.net
(mail-nwsmtp-smtp-production-main-29.myt.yp-c.yandex.net
[IPv6:2a02:6b8:c12:3faf:0:640:537b:0])
by forward102b.mail.yandex.net (Yandex) with ESMTP id 7038860043
for <bug-gnu-emacs@HIDDEN>; Sat, 8 Jul 2023 10:52:17 +0300 (MSK)
Received: by mail-nwsmtp-smtp-production-main-29.myt.yp-c.yandex.net
(smtp/Yandex) with ESMTPSA id 7qQkXH7DTiE0-V7tI57ST;
Sat, 08 Jul 2023 10:52:15 +0300
X-Yandex-Fwd: 1
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ya.ru; s=mail;
t=1688802736; bh=4q4vo5pWJjU5Cwng/SH7W0NYIfQ9UIxmL1HbxrTWvp4=;
h=Message-ID:Date:Subject:To:From;
b=neR9XJEcC8bBRvmtpqInQCKZ4mUMBJDGEAEtZN+H+1d8IrqMKqrM41rk1AeRcOjjb
XfSEqyeFvubm9OKbhBvim09ISWKrsNoycqW5xcM4tWtoHzNPBG1T7zHumVILRIenSz
P+ecVO0JF8ZZiLzwRRPaC5NDeG6OaB4IHCzNn16Y=
Authentication-Results: mail-nwsmtp-smtp-production-main-29.myt.yp-c.yandex.net;
dkim=pass header.i=@ya.ru
From: Ivan Sokolov <ivan-p-sokolov@HIDDEN>
To: bug-gnu-emacs@HIDDEN
Subject: [PATCH] Add new command vc-dir-copy-filename-as-kill
Date: Sat, 08 Jul 2023 10:51:53 +0300
Message-ID: <87bkgmq1ra.fsf@HIDDEN>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="=-=-="
Received-SPF: pass client-ip=2a02:6b8:c02:900:1:45:d181:d102;
envelope-from=ivan-p-sokolov@HIDDEN; helo=forward102b.mail.yandex.net
X-Spam_score_int: -27
X-Spam_score: -2.8
X-Spam_bar: --
X-Spam_report: (-2.8 / 5.0 requ) BAYES_00=-1.9, DKIM_SIGNED=0.1,
DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, FREEMAIL_FROM=0.001,
RCVD_IN_DNSWL_LOW=-0.7, SPF_HELO_NONE=0.001, SPF_PASS=-0.001,
T_SCC_BODY_TEXT_LINE=-0.01 autolearn=ham autolearn_force=no
X-Spam_action: no action
X-Spam-Score: -1.3 (-)
X-Debbugs-Envelope-To: submit
X-BeenThere: debbugs-submit <at> debbugs.gnu.org
X-Mailman-Version: 2.1.18
Precedence: list
List-Id: <debbugs-submit.debbugs.gnu.org>
List-Unsubscribe: <https://debbugs.gnu.org/cgi-bin/mailman/options/debbugs-submit>,
<mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=unsubscribe>
List-Archive: <https://debbugs.gnu.org/cgi-bin/mailman/private/debbugs-submit/>
List-Post: <mailto:debbugs-submit <at> debbugs.gnu.org>
List-Help: <mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=help>
List-Subscribe: <https://debbugs.gnu.org/cgi-bin/mailman/listinfo/debbugs-submit>,
<mailto:debbugs-submit-request <at> debbugs.gnu.org?subject=subscribe>
Errors-To: debbugs-submit-bounces <at> debbugs.gnu.org
Sender: "Debbugs-submit" <debbugs-submit-bounces <at> debbugs.gnu.org>
X-Spam-Score: -2.3 (--)
--=-=-=
Content-Type: text/plain
Hi!
This new command is inspired by `dired-copy-filename-as-kill', but has a
few differences, that can be easily reversed if there is such need.
Changes:
1. even single filename is quoted;
2. filenames are relative by default, prefix makes them absolute;
3. value of prefix argument does not matter, in dired it has to be 0.
I bound it to the `w' key because `w' is currently unbound in
vc-dir-mode-map and it is what is used in dired.
--=-=-=
Content-Type: text/x-patch
Content-Disposition: attachment;
filename=0001-Add-new-command-vc-dir-copy-filename-as-kill.patch
Content-Description: [PATCH] Add new command vc-dir-copy-filename-as-kill
From d2edc8052d3f8a34b4f28f6721cb2b2ceca1d4a5 Mon Sep 17 00:00:00 2001
From: Ivan Sokolov <ivan-p-sokolov@HIDDEN>
Date: Sat, 8 Jul 2023 10:34:18 +0300
Subject: [PATCH] Add new command vc-dir-copy-filename-as-kill
---
lisp/vc/vc-dir.el | 19 +++++++++++++++++++
1 file changed, 19 insertions(+)
diff --git a/lisp/vc/vc-dir.el b/lisp/vc/vc-dir.el
index 53d58870b32..e348f1027ab 100644
--- a/lisp/vc/vc-dir.el
+++ b/lisp/vc/vc-dir.el
@@ -353,6 +353,7 @@ See `run-hooks'."
(define-key map (kbd "M-s a C-s") #'vc-dir-isearch)
(define-key map (kbd "M-s a M-C-s") #'vc-dir-isearch-regexp)
(define-key map "G" #'vc-dir-ignore)
+ (define-key map "w" #'vc-dir-copy-filename-as-kill)
(let ((branch-map (make-sparse-keymap)))
(define-key map "b" branch-map)
@@ -930,6 +931,24 @@ system."
(interactive)
(view-file (vc-dir-current-file)))
+(defun vc-dir-copy-filename-as-kill (&optional absolutep)
+ "Copy names of marked files (or file under cursor) into the kill ring.
+If there are severals names, they will be separated by a space.
+Names are always quoted using `shell-quote-argument'.
+
+If ABSOLUTEP use the absolute names, otherwise names are relative
+to the `default-directory'."
+ (interactive "P")
+ (let ((files (or (vc-dir-marked-files)
+ (list (vc-dir-current-file)))))
+ (unless absolutep
+ (setq files (mapcar #'file-relative-name files)))
+ (let ((string (mapconcat #'shell-quote-argument files " ")))
+ (if (eq last-command 'kill-region)
+ (kill-append string nil)
+ (kill-new string))
+ (message "%s" string))))
+
(defun vc-dir-isearch ()
"Search for a string through all marked buffers using Isearch."
(interactive)
--
2.40.1
--=-=-=--
Ivan Sokolov <ivan-p-sokolov@HIDDEN>:bug-gnu-emacs@HIDDEN.
Full text available.bug-gnu-emacs@HIDDEN:bug#64531; Package emacs.
Full text available.
GNU bug tracking system
Copyright (C) 1999 Darren O. Benham,
1997 nCipher Corporation Ltd,
1994-97 Ian Jackson.