From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.io!.POSTED.blaine.gmane.org!not-for-mail From: Michael Albinus Newsgroups: gmane.emacs.bugs Subject: bug#61748: 27.2; Eglot should use shell-file-name when launching the language server for a remote file Date: Thu, 02 Mar 2023 10:14:17 +0100 Message-ID: <878rgfecra.fsf@gmx.de> References: <87sfeqq8z5.fsf@gmail.com> <75486564.6945596.1677592772065.JavaMail.root@zimbra60-e10.priv.proxad.net> <87r0u9devu.fsf@gmx.de> <87bkld6ctm.fsf@gmx.de> <87356p6b7g.fsf@gmx.de> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Injection-Info: ciao.gmane.io; posting-host="blaine.gmane.org:116.202.254.214"; logging-data="31315"; mail-complaints-to="usenet@ciao.gmane.io" User-Agent: Gnus/5.13 (Gnus v5.13) Cc: jeberger@free.fr, 61748@debbugs.gnu.org To: =?UTF-8?Q?Jo=C3=A3o_?= =?UTF-8?Q?T=C3=A1vora?= Original-X-From: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane-mx.org@gnu.org Thu Mar 02 10:16:05 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 1pXf2i-0007zy-Mn for geb-bug-gnu-emacs@m.gmane-mx.org; Thu, 02 Mar 2023 10:16:04 +0100 Original-Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1pXf1o-0008Cu-75; Thu, 02 Mar 2023 04:15:09 -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 1pXf1k-00086s-8G for bug-gnu-emacs@gnu.org; Thu, 02 Mar 2023 04:15:04 -0500 Original-Received: from debbugs.gnu.org ([209.51.188.43]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1pXf1i-0000QU-QM for bug-gnu-emacs@gnu.org; Thu, 02 Mar 2023 04:15:03 -0500 Original-Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1pXf1i-0007K1-Ft for bug-gnu-emacs@gnu.org; Thu, 02 Mar 2023 04:15:02 -0500 X-Loop: help-debbugs@gnu.org Resent-From: Michael Albinus Original-Sender: "Debbugs-submit" Resent-CC: bug-gnu-emacs@gnu.org Resent-Date: Thu, 02 Mar 2023 09:15:02 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 61748 X-GNU-PR-Package: emacs Original-Received: via spool by 61748-submit@debbugs.gnu.org id=B61748.167774846728088 (code B ref 61748); Thu, 02 Mar 2023 09:15:02 +0000 Original-Received: (at 61748) by debbugs.gnu.org; 2 Mar 2023 09:14:27 +0000 Original-Received: from localhost ([127.0.0.1]:55749 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1pXf19-0007Iy-Ce for submit@debbugs.gnu.org; Thu, 02 Mar 2023 04:14:27 -0500 Original-Received: from mout.gmx.net ([212.227.15.18]:47083) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1pXf17-0007Ik-Dw for 61748@debbugs.gnu.org; Thu, 02 Mar 2023 04:14:26 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=gmx.de; s=s31663417; t=1677748458; i=michael.albinus@gmx.de; bh=o6alnf2JaX3YlhLX/i1X4a6E9SeZhVsfvWwaw8sH9bU=; h=X-UI-Sender-Class:From:To:Cc:Subject:In-Reply-To:References:Date; b=roJliiRO8raUqxd5f214NPEn8BCilU6bKofmxbNIXSr9cCKDFcuHsBQBuWRd28LfS p0+E7mePHsl8oh4K/9H67ioPsGxrzMrk+hsMhsYndnCjxwcnyu78W0rO1L327b1sjK MqYwuXgYccBS4Rkypq8O3sVIgGLI+c0JGe5o46jE/1poYNK7CD1KHN7J/tqWglcXqP 4jFaKGhSv2e7rSuogt9gvRaXe9QfI7oEGLuWVfU/eWOHfq/NZOy1TUyrlecB5J78z1 rslDM7CoKfdYquiP0cWl4yw2lw9rlm25Wbil5hfkbMRwwfAQsD9zDQtyJLQuubX/HJ u9uPY1tIXQN9A== X-UI-Sender-Class: 724b4f7f-cbec-4199-ad4e-598c01a50d3a Original-Received: from gandalf.gmx.de ([185.89.39.22]) by mail.gmx.net (mrgmx004 [212.227.17.190]) with ESMTPSA (Nemesis) id 1MJE6L-1psD4L1AwV-00Kksx; Thu, 02 Mar 2023 10:14:18 +0100 In-Reply-To: ("=?UTF-8?Q?Jo=C3=A3o_?= =?UTF-8?Q?T=C3=A1vora?="'s message of "Tue, 28 Feb 2023 16:28:11 +0000") X-Provags-ID: V03:K1:r4/CEEqEnPBX6dcFa4JW7Dzn5Lxwd6lhx3MWcFy4/J8k+nBE+31 Qcy+qjjwDMW98wgijbBNmXyqNr9I9jJzHFCOrh0VYqS2bZNS6SsmSjwzmvId5m4HXvbM8kk BZz8hUey793qQ9UeTEONlqSeb2zFbQ1VcrFwhjIGP9SFbCCfbbr4Mq/v5m1D2IqJAyUzuUx iHZ0Ls+Y8k0CnkYkdeqcw== UI-OutboundReport: notjunk:1;M01:P0:8kxUlLzaozw=;ZdgdcAo0SmozKryisMFsscD9tTy WJibo7DzjYvezTlWcsOxfBArD8weJYvB5yq0mjr74LLJ7NJVpdlBOIV82cvaYlLD2NSGCp/1p Rc4a6y7CwlO8W+bh8HTGz5sq1vCOLUSTjc/1Dx+oeVHOTMuDKisVI0h9YfuelCrUxrbMatJwY /9ceSFSv1okeZpBiuZqSYT0EKaLEWIjhSp/TCr3qR84xgM9zqgSwO4/TFkWJfEKtFBvxSAWUk lmi+GcY+AC6+W3MA9gxx8yoBzVFiH+B2t8BUDVV5wg4XBg3CRoTu2mwD1rP1Bm6HnebLLuea5 kuVprtBmD2DZTZDW+47EdbXY22jmY/0Em7dGWRz4OujOAxl8I89r+cbITb4jjvlkKwlLzc1P3 wDI2BbVcKbs3mc3Jm6a7xyBg99h37t2m15Tep5SWyB3COkYKU2bdd8W2SQjqUr7jeO6ZfmAqn qwlDF3KCgiHo/Tg4q1LyOT78IYS6JpQRzvQ+L1NjQ5UiwBzzA8h1UWUYljj2NeWkdtunzKjhd M0i0WcJurtMaFkAZk8SZsstEPItOxU/o47EadrAmElcIk/i6xfx15U8NES997krWpU0njU45Y 8zvfOmC2IE4pwVmBvA/eM1v59u51OqdI1KZc5n4Iyo1SMF1aMY7Woa0jpi1mj5/H617G1IYmz Pm2/TgBPnEldKZtDpLoxopU3K4qP8ROr3CsB6cUjNf+CcLuKN+ckdFQKzIxBk46aQQWTZJjwL 35P4kx7Js+hk3fZOBHaxfHpCUg3tqLVPD/NYLeAvJn0v+J/nwLG3pCCL4dX1NV0DuYzm3lgk 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:257120 Archived-At: Jo=C3=A3o T=C3=A1vora writes: Hi Jo=C3=A3o, >> >> I don't expect a user to solve it. And a user shouldn't expect that >> >> changing a variable is sufficient. >> > >> > I respectfully but diametrically disagree here. >> > >> > Here's an untested patch that could solve this. It can probably >> > be made smarter by only invalidating the cache when a change >> > to the variable is detected. >> >> Hmm. It will delay startup of remote processes, if tramp-remote-path >> doesn't change for different processes. Perhaps it is negligable, but at >> least for slow connections people will be hit by this. >> >> I'll play with it. Perhaps there is a smarter version of it, as you say. > > Here's another still untested but smarter patch. Caches fully > on until user messes with tramp-remote-path, else immediately > flushed. Just flushing the connection property "remote-path" is not sufficient I believe. You still must cleanup the process. This is because the remote PATH environment is set when starting the process. And I fail to understand, why connection-local variables don't serve the purpose. They are described wrt to the remote path in --8<---------------cut here---------------start------------->8--- (info "(tramp) Remote programs") --8<---------------cut here---------------end--------------->8--- If this isn't sufficient, we must improve this, instead of introducing another mechanism. And remember, remote processes are not designed to be as flexible as local processes, where you could simply call (setenv "PATH" ...) Best regards, Michael.