From 3627b82fd33663ff0e9b6252e2836fd33cc889fd Mon Sep 17 00:00:00 2001 From: Henrik Lissner Date: Sat, 16 Sep 2023 15:32:54 +0200 Subject: [PATCH] docs: revise and fix nerd-icons doctor check Also fixes an issue where errors weren't counted, so the explanation would never appear. Fix: #7431 --- lisp/cli/doctor.el | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/lisp/cli/doctor.el b/lisp/cli/doctor.el index d8fd50991..5b4ecd5dc 100644 --- a/lisp/cli/doctor.el +++ b/lisp/cli/doctor.el @@ -260,12 +260,13 @@ in." (dolist (font nerd-icons-font-names) (if (save-excursion (re-search-backward font nil t)) (success! "Found font %s" font) - (print! (warn "Warning: couldn't find %S font") font))) + (print! (warn "%S font is not installed on your system") font) + (cl-incf errors))) (when (> errors 0) - (explain! "Some nerd-icons fonts were missing.\n\n" - "You can install them by running `M-x nerd-icons-install-fonts' within Emacs.\n" - "This could also mean you've installed them in non-standard locations, in which " - "case feel free to ignore this warning."))))))))) + (explain! "Some needed fonts are not properly installed on your system. To download and " + "install them, run `M-x nerd-icons-install-fonts' from within Doom Emacs. " + "However, on Windows this command will only download them; the fonts must " + "be installed manually afterwards."))))))))) (print! (start "Checking for stale elc files in your DOOMDIR...")) (when (file-directory-p doom-user-dir)