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#63648: 29.0.90; project.el: with switch-use-entire-map, switch-project errors on non-project commands Date: Sun, 5 Nov 2023 02:55:41 +0200 Message-ID: References: <86edishisp.fsf@mail.linkov.net> <6fc81cbf-a21f-c5b4-aa56-e8518b8570d7@gutov.dev> <86msxgatuy.fsf@mail.linkov.net> <86y1gynr2u.fsf@mail.linkov.net> <7c72fd8c-c3f6-a974-8a4b-a081f7a9fe1a@gutov.dev> <86ttqmv7c6.fsf@mail.linkov.net> <56e7d865-0456-db7d-3a5d-1235e62508bf@gutov.dev> <865y31onju.fsf@mail.linkov.net> <8562bdad-e487-886a-af49-7f67d8d14b7a@gutov.dev> <86msw9g5yb.fsf@mail.linkov.net> <56909bdd-14ee-f68c-178f-5057efcd6150@gutov.dev> <86wmva7iey.fsf@mail.linkov.net> <86bkckwnf8.fsf@mail.linkov.net> <86h6mag077.fsf@mail.linkov.net> <86v8akoyve.fsf@mail.linkov.net> <86sf5lxyow.fsf@mail.linkov.net> 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="13444"; 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: Spencer Baugh , 63648@debbugs.gnu.org, sbaugh@catern.com To: Juri Linkov Original-X-From: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane-mx.org@gnu.org Sun Nov 05 01:56:50 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 1qzRRY-0003Hg-9N for geb-bug-gnu-emacs@m.gmane-mx.org; Sun, 05 Nov 2023 01:56:49 +0100 Original-Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1qzRRE-0002kW-3F; Sat, 04 Nov 2023 20:56:28 -0400 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 1qzRRC-0002jq-5H for bug-gnu-emacs@gnu.org; Sat, 04 Nov 2023 20:56:26 -0400 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 1qzRRB-0004Me-St for bug-gnu-emacs@gnu.org; Sat, 04 Nov 2023 20:56:25 -0400 Original-Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1qzRRl-0005Zc-Vw for bug-gnu-emacs@gnu.org; Sat, 04 Nov 2023 20:57:02 -0400 X-Loop: help-debbugs@gnu.org Resent-From: Dmitry Gutov Original-Sender: "Debbugs-submit" Resent-CC: bug-gnu-emacs@gnu.org Resent-Date: Sun, 05 Nov 2023 00:57:01 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 63648 X-GNU-PR-Package: emacs Original-Received: via spool by 63648-submit@debbugs.gnu.org id=B63648.169914579021381 (code B ref 63648); Sun, 05 Nov 2023 00:57:01 +0000 Original-Received: (at 63648) by debbugs.gnu.org; 5 Nov 2023 00:56:30 +0000 Original-Received: from localhost ([127.0.0.1]:35912 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1qzRRG-0005Yn-6m for submit@debbugs.gnu.org; Sat, 04 Nov 2023 20:56:30 -0400 Original-Received: from out4-smtp.messagingengine.com ([66.111.4.28]:37095) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1qzRRD-0005YX-Ex for 63648@debbugs.gnu.org; Sat, 04 Nov 2023 20:56:28 -0400 Original-Received: from compute6.internal (compute6.nyi.internal [10.202.2.47]) by mailout.nyi.internal (Postfix) with ESMTP id B97C85C00D6; Sat, 4 Nov 2023 20:55:44 -0400 (EDT) Original-Received: from mailfrontend2 ([10.202.2.163]) by compute6.internal (MEProxy); Sat, 04 Nov 2023 20:55:44 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gutov.dev; h=cc :cc:content-transfer-encoding:content-type:content-type:date :date:from:from:in-reply-to:in-reply-to:message-id:mime-version :references:reply-to:sender:subject:subject:to:to; s=fm2; t= 1699145744; x=1699232144; bh=6Gxr6Y1/58CEl4GDz+rzw3U72gzB6TajorQ XWjKmsNk=; b=MkRUT/2ttvygIfaD/Jq12G8kMQDxyFhiWlm7hxeBB90az9An8CS 6R//ix8DZM6JmygN1vE2rGyx7Aq6iePqisiFX4AkG+t4uvaARsH4yUE2C/36m7kv LUgm0+f+ifYXVcXfHv+YnOK7VHVoDE+ulVaSOKdaaKySenvWH+apIuEAaYS0fzhi 8YZeOjIueAcbjTdGwnOLiofpKVogmDOtqeQtU65WS2puH1WyxrdJ/LHy82n89Ved lljXRYvIr8RS5npcBZ9oSUJZCowx/WmOhN7aoObGspfUEjG67nujP8G43cuTOUf6 iv1xybdE2+jPa5yv79InGLCkAxN9nX9eNeQ== DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:cc:content-transfer-encoding :content-type:content-type:date:date:feedback-id:feedback-id :from:from:in-reply-to:in-reply-to:message-id:mime-version :references:reply-to:sender:subject:subject:to:to:x-me-proxy :x-me-proxy:x-me-sender:x-me-sender:x-sasl-enc; s=fm3; t= 1699145744; x=1699232144; bh=6Gxr6Y1/58CEl4GDz+rzw3U72gzB6TajorQ XWjKmsNk=; b=tSSrjAY7+iIiGbZ4F4c+qqqVuZ/jOsZHJG8XBWN9jWMTPnhchIZ 6qCHYynFM6a8eJXM5et/gXkSl7CDPt532Teep47R8Kq+ivPn2A+eZtvJE9fZvv3v atlsfb+TwX0buSQv8udVNPBlnIsehkhXntfEgBkNBB3t0oVqgHdCAHblW2Vb24gu o/76AogmAD/rhupyawhvbudZkMa/SNFlKIPnz1vM/DTIj2sukS0e3/wObhtwdT1j 4kmOQo77/3a89lO6A6BxNKyvsBrNDVNTfLlnmeqhOsweIrTr223yO4efaAV9VV0i zEaJ1g2YIucGsD8zLf2WRmsy3ODNZSdrvug== X-ME-Sender: X-ME-Received: X-ME-Proxy-Cause: gggruggvucftvghtrhhoucdtuddrgedvkedrudduuddgvdejucetufdoteggodetrfdotf fvucfrrhhofhhilhgvmecuhfgrshhtofgrihhlpdfqfgfvpdfurfetoffkrfgpnffqhgen uceurghilhhouhhtmecufedttdenucesvcftvggtihhpihgvnhhtshculddquddttddmne cujfgurhepkfffgggfuffvvehfhfgjtgfgsehtjeertddtfeejnecuhfhrohhmpeffmhhi thhrhicuifhuthhovhcuoegumhhithhrhiesghhuthhovhdruggvvheqnecuggftrfgrth htvghrnhepiefgteevheevveffheeltdeukeeiieekueefgedugfefgefhudelgfefveel vdevnecuvehluhhsthgvrhfuihiivgeptdenucfrrghrrghmpehmrghilhhfrhhomhepug hmihhtrhihsehguhhtohhvrdguvghv X-ME-Proxy: Feedback-ID: i0e71465a:Fastmail Original-Received: by mail.messagingengine.com (Postfix) with ESMTPA; Sat, 4 Nov 2023 20:55:43 -0400 (EDT) Content-Language: en-US In-Reply-To: <86sf5lxyow.fsf@mail.linkov.net> 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:273796 Archived-At: On 04/11/2023 19:28, Juri Linkov wrote: >>>> How about this, then? >>>> >>>> project-switch-use-global-map is obsoleted, all variations are accessible >>>> through customizing project-switch-commands. Should have better >>>> discoverability for project-prefix-or-any-command than simply having it >>>> included. >>> I don't know, it's extra trouble for users to migrate >>> from the obsoleted project-switch-use-entire-map. >>> But maybe ok. >> >> I couldn't find a better alternative to keep the behavior compatible. >> >> Either we have the user customize an existing var (this was the option >> where they'd set project-switch-commands to >> `project-prefix-or-any-command', which you didn't like), or we would add >> a yet another custom var which would, conceptually, conflict with the two >> existing ones. > > I don't dislike it. Very good. > I proposed a new option project-switch-use-global-map. I didn't know how the reconcile the existence of both project-switch-use-entire-map and project-switch-use-global-map. Saying in both docstrings that one takes priority over the other (and possibly repeating that in the doc for project-switch-commands) would be awkward. > But when project-switch-commands should be customized instead of new option, > then I'd prefer to use function names like `project-prefix-or-any-command' > instead of symbols `short-or-any' and `short-keys'. > >> Unifying all three alternatives in one option seems like the logical choice. > > To avoid obsoleting `project-switch-use-entire-map', `short-keys' > could be removed. Then `short-or-any' could be replaced by > `project-prefix-or-any-command'. This will introduce minimal changes. All right, that sounds like a mix between v4 and v5. I've pushed 41e801fea1c to master, please take a look. In the future, perhaps we'd want to obsolete the option project-switch-use-entire-map anyway, replacing it with a separate command to be used in the same way as the newly-added one. But for now the change is maximally backward-compatible. I'm still not sure personally about the prompt in project-prefix-or-any-command (short and not differentiating from project-any-command), but let's see what others think and propose later.