emacs-diffs
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

master 083707b: Don't leak result of nested byte-compilation to outer le


From: Lars Ingebrigtsen
Subject: master 083707b: Don't leak result of nested byte-compilation to outer level
Date: Tue, 27 Oct 2020 04:47:38 -0400 (EDT)

branch: master
commit 083707bbab74a7fcc60fa566156224924ffc3c64
Author: Paul Pogonyshev <pogonyshev@gmail.com>
Commit: Lars Ingebrigtsen <larsi@gnus.org>

    Don't leak result of nested byte-compilation to outer level
    
    * lisp/emacs-lisp/bytecomp.el (byte-compile-file): Bind
    `byte-compiler-error-flag' instead of setting it (bug#41065).
    This fixes a problem of "leaking" the flag when compiling
    something that then compiles something that errors out (i.e., an
    "inner" compile).
---
 lisp/emacs-lisp/bytecomp.el | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/lisp/emacs-lisp/bytecomp.el b/lisp/emacs-lisp/bytecomp.el
index a547b67..cbda16d 100644
--- a/lisp/emacs-lisp/bytecomp.el
+++ b/lisp/emacs-lisp/bytecomp.el
@@ -1938,7 +1938,7 @@ See also `emacs-lisp-byte-compile-and-load'."
         (byte-compile--known-dynamic-vars
          (byte-compile--load-dynvars (getenv "EMACS_DYNVARS_FILE")))
        target-file input-buffer output-buffer
-       byte-compile-dest-file)
+       byte-compile-dest-file byte-compiler-error-flag)
     (setq target-file (byte-compile-dest-file filename))
     (setq byte-compile-dest-file target-file)
     (with-current-buffer
@@ -2000,7 +2000,6 @@ See also `emacs-lisp-byte-compile-and-load'."
          'no-byte-compile)
       (when byte-compile-verbose
        (message "Compiling %s..." filename))
-      (setq byte-compiler-error-flag nil)
       ;; It is important that input-buffer not be current at this call,
       ;; so that the value of point set in input-buffer
       ;; within byte-compile-from-buffer lingers in that buffer.



reply via email to

[Prev in Thread] Current Thread [Next in Thread]