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#61024: 29.0.60; [PATCH] Eshell errors out when trying to redirect output of a remote process Date: Mon, 23 Jan 2023 16:50:14 +0100 Message-ID: <877cxdi72h.fsf@gmx.de> References: <230f0e06-f47a-6a0d-93de-222ca16f7213@gmail.com> <83wn5didxu.fsf@gnu.org> Mime-Version: 1.0 Content-Type: text/plain Content-Transfer-Encoding: quoted-printable Injection-Info: ciao.gmane.io; posting-host="blaine.gmane.org:116.202.254.214"; logging-data="16710"; mail-complaints-to="usenet@ciao.gmane.io" User-Agent: Gnus/5.13 (Gnus v5.13) Cc: 61024@debbugs.gnu.org, Jim Porter To: Eli Zaretskii Original-X-From: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane-mx.org@gnu.org Mon Jan 23 16:51:26 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 1pJz6U-00041o-2F for geb-bug-gnu-emacs@m.gmane-mx.org; Mon, 23 Jan 2023 16:51:26 +0100 Original-Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1pJz68-00085u-Ng; Mon, 23 Jan 2023 10:51:04 -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 1pJz67-00085V-9O for bug-gnu-emacs@gnu.org; Mon, 23 Jan 2023 10:51:03 -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 1pJz67-0006H7-1O for bug-gnu-emacs@gnu.org; Mon, 23 Jan 2023 10:51:03 -0500 Original-Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1pJz66-0004w3-DV for bug-gnu-emacs@gnu.org; Mon, 23 Jan 2023 10:51: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: Mon, 23 Jan 2023 15:51:02 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 61024 X-GNU-PR-Package: emacs X-GNU-PR-Keywords: patch Original-Received: via spool by 61024-submit@debbugs.gnu.org id=B61024.167448902518914 (code B ref 61024); Mon, 23 Jan 2023 15:51:02 +0000 Original-Received: (at 61024) by debbugs.gnu.org; 23 Jan 2023 15:50:25 +0000 Original-Received: from localhost ([127.0.0.1]:55131 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1pJz5U-0004uz-LM for submit@debbugs.gnu.org; Mon, 23 Jan 2023 10:50:24 -0500 Original-Received: from mout.gmx.net ([212.227.15.15]:54017) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1pJz5S-0004uj-Df for 61024@debbugs.gnu.org; Mon, 23 Jan 2023 10:50:23 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=gmx.de; s=s31663417; t=1674489015; bh=DW0/gjnCnUBtsaTXDRz5jcuR5lJMI56HbsX+V46b/e8=; h=X-UI-Sender-Class:From:To:Cc:Subject:In-Reply-To:References:Date; b=rDQuAw3F+APYhmHKCQABImpMAaJbvVss1xfF7I/i/EapJkLsGDuvXO2XXd77IDZLS GCc2eXplwTmvtDDgTK1KmSUfwpPsy5nB2NAJ4BThre7AOf2so2pWxQjtYacAeS8Sx+ BWKJhVczQZcytHcA3kyPQYhO9tPwSGclJnciekoj3divH8V4lyFV1mL7AnhnsXoOvb o0ZJ2odtR1ugrl4/DZnZLPh/elnhhCDWv5LKdDIatvxdikWOB57blV8/ilWlOtoDWE crhniClWyvOOiCZbU+6M6g1jm0Fi7vpe2/xSRvNIygTJIWeHkhhLty9tjkq9rzeN0c MqP2FKHJkrdhg== X-UI-Sender-Class: 724b4f7f-cbec-4199-ad4e-598c01a50d3a Original-Received: from gandalf.gmx.de ([185.89.37.45]) by mail.gmx.net (mrgmx004 [212.227.17.190]) with ESMTPSA (Nemesis) id 1MzyuS-1oVspd1M6I-00x32B; Mon, 23 Jan 2023 16:50:15 +0100 In-Reply-To: <83wn5didxu.fsf@gnu.org> (Eli Zaretskii's message of "Mon, 23 Jan 2023 15:21:49 +0200") X-Provags-ID: V03:K1:AkK0PcBABtXhBDT/F7cdY8526fOHOSI+hHpH9ou5+ageKYO0lGN Clr8FCCxThyaXWk2WqaSxLADxloHWMHYyTxyA7h+brzBidgQdvMhZBsFl1ECkB0C0f4FjTe to1ZVGIWLv0tx+st+ZBvYI+TizzRSHxJ4YUUbH9EEKjxluHXXGqVho1lDhX4lleuwQoWtLE vTeZpeo0JTEDWwZbGEYOg== UI-OutboundReport: notjunk:1;M01:P0:OVMxTnJdbXw=;DC6lFVOxqpD6wBOc8uUrkv6a4dQ /NwArPQMFaypKopdKdVkbLO5ENL6rWZ2ak/6vOmpRtmce6g71wLGmHTp6nD6kVh3Md7YEx41D MNrgNtSR7co394hQOxWBo1oZ2pI3lNSP+/npSX/BU3El+gF+iLQ2aIC4/1GeI4LnE19p8r+6J JMsIUSPNh/m2b+aUvyzfohzhYcTIE1mfwHl+taa1oMGyiksXaGt6UQZkbB13aYq8au1Nts0tE CCleB2QDDyhNt6z2iCNWZuvkKjGyROhGDtlkSJci+PJksDVP8O9r32Bo1JqRYm4bW0Ry1d3Ie w1nd9xYc+JyOOjWWw0yYBX/cMLsnJsAvI8eGDd1552AY/ypD0HWzq8SP131Z4MbkRQlNRa0G6 ggKe1xpWVZ7KEaxQVOTrkBN8NoBfiDN/Tq2bCZjqmbBy+OHHuHktOzSYhwmC0/a9uxvOBpyyE 2U9YZhNUwRsBsUd4XqC0djwkxPi7fQhMn0Rnn8I4f51QSqBAA58vCa5B4JEjLohE/qU1WV0WJ buHjNXlSoRM4YwopCQk3NeWTdFuv9RBRaOxTMv5xZ9bCmfz3JZ5vbSiBCSxZ3v7TWe3udiFmK H9A/8btV1d+3gZZnkR4zSqQg0PLyB0afuzeHyTFzam2IHQxthpGhOZ0xb9il+qmyhXh/DWzMu To5X4sPDQjIq4VKKeZs2+hIvYYQdj8Dchi9HkeFtXL0eZNmPC6rV+wsQbO5AvX5q2JSG3tEkt SuRfjLkfriRGIV/YJBm67uZPkm1b0c7oEFuW5nqlDWSi0OThKkoq1bdkXTa2mFM/GzibGZ+s 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:254008 Archived-At: Eli Zaretskii writes: Hi, >> ~ $ cd /sudo:: >> /sudo:root@host:~ # *echo hi there > # >> Wrong type argument: "bufferp echo-stderr" >> >> (Any other Tramp method should reproduce this issue.) That's because th= e >> echo command is being redirected to two places: stdout is going to the >> buffer "foo", and stderr is going to the Eshell buffer (via a pipe >> process). However, that doesn't work for Tramp processes. >> >> Attached is a fix with a test. I think this should definitely go in >> Emacs 29, since it's a pretty bad regression. > > Fine by me, unless Michael objects or has better ideas. Pipe processes are always tricky for Tramp. I *believe* redirecting stdout and stderr to different buffers shall work, see "test6" of tramp-test30-make-process. But I remember also that this was painful to implement and test. >> I'm actually surprised no one has filed a bug on this already. > > Maybe it tells us how many people use this combination? So do we want to reopen this can of worms, and see what happens with Tramp? But perhaps it is a problem in Eshell, which tries to create its own pipe process? Best regards, Michael.