From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!.POSTED!not-for-mail From: Reuben Thomas Newsgroups: gmane.emacs.bugs Subject: bug#18133: Suppressing asynchronous command output Date: Wed, 28 Jun 2017 22:45:43 +0100 Message-ID: References: <83eg100vy5.fsf@gnu.org> <585C132B.1030709@gmx.at> <585C347D.9050309@gmx.at> <585D740B.40303@gmx.at> <585D8120.1090300@gmx.at> <87fula1dbk.fsf@mail.linkov.net> <83r34tx53w.fsf@gnu.org> <83eg0twwje.fsf@gnu.org> <83r34sujou.fsf@gnu.org> <8737h6z63w.fsf@mail.linkov.net> <87mvfdowg5.fsf@mail.linkov.net> <83zijch4nc.fsf@gnu.org> NNTP-Posting-Host: blaine.gmane.org Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="001a113d4232334aa005530c1be1" X-Trace: blaine.gmane.org 1498686383 18062 195.159.176.226 (28 Jun 2017 21:46:23 GMT) X-Complaints-To: usenet@blaine.gmane.org NNTP-Posting-Date: Wed, 28 Jun 2017 21:46:23 +0000 (UTC) Cc: 18133@debbugs.gnu.org, Juri Linkov To: Eli Zaretskii Original-X-From: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane.org@gnu.org Wed Jun 28 23:46:14 2017 Return-path: Envelope-to: geb-bug-gnu-emacs@m.gmane.org Original-Received: from lists.gnu.org ([208.118.235.17]) by blaine.gmane.org with esmtp (Exim 4.84_2) (envelope-from ) id 1dQKmk-00040i-ID for geb-bug-gnu-emacs@m.gmane.org; Wed, 28 Jun 2017 23:46:06 +0200 Original-Received: from localhost ([::1]:35804 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1dQKmp-0003sD-MC for geb-bug-gnu-emacs@m.gmane.org; Wed, 28 Jun 2017 17:46:11 -0400 Original-Received: from eggs.gnu.org ([2001:4830:134:3::10]:55934) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1dQKmk-0003s8-6a for bug-gnu-emacs@gnu.org; Wed, 28 Jun 2017 17:46:07 -0400 Original-Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1dQKmh-0001z7-2J for bug-gnu-emacs@gnu.org; Wed, 28 Jun 2017 17:46:06 -0400 Original-Received: from debbugs.gnu.org ([208.118.235.43]:40363) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1dQKmg-0001yq-S7 for bug-gnu-emacs@gnu.org; Wed, 28 Jun 2017 17:46:02 -0400 Original-Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1dQKmg-0006ep-6R for bug-gnu-emacs@gnu.org; Wed, 28 Jun 2017 17:46:02 -0400 X-Loop: help-debbugs@gnu.org Resent-From: Reuben Thomas Original-Sender: "Debbugs-submit" Resent-CC: bug-gnu-emacs@gnu.org Resent-Date: Wed, 28 Jun 2017 21:46:02 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 18133 X-GNU-PR-Package: emacs X-GNU-PR-Keywords: Original-Received: via spool by 18133-submit@debbugs.gnu.org id=B18133.149868635325577 (code B ref 18133); Wed, 28 Jun 2017 21:46:02 +0000 Original-Received: (at 18133) by debbugs.gnu.org; 28 Jun 2017 21:45:53 +0000 Original-Received: from localhost ([127.0.0.1]:43040 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1dQKmW-0006eS-Vq for submit@debbugs.gnu.org; Wed, 28 Jun 2017 17:45:53 -0400 Original-Received: from mail-oi0-f42.google.com ([209.85.218.42]:36150) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1dQKmU-0006eF-Di for 18133@debbugs.gnu.org; Wed, 28 Jun 2017 17:45:51 -0400 Original-Received: by mail-oi0-f42.google.com with SMTP id p187so52530645oif.3 for <18133@debbugs.gnu.org>; Wed, 28 Jun 2017 14:45:50 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=sc3d.org; s=google; h=mime-version:in-reply-to:references:from:date:message-id:subject:to :cc; bh=AQv8DzXQyS8uCREWaCHaSIjT337b1ffJQLWl+JZOmeY=; b=mS6h/90VpPLsbKKA5HawgBBypFMY5L/ljaKqf8XIVwy9/2Vw1Qy/GIEHzSoOl5hwA5 QxcT7sUBRwdLoD52umKfSM7Js0xwI+yNgz4iKzBimKuGu31drob1+TRqxciDUEmySyWI nxtjlh/fGLZYO5eVaG6QH8UCbIpT3Cy7b2c9Y= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:in-reply-to:references:from:date :message-id:subject:to:cc; bh=AQv8DzXQyS8uCREWaCHaSIjT337b1ffJQLWl+JZOmeY=; b=l4a5vBJSGO9R110wUg+uiQQ+32ELng6WDrOSdw1c+0V+iICe/RNXwimksqn2/GUFy4 oAMfPx/ZN+lcQO2Mppdgt0ZF7zzDDZkw/vS7ifwYV6kBEaMI6u6BtwgncjIgsrQv+sFI FFul2bdD6gPw0Bx2gsaT39A4+SD4LxuZU0zcuZT0z9GUzcCZbGE1bE/8uOF370P/regO RFt6EnWVPaEsnL/qtEAEkLGzlopbn6ItKR4bVq6MoMW07fh1d77rGDqf8QV0oBFNZaKI DYRo2LJ3FaliTQq6/ey8TRV9wOGTUQylA5XVK8Iw/9L1izqZmftTmn8lN/6hxWqsu5/O EAcA== X-Gm-Message-State: AKS2vOxFU9MRLIc/YrM99Vakl3AHbXtDKj4fYaf0vwqeaWiq+KNXvcdN Fxm0zCcfIeHScmwnu7IBdDMx7iz4g8c1 X-Received: by 10.202.180.4 with SMTP id d4mr8515986oif.188.1498686344590; Wed, 28 Jun 2017 14:45:44 -0700 (PDT) Original-Received: by 10.202.193.136 with HTTP; Wed, 28 Jun 2017 14:45:43 -0700 (PDT) In-Reply-To: <83zijch4nc.fsf@gnu.org> X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 208.118.235.43 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.org@gnu.org Original-Sender: "bug-gnu-emacs" Xref: news.gmane.org gmane.emacs.bugs:134001 Archived-At: --001a113d4232334aa005530c1be1 Content-Type: multipart/alternative; boundary="001a113d4232334a9a05530c1bdf" --001a113d4232334a9a05530c1bdf Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable On 31 December 2016 at 08:41, Eli Zaretskii wrote: > > From: Reuben Thomas > > Date: Sat, 31 Dec 2016 00:19:21 +0000 > > Cc: Eli Zaretskii , martin rudalics , > 18133@debbugs.gnu.org > > > > Thanks, Juri; can you converge with Eli, and then I'll have a coherent > proposal I can program? > > I'm okay with Juri's approach as well. > =E2=80=8BHere is a patch that attempts to implement Juri's approach.=E2=80= =8B =E2=80=8BHowever, at the moment it doesn't work. I think the problem is to = do with combining process filters. As suggested in the Emacs manual, I use add-function to combine the new process filter with the default comint-output-filter. But no matter what I try, the original filter does not seem to be run any more, even if I pass a trivial function to add-function. I've studied the documentation, and I can't see what I'm doing wrong. --=20 https://rrt.sc3d.org --001a113d4232334a9a05530c1bdf Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable
On 3= 1 December 2016 at 08:41, Eli Zaretskii <eliz@gnu.org> wrote:
=
> From: Reuben Thomas <rrt@sc3d.org>
> Date: Sat, 31 Dec 2016 00:19:21 +0000
> Cc: Eli Zaretskii <eliz@gnu.org= >, martin rudalics <rudalics@gmx.a= t>, 18133@debbugs.gnu.org
>
> Thanks, Juri; can you converge with Eli, and then I'll have a cohe= rent proposal I can program?

