From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.io!.POSTED.ciao.gmane.io!not-for-mail From: Michael Albinus Newsgroups: gmane.emacs.bugs Subject: bug#41099: 28.0.50; TRAMP process-file ignores exit status of remote process Date: Thu, 14 May 2020 17:49:40 +0200 Message-ID: <871rnmjz3f.fsf@gmx.de> References: <871rnyyyio.fsf@gmx.de> <87sggdmp9s.fsf@gmx.de> <87eerxwbjo.fsf@gmx.de> <87zhalrxyf.fsf@gmx.de> <87o8r1rmwv.fsf@gmx.de> <871rnwrvvn.fsf@gmx.de> <87v9kzl2ga.fsf@gmail.com> <87a72akcgz.fsf@gmx.de> <85zhaafw4k.fsf@gmail.com> Mime-Version: 1.0 Content-Type: text/plain Injection-Info: ciao.gmane.io; posting-host="ciao.gmane.io:159.69.161.202"; logging-data="7577"; mail-complaints-to="usenet@ciao.gmane.io" User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/28.0.50 (gnu/linux) Cc: 41099@debbugs.gnu.org, Noam Postavsky To: Philipp Stephani Original-X-From: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane-mx.org@gnu.org Thu May 14 17:50:44 2020 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 1jZG8C-0001t1-45 for geb-bug-gnu-emacs@m.gmane-mx.org; Thu, 14 May 2020 17:50:44 +0200 Original-Received: from localhost ([::1]:36710 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1jZG8A-0005FC-Vk for geb-bug-gnu-emacs@m.gmane-mx.org; Thu, 14 May 2020 11:50:43 -0400 Original-Received: from eggs.gnu.org ([2001:470:142:3::10]:54548) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1jZG7X-00053t-1B for bug-gnu-emacs@gnu.org; Thu, 14 May 2020 11:50:03 -0400 Original-Received: from debbugs.gnu.org ([209.51.188.43]:51218) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1jZG7W-0007HJ-MK for bug-gnu-emacs@gnu.org; Thu, 14 May 2020 11:50:02 -0400 Original-Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1jZG7W-0007uq-K1 for bug-gnu-emacs@gnu.org; Thu, 14 May 2020 11:50:02 -0400 X-Loop: help-debbugs@gnu.org Resent-From: Michael Albinus Original-Sender: "Debbugs-submit" Resent-CC: bug-gnu-emacs@gnu.org Resent-Date: Thu, 14 May 2020 15:50:02 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 41099 X-GNU-PR-Package: emacs Original-Received: via spool by 41099-submit@debbugs.gnu.org id=B41099.158947139030405 (code B ref 41099); Thu, 14 May 2020 15:50:02 +0000 Original-Received: (at 41099) by debbugs.gnu.org; 14 May 2020 15:49:50 +0000 Original-Received: from localhost ([127.0.0.1]:34531 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1jZG7J-0007uL-So for submit@debbugs.gnu.org; Thu, 14 May 2020 11:49:50 -0400 Original-Received: from mout.gmx.net ([212.227.15.18]:40639) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1jZG7H-0007u3-RV for 41099@debbugs.gnu.org; Thu, 14 May 2020 11:49:48 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=gmx.net; s=badeba3b8450; t=1589471381; bh=IYt6q01CdZT94BvTSGPCOTK0/i3S2XszuKg/691FdGE=; h=X-UI-Sender-Class:From:To:Cc:Subject:References:Date:In-Reply-To; b=gf+CulKKewkR9TXUm5lCH2AMTcPyOIG8FkEd5MTcEofcap6xRlFoxmk60d+DE2Qwn bvcxLBTNZCrRNlOz471EE5cMYMSFL7vTJHiup5JWQia5wphMa+pHft0DUAkMYrQDNK 1i9ufUupFV4ATZ6smU8q+eGSKfAJ1Pf5eFlmUdxA= X-UI-Sender-Class: 01bb95c1-4bf8-414a-932a-4f6e2808ef9c Original-Received: from gandalf.gmx.de ([79.140.119.151]) by mail.gmx.com (mrgmx005 [212.227.17.190]) with ESMTPSA (Nemesis) id 1Mf0BM-1iu9vz0EDy-00gaYd; Thu, 14 May 2020 17:49:41 +0200 In-Reply-To: (Philipp Stephani's message of "Thu, 14 May 2020 16:48:22 +0200") X-Provags-ID: V03:K1:DUfugTiDA1m7/vYC2lMtjzDX46ee5o9469hM5RKomYMZbo+pv6z 0yus8OQbDHJZi1bxSgDY3mUd+NNvI+WaBtGct9jMYf3mMW0rWO4IaEGoP5KOJu2NpVYie6P k1zRIT10W1CyWSqGOHNeN+oTt5AZ3cx32JtuRHC85cojlHRJ3nMIpj1+XDBzdAzWdmL8fFL lluy49MuE3gZywpI7db2w== X-UI-Out-Filterresults: notjunk:1;V03:K0:3/Q8I0xZ2lU=:qop0fOm1aj5VhuJlXymTNx zFjw+Nv2/JrRo9meTKSx7hcumYUqgtNsZzBwDSx4pyR4DDLGDUMfHIyzhtdI0apn5NG4HoyDr JdWSUxCvu13BeaoGIua0b04WljtmAPwFSAjGlsLBN5EQwUSr+3nvjbWsZ8PSxxXXLwB3yZG4y xKiHko5xlbxx0GJc0Usaq/1ZNCjIguaOtnk3pH/cSeSzkJRE9CRNRJ//y0Q3+BXV2VB7r+EGd BRq0h7YNtbAu0qkhPMItciWLzWN+BDINkkAFt7oefGkDcu2HXzVPyxy+3o6rm4al2nF8/+VDW MzPR85Y8nZm55Y2sA1CCMGjEW/WQZIirq++HzIrryVH0mr+vLtf9PgDVKR5E3/sJu3A4GDKNk gv9YFGLRN/dD0EZsXL/PiNCEHtgu6vIRxzkzzGV+CgtiFInM1zC2Jkru2j198ymPri1x3HQYw 8k1SsMyGsKjVqRqtZwve+1uocj6oc5/6B5h9xtFYr/t4h49U40080cJlDwYHXxjYGwaaXSGAf xhJjuKDXK8skiUDzhruZ45AnQc2VJ8C/zxcQabCGwy3/C5KaYRlXrDUVVpIJUB8m/FqEAMqmc zM+DiETmtZpMTV3fQLkHIXo8mtNwbLqeadxy3sQlEcO4zz3WHp30OCUh0DtmCsee52iB1WhPA XWwHNcSGQSVS+0DMkCZoN1dEZiMoq6sZPVXyYleIaAp/WxUFiHBL/w2taxY/V6lxHSP+aK25i UCOR/LhZd8gWH2oGDSH7V86Y1x4ehweum5W2mpWqOYZwq5Bs50sAwAs+yO/19askn+zsQ67X 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" Xref: news.gmane.io gmane.emacs.bugs:180218 Archived-At: Philipp Stephani writes: Hi Philipp & Noam, >> >> - we don't return a string in case a signal has interrupted the process, >> >> Since we don't have a reliable way to detect signals, I think this is >> the only viable option. > > I'd expect the vast majority of programs to avoid such exit codes, > precisely because they would want to allow portable usage in shell > scripts. So I expect that the current behavior in master provides the > "correct" result in the majority of cases. I understand (and sympathize) both positions. However, Tramp has returned for decades no strings for process-file, so I don't expect any code in the wild which expects this. What about a user option, tramp-process-file-return-signal-string? If non-nil, it returns a string when a signal is assumed for exit codes > 128. If nil (the default), the exit code is always returned as natnum. This would also fit the principle of least surprise, because Tramp hasn't returned strings since ever. A better variable name would also be appreciated :-) Best regards, Michael.