emacs-diffs
[Top][All Lists]
Advanced

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

master df842a737d6: * lisp/emacs-lisp/package.el (package-activate-all):


From: Stefan Monnier
Subject: master df842a737d6: * lisp/emacs-lisp/package.el (package-activate-all): Fix second-order warning
Date: Tue, 5 Dec 2023 15:02:53 -0500 (EST)

branch: master
commit df842a737d6cb2b70d9672a64826e04bb8249167
Author: Stefan Monnier <monnier@iro.umontreal.ca>
Commit: Stefan Monnier <monnier@iro.umontreal.ca>

    * lisp/emacs-lisp/package.el (package-activate-all): Fix second-order 
warning
---
 lisp/emacs-lisp/package.el | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

diff --git a/lisp/emacs-lisp/package.el b/lisp/emacs-lisp/package.el
index 0a87f771ab9..bed6e74c921 100644
--- a/lisp/emacs-lisp/package.el
+++ b/lisp/emacs-lisp/package.el
@@ -1732,8 +1732,13 @@ The variable `package-load-list' controls which packages 
to load."
                  t)))
         (progn
           (require 'package)
-          (declare-function package--activate-all "package" ())
-          (package--activate-all))))))
+          ;; Silence the "unknown function" warning when this is compiled
+          ;; inside `loaddefs.el'.
+          ;; FIXME: We use `with-no-warnings' because the effect of
+          ;; `declare-function' is currently not scoped, so if we use
+          ;; it here, we end up with a redefinition warning instead :-)
+          (with-no-warnings
+            (package--activate-all)))))))
 
 (defun package--activate-all ()
   (dolist (elt (package--alist))



reply via email to

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