* lisp/emacs-lisp/package.el (package-activate-all): Fix second-order warning

df842a73 · Stefan Monnier · 4f0239d8 · df842a73
Commit df842a73 authored Dec 05, 2023 by Stefan Monnier
Hide whitespace changes
Inline Side-by-side

Showing with 7 additions and 2 deletions

lisp/emacs-lisp/package.el lisp/emacs-lisp/package.el +7 -2

No files found.
--- 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))