From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!.POSTED.blaine.gmane.org!not-for-mail From: Noam Postavsky Newsgroups: gmane.emacs.bugs Subject: bug#33016: 26.1; (make-process ...) doesn't signal an error, when executable given as absolute Windows path does not exist Date: Thu, 11 Apr 2019 13:34:25 -0400 Message-ID: References: <83y34k728d.fsf@gnu.org> <83bm1f6yfl.fsf@gnu.org> <8336mo63k1.fsf@gnu.org> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="0000000000007fd8c3058644993b" Injection-Info: blaine.gmane.org; posting-host="blaine.gmane.org:195.159.176.226"; logging-data="232400"; mail-complaints-to="usenet@blaine.gmane.org" Cc: 33016@debbugs.gnu.org, Klaus-Dieter Bauer To: Eli Zaretskii Original-X-From: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane.org@gnu.org Thu Apr 11 19:35:30 2019 Return-path: Envelope-to: geb-bug-gnu-emacs@m.gmane.org Original-Received: from lists.gnu.org ([209.51.188.17]) by blaine.gmane.org with esmtps (TLS1.0:RSA_AES_256_CBC_SHA1:256) (Exim 4.89) (envelope-from ) id 1hEdbl-000yLN-ON for geb-bug-gnu-emacs@m.gmane.org; Thu, 11 Apr 2019 19:35:30 +0200 Original-Received: from localhost ([127.0.0.1]:52668 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1hEdbk-0002Ay-Pu for geb-bug-gnu-emacs@m.gmane.org; Thu, 11 Apr 2019 13:35:28 -0400 Original-Received: from eggs.gnu.org ([209.51.188.92]:52655) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1hEdbU-00028h-9W for bug-gnu-emacs@gnu.org; Thu, 11 Apr 2019 13:35:13 -0400 Original-Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1hEdbR-0007ch-JJ for bug-gnu-emacs@gnu.org; Thu, 11 Apr 2019 13:35:11 -0400 Original-Received: from debbugs.gnu.org ([209.51.188.43]:41586) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1hEdbK-0007Ub-Gs for bug-gnu-emacs@gnu.org; Thu, 11 Apr 2019 13:35:07 -0400 Original-Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1hEdbK-0004LA-2j for bug-gnu-emacs@gnu.org; Thu, 11 Apr 2019 13:35:02 -0400 X-Loop: help-debbugs@gnu.org Resent-From: Noam Postavsky Original-Sender: "Debbugs-submit" Resent-CC: bug-gnu-emacs@gnu.org Resent-Date: Thu, 11 Apr 2019 17:35:02 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 33016 X-GNU-PR-Package: emacs Original-Received: via spool by 33016-submit@debbugs.gnu.org id=B33016.155500408316654 (code B ref 33016); Thu, 11 Apr 2019 17:35:02 +0000 Original-Received: (at 33016) by debbugs.gnu.org; 11 Apr 2019 17:34:43 +0000 Original-Received: from localhost ([127.0.0.1]:55130 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1hEdb1-0004KY-74 for submit@debbugs.gnu.org; Thu, 11 Apr 2019 13:34:43 -0400 Original-Received: from mail-oi1-f171.google.com ([209.85.167.171]:36954) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1hEday-0004KL-GE for 33016@debbugs.gnu.org; Thu, 11 Apr 2019 13:34:41 -0400 Original-Received: by mail-oi1-f171.google.com with SMTP id v84so5632754oif.4 for <33016@debbugs.gnu.org>; Thu, 11 Apr 2019 10:34:40 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=Usd7xCP2GBKcaW4Qu5bHgekylGlwnPkcP58yUCoQq7E=; b=lbq6dv/8k2ypX6mfrhBqrlPfQMZtX5VQWY5pFZP+WOu60FOFYMioDdLDMzRyxfOJHN QxCwHzT2vL6dT8Wr3TDjmZXJqyer5K4BD4eeMwkypMvOPV/Cbiqt0Q3yFZJ+nl2tunA0 GGpnf+Ddv2U0vZcjArgLu17USCskib1G1iMc09Ry3LRMiax5M2UlXJ7Zmv7pNDFBCudE 9y0uizccSWo98YGDh3DIuNrhK5S06Z5me06Mh3fEgOWiU/6jLmkO4xjJhNhkNU4kZ6iv 3ARTLenOeD+lOeBAzZmHb6lp5DChWZiitnqLWKRlQXpjZhmkPCskbbH5ho+w5vaXykqX ebJQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=Usd7xCP2GBKcaW4Qu5bHgekylGlwnPkcP58yUCoQq7E=; b=eq5m7z/CX4jkDoGT7xORkc68AFvvQhA2SHdyDqlDFKUluyqECnQu4Ydzw0SSSVsfND OkQkxaJr2LteK1nkHp1ZWWE0Ojrt/O85XNL5dljZkRJhR2rezxfaPhVUlix6kf6f6nlJ +9hJLGC+MftwfM3V73hRvgcd8hsOuCd3poICFh5lv1c7I2VyUYAQbPavr2vShI4IcEKZ LEaK8EZXvqjssvGpYDIJ+aVr7DrojP0yXHTGzikhsIRFxj9Wbhc7crkr+vE9ADObGl1u dghCSQXy3lbxSIr5dSLKR3wKQR8M/ndVOL0S+YSk5O967DsMES5+Bok5k368UDfJAfSn u6og== X-Gm-Message-State: APjAAAUfx7q0vU0ECYE+TjtgWlzQo4vmK4lBO4St0Vw04XtTB3H+hzNW k6fcaBwBD0qbK1Tx1x+PVm9x484WueAfQipzPlQ= X-Google-Smtp-Source: APXvYqwY6Zma38McGvvkifUwjIZgIGadQI7sMU1zJOeP5F6MaFqfMAexqpEYvTZLsNsNTGY3ci4Lh4UNEg5zUADOUzQ= X-Received: by 2002:aca:32c2:: with SMTP id y185mr7602772oiy.177.1555004074663; Thu, 11 Apr 2019 10:34:34 -0700 (PDT) In-Reply-To: <8336mo63k1.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: 209.51.188.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:157512 Archived-At: --0000000000007fd8c3058644993b Content-Type: text/plain; charset="UTF-8" On Thu, 11 Apr 2019 at 10:05, Eli Zaretskii wrote: > > static intmax_t when_entered_debugger; > > > > So I guess we'd need some way of resetting it from Lisp? > > Doesn't it work to simply set its value before the second test? Yes, or in each test, since the tests don't necessarily have knowledge of what order they're called in (I think it's currently alphabetical order of test name). See attached diff (against the state of my v3 patch), but it seems a bit silly to make the variable Lisp accessible just for this obscure test case. I don't see any other way though. > > As far as I can tell, the normal debugger resets it by calling > > recursive-edit, but there's no way to return from that without human > > intervention (I think?). > > Doesn't abort-recursive-edit work noninteractively? Yes, but how can I arrange for it to be called without stopping to read commands from the user first? E.g., in the following abort-recursive-edit is too late to do any good: (progn (recursive-edit) (abort-recursive-edit)) Using pre-command-hook is also too late, the user has to type something to trigger the beginning of a certain command first. (let ((pre-command-hook #'abort-recursive-edit)) (recursive-edit)) > > + ;; On Windows, "nul.FOO" is the empty file for any > > + ;; FOO, in any directory. So this passes Emacs' > > Instead of "is the empty file", I'd say something like "resolves to > the null device, reading from which sets the EOF condition". Hmm, while technically more accurate, it seems like a little too much detail to be useful; I think saying "acts like an always-empty file" should be enough. --0000000000007fd8c3058644993b Content-Type: application/octet-stream; name="reset-when-entered.debugger.diff" Content-Disposition: attachment; filename="reset-when-entered.debugger.diff" Content-Transfer-Encoding: base64 Content-ID: X-Attachment-Id: f_juctl16w0 ZGlmZiAtLWdpdCBpL3NyYy8uZ2RiaW5pdCB3L3NyYy8uZ2RiaW5pdAppbmRleCBiOGIzMDMxLi4z MmIyNjgwIDEwMDY0NAotLS0gaS9zcmMvLmdkYmluaXQKKysrIHcvc3JjLy5nZGJpbml0CkBAIC0x MjI3LDYgKzEyMjcsNyBAQCBpZiBkZWZpbmVkX0hBVkVfWF9XSU5ET1dTCiAgIGJyZWFrIHhfZXJy b3JfcXVpdHRlcgogZW5kCiAKK2JyZWFrIEZyZWRyYXdfZGlzcGxheQogCiAjIFB1dCB0aGUgUHl0 aG9uIGNvZGUgYXQgdGhlIGVuZCBvZiAuZ2RiaW5pdCBzbyB0aGF0IGlmIEdEQiBkb2VzIG5vdAog IyBzdXBwb3J0IFB5dGhvbiwgR0RCIHdpbGwgZG8gYWxsIHRoZSBhYm92ZSBpbml0aWFsaXphdGlv bnMgYmVmb3JlCmRpZmYgLS1naXQgaS9zcmMvZXZhbC5jIHcvc3JjL2V2YWwuYwppbmRleCBlOWYx MThjLi42YzFhMmJiIDEwMDY0NAotLS0gaS9zcmMvZXZhbC5jCisrKyB3L3NyYy9ldmFsLmMKQEAg LTU5LDcgKzU5LDcgQEAgTGlzcF9PYmplY3QgVnJ1bl9ob29rczsKICAgIHNpZ25hbCB0aGUgZXJy b3IgaW5zdGVhZCBvZiBlbnRlcmluZyBhbiBpbmZpbml0ZSBsb29wIG9mIGRlYnVnZ2VyCiAgICBp bnZvY2F0aW9ucy4gICovCiAKLXN0YXRpYyBpbnRtYXhfdCB3aGVuX2VudGVyZWRfZGVidWdnZXI7 CisvL3N0YXRpYyBpbnRtYXhfdCB3aGVuX2VudGVyZWRfZGVidWdnZXI7CiAKIC8qIFRoZSBmdW5j dGlvbiBmcm9tIHdoaWNoIHRoZSBsYXN0IGBzaWduYWwnIHdhcyBjYWxsZWQuICBTZXQgaW4KICAg IEZzaWduYWwuICAqLwpAQCAtNDE3MCw2ICs0MTcwLDEyIEBAIE5vdGUgdGhhdCBgZGVidWctb24t ZXJyb3InLCBgZGVidWctb24tcXVpdCcgYW5kIGZyaWVuZHMKIHN0aWxsIGRldGVybWluZSB3aGV0 aGVyIHRvIGhhbmRsZSB0aGUgcGFydGljdWxhciBjb25kaXRpb24uICAqLyk7CiAgIFZkZWJ1Z19v bl9zaWduYWwgPSBRbmlsOwogCisgIERFRlNZTSAoUXdoZW5fZW50ZXJlZF9kZWJ1Z2dlciwgIndo ZW4tZW50ZXJlZC1kZWJ1Z2dlciIpOworICBERUZWQVJfSU5UICgid2hlbi1lbnRlcmVkLWRlYnVn Z2VyIiwgd2hlbl9lbnRlcmVkX2RlYnVnZ2VyLAorICAgICAgICAgICAgICBkb2M6IC8qIFRoZSBu dW1iZXIgb2Yga2V5Ym9hcmQgZXZlbnRzIGFzIG9mIGxhc3QgdGltZSBgZGVidWdnZXInIHdhcyBj YWxsZWQuCitVc2VkIHRvIGF2b2lkIGluZmluaXRlIGxvb3BzIGlmIHRoZSBkZWJ1Z2dlciBpdHNl bGYgaGFzIGFuIGVycm9yLgorRG9uJ3Qgc2V0IHRoaXMgdW5sZXNzIHlvdSdyZSBzdXJlIHRoYXQg Y2FuJ3QgaGFwcGVuLiAgKi8pOworCiAgIC8qIFdoZW4gbGV4aWNhbCBiaW5kaW5nIGlzIGJlaW5n IHVzZWQsCiAgICBWaW50ZXJuYWxfaW50ZXJwcmV0ZXJfZW52aXJvbm1lbnQgaXMgbm9uLW5pbCwg YW5kIGNvbnRhaW5zIGFuIGFsaXN0CiAgICBvZiBsZXhpY2FsbHktYm91bmQgdmFyaWFibGUsIG9y ICh0KSwgaW5kaWNhdGluZyBhbiBlbXB0eQpkaWZmIC0tZ2l0IGkvdGVzdC9zcmMvY2FsbHByb2Mt dGVzdHMuZWwgdy90ZXN0L3NyYy9jYWxscHJvYy10ZXN0cy5lbAppbmRleCBhMjcyOGEzLi4yOGY3 OTc1IDEwMDY0NAotLS0gaS90ZXN0L3NyYy9jYWxscHJvYy10ZXN0cy5lbAorKysgdy90ZXN0L3Ny Yy9jYWxscHJvYy10ZXN0cy5lbApAQCAtNDMsNyArNDMsMTIgQEAKICAgKHNraXAtdW5sZXNzIChl cSBzeXN0ZW0tdHlwZSAnd2luZG93cy1udCkpCiAgIChsZXQqICgoZGVidWctb24tZXJyb3IgdCkK ICAgICAgICAgIChoYXZlLWNhbGxlZC1kZWJ1Z2dlciBuaWwpCi0gICAgICAgICAoZGVidWdnZXIg KGxhbWJkYSAoJnJlc3QgXykgKHNldHEgaGF2ZS1jYWxsZWQtZGVidWdnZXIgdCkpKSkKKyAgICAg ICAgIChkZWJ1Z2dlciAobGFtYmRhICgmcmVzdCBfKQorICAgICAgICAgICAgICAgICAgICAgKHNl dHEgaGF2ZS1jYWxsZWQtZGVidWdnZXIgdCkKKyAgICAgICAgICAgICAgICAgICAgIDs7IEFsbG93 IGVudGVyaW5nIHRoZSBkZWJ1Z2dlciBsYXRlciBpbiB0aGUgc2FtZQorICAgICAgICAgICAgICAg ICAgICAgOzsgdGVzdCBydW4sIGJlZm9yZSBnb2luZyBiYWNrIHRvIHRoZSBjb21tYW5kCisgICAg ICAgICAgICAgICAgICAgICA7OyBsb29wLgorICAgICAgICAgICAgICAgICAgICAgKHNldHEgd2hl bi1lbnRlcmVkLWRlYnVnZ2VyIC0xKSkpKQogICAgIChzaG91bGQgKGVxIDpnb3QtZXJyb3IgOzsg Tk9URTogYHNob3VsZC1lcnJvcicgd291bGQgaW5oaWJpdCBkZWJ1Z2dlci4KICAgICAgICAgICAg ICAgICAoY29uZGl0aW9uLWNhc2UtdW5sZXNzLWRlYnVnICgpCiAgICAgICAgICAgICAgICAgICAg IDs7IE9uIFdpbmRvd3MsICJudWwuRk9PIiBpcyB0aGUgZW1wdHkgZmlsZSBmb3IgYW55CmRpZmYg LS1naXQgaS90ZXN0L3NyYy9wcm9jZXNzLXRlc3RzLmVsIHcvdGVzdC9zcmMvcHJvY2Vzcy10ZXN0 cy5lbAppbmRleCAyNWE1MmI3Li42NDBmMjkyIDEwMDY0NAotLS0gaS90ZXN0L3NyYy9wcm9jZXNz LXRlc3RzLmVsCisrKyB3L3Rlc3Qvc3JjL3Byb2Nlc3MtdGVzdHMuZWwKQEAgLTIyMCw3ICsyMjAs MTIgQEAgcHJvY2Vzcy10ZXN0cy0tbWl4YWJsZQogICAoc2tpcC11bmxlc3MgKGVxIHN5c3RlbS10 eXBlICd3aW5kb3dzLW50KSkKICAgKGxldCogKChkZWJ1Zy1vbi1lcnJvciB0KQogICAgICAgICAg KGhhdmUtY2FsbGVkLWRlYnVnZ2VyIG5pbCkKLSAgICAgICAgIChkZWJ1Z2dlciAobGFtYmRhICgm cmVzdCBfKSAoc2V0cSBoYXZlLWNhbGxlZC1kZWJ1Z2dlciB0KSkpKQorICAgICAgICAgKGRlYnVn Z2VyIChsYW1iZGEgKCZyZXN0IF8pCisgICAgICAgICAgICAgICAgICAgICAoc2V0cSBoYXZlLWNh bGxlZC1kZWJ1Z2dlciB0KQorICAgICAgICAgICAgICAgICAgICAgOzsgQWxsb3cgZW50ZXJpbmcg dGhlIGRlYnVnZ2VyIGxhdGVyIGluIHRoZSBzYW1lCisgICAgICAgICAgICAgICAgICAgICA7OyB0 ZXN0IHJ1biwgYmVmb3JlIGdvaW5nIGJhY2sgdG8gdGhlIGNvbW1hbmQKKyAgICAgICAgICAgICAg ICAgICAgIDs7IGxvb3AuCisgICAgICAgICAgICAgICAgICAgICAoc2V0cSB3aGVuLWVudGVyZWQt ZGVidWdnZXIgLTEpKSkpCiAgICAgKHNob3VsZCAoZXEgOmdvdC1lcnJvciA7OyBOT1RFOiBgc2hv dWxkLWVycm9yJyB3b3VsZCBpbmhpYml0IGRlYnVnZ2VyLgogICAgICAgICAgICAgICAgIChjb25k aXRpb24tY2FzZS11bmxlc3MtZGVidWcgKCkKICAgICAgICAgICAgICAgICAgICAgOzsgRW1hY3Mg ZG9lc24ndCBzZWFyY2ggZm9yIGFic29sdXRlIGZpbGVuYW1lcywgc28K --0000000000007fd8c3058644993b--