I'm okay with Juri's approach as well.

=E2=80=8BHere is a patch that attempts to implement Juri's approach.= =E2=80=8B

= =E2=80=8BHowever, at the moment it doesn't work. I think the problem is= to do with combining process filters. As suggested in the Emacs manual, I = use add-function to combine the new process filter with the default comint-= output-filter. But no matter what I try, the original filter does not seem = to be run any more, even if I pass a trivial function to add-function. I= 9;ve studied the documentation, and I can't see what I'm doing wron= g.

--
--001a113d4232334a9a05530c1bdf-- --001a113d4232334aa005530c1be1 Content-Type: text/x-patch; charset="US-ASCII"; name="0001-Allow-async-command-output-buffer-to-be-shown-only-o.patch" Content-Disposition: attachment; filename="0001-Allow-async-command-output-buffer-to-be-shown-only-o.patch" Content-Transfer-Encoding: base64 X-Attachment-Id: f_j4hj35750 RnJvbSBlZTc5ZWFkOWYyZjEzMmU4NTgwNmM3NWYzMGVjODYxYmIwOTRmMzg3IE1vbiBTZXAgMTcg MDA6MDA6MDAgMjAwMQpGcm9tOiBSZXViZW4gVGhvbWFzIDxycnRAc2MzZC5vcmc+CkRhdGU6IFdl ZCwgMjggSnVuIDIwMTcgMjI6NDA6MzMgKzAxMDAKU3ViamVjdDogW1BBVENIXSBBbGxvdyBhc3lu YyBjb21tYW5kIG91dHB1dCBidWZmZXIgdG8gYmUgc2hvd24gb25seSBvbiBvdXRwdXQKCiogbGlz cC9zaW1wbGUuZWwgKGFzeW5jLXNoZWxsLWNvbW1hbmQtZGlzcGxheS1idWZmZXIpOiBBZGQKZGVm Y3VzdG9tLgooc2hlbGwtY29tbWFuZCk6IFVzZSB0aGUgbmV3IGRlZmN1c3RvbSB0byBkZXRlcm1p bmUgd2hldGhlciB0byBzaG93CnRoZSBidWZmZXIgaW1tZWRpYXRlbHksIG9yIGFkZCBhIHByb2Nl c3MgZmlsdGVyIHRoYXQgc2hvd3MgaXQgb25seQp3aGVuIHRoZXJlIGlzIHNvbWUgb3V0cHV0LgoK VGhhbmtzIHRvIEp1cmkgTGlua292IGFuZCBFbGkgWmFyZXRza2lpIGZvciBhZHZpY2UgYW5kIGd1 aWRhbmNlLgotLS0KIGxpc3Avc2ltcGxlLmVsIHwgMjMgKysrKysrKysrKysrKysrKysrKysrLS0K IDEgZmlsZSBjaGFuZ2VkLCAyMSBpbnNlcnRpb25zKCspLCAyIGRlbGV0aW9ucygtKQoKZGlmZiAt LWdpdCBhL2xpc3Avc2ltcGxlLmVsIGIvbGlzcC9zaW1wbGUuZWwKaW5kZXggYTU1NjVhYi4uMmYy ODg3ZSAxMDA2NDQKLS0tIGEvbGlzcC9zaW1wbGUuZWwKKysrIGIvbGlzcC9zaW1wbGUuZWwKQEAg LTMyNzEsNiArMzI3MSwxNyBAQCBhc3luYy1zaGVsbC1jb21tYW5kLWJ1ZmZlcgogICA6Z3JvdXAg J3NoZWxsCiAgIDp2ZXJzaW9uICIyNC4zIikKIAorKGRlZmN1c3RvbSBhc3luYy1zaGVsbC1jb21t YW5kLWRpc3BsYXktYnVmZmVyIHQKKyAgIldoZXRoZXIgdG8gZGlzcGxheSB0aGUgY29tbWFuZCBi dWZmZXIgaW1tZWRpYXRlbHkuCitJZiB0LCBkaXNwbGF5IHRoZSBidWZmZXIgaW1tZWRpYXRlbHk7 IGlmIG5pbCwgd2FpdCB1bnRpbCB0aGVyZQoraXMgb3V0cHV0LiIKKyAgOnR5cGUgJyhjaG9pY2Ug KGNvbnN0IDp0YWcgIkRpc3BsYXkgYnVmZmVyIGltbWVkaWF0ZWx5IgorCQkJdCkKKwkJIChjb25z dCA6dGFnICJEaXNwbGF5IGJ1ZmZlciBvbiBvdXRwdXQiCisJCQluaWwpKQorICA6Z3JvdXAgJ3No ZWxsCisgIDp2ZXJzaW9uICIyNi4xIikKKwogKGRlZnVuIHNoZWxsLWNvbW1hbmQtLXNhdmUtcG9z LW9yLWVyYXNlICgpCiAgICJTdG9yZSBhIGJ1ZmZlciBwb3NpdGlvbiBvciBlcmFzZSB0aGUgYnVm ZmVyLgogU2VlIGBzaGVsbC1jb21tYW5kLWRvbnQtZXJhc2UtYnVmZmVyJy4iCkBAIC0zNTE3LDcg KzM1MjgsNiBAQCBzaGVsbC1jb21tYW5kCiAJCSAgICAoc2V0cSBidWZmZXIgKGdldC1idWZmZXIt Y3JlYXRlCiAJCQkJICAob3Igb3V0cHV0LWJ1ZmZlciAiKkFzeW5jIFNoZWxsIENvbW1hbmQqIikp KSkpKQogCQkod2l0aC1jdXJyZW50LWJ1ZmZlciBidWZmZXIKLQkJICAoZGlzcGxheS1idWZmZXIg YnVmZmVyICcobmlsIChhbGxvdy1uby13aW5kb3cgLiB0KSkpCiAgICAgICAgICAgICAgICAgICAo c2hlbGwtY29tbWFuZC0tc2F2ZS1wb3Mtb3ItZXJhc2UpCiAJCSAgKHNldHEgZGVmYXVsdC1kaXJl Y3RvcnkgZGlyZWN0b3J5KQogCQkgIChzZXRxIHByb2MgKHN0YXJ0LXByb2Nlc3MgIlNoZWxsIiBi dWZmZXIgc2hlbGwtZmlsZS1uYW1lCkBAIC0zNTI4LDcgKzM1MzgsMTYgQEAgc2hlbGwtY29tbWFu ZAogCQkgIDs7IFVzZSB0aGUgY29taW50IGZpbHRlciBmb3IgcHJvcGVyIGhhbmRsaW5nIG9mIGNh cnJpYWdlIG1vdGlvbgogCQkgIDs7IChzZWUgYGNvbWludC1pbmhpYml0LWNhcnJpYWdlLW1vdGlv bicpLC4KIAkJICAoc2V0LXByb2Nlc3MtZmlsdGVyIHByb2MgJ2NvbWludC1vdXRwdXQtZmlsdGVy KQotCQkgICkpCisgICAgICAgICAgICAgICAgICAoaWYgYXN5bmMtc2hlbGwtY29tbWFuZC1kaXNw bGF5LWJ1ZmZlcgorICAgICAgICAgICAgICAgICAgICAgIChkaXNwbGF5LWJ1ZmZlciBidWZmZXIg JyhuaWwgKGFsbG93LW5vLXdpbmRvdyAuIHQpKSkKKyAgICAgICAgICAgICAgICAgICAgKGFkZC1m dW5jdGlvbiA6YmVmb3JlIChwcm9jZXNzLWZpbHRlciBwcm9jKQorICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgIGAobGFtYmRhIChwcm9jZXNzIHN0cmluZykKKyAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgICAod2hlbiAoYW5kICg9IDAgKGJ1ZmZlci1zaXplIChwcm9j ZXNzLWJ1ZmZlciBwcm9jZXNzKSkpCisgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgICAgICAoZXEgKGJ1ZmZlci1uYW1lIChwcm9jZXNzLWJ1ZmZlciBwcm9jZXNzKSkK KyAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAsKG9y IG91dHB1dC1idWZmZXIgIipBc3luYyBTaGVsbCBDb21tYW5kKiIpKSkKKyAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgICAgICAgICAgIChkaXNwbGF5LWJ1ZmZlciAocHJvY2Vzcy1idWZmZXIg cHJvY2VzcykpKSkKKyAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICApKQorICAgICAg ICAgICAgICAgICAgKSkKIAkgICAgOzsgT3RoZXJ3aXNlLCBjb21tYW5kIGlzIGV4ZWN1dGVkIHN5 bmNocm9ub3VzbHkuCiAJICAgIChzaGVsbC1jb21tYW5kLW9uLXJlZ2lvbiAocG9pbnQpIChwb2lu dCkgY29tbWFuZAogCQkJCSAgICAgb3V0cHV0LWJ1ZmZlciBuaWwgZXJyb3ItYnVmZmVyKSkpKSkp KQotLSAKMi43LjQKCg== --001a113d4232334aa005530c1be1--