On Fri, Feb 28, 2025 at 09:35:27PM +0100, Farblos wrote: > Hi Augustin, > > thanks for your quick fix, again. Only I have a couple of > minor and not-so minor nits with respect to its > implementation, namely the newly introduced variable > `ispell-load-nomessage':
Thanks, variable renamed to `debian-ispell-load-nomessage' and "emacsen-ispell-default.el" simplified. > 1. Personally and honestly, I don't think that a new variable > would be necessary here. As the doc string for function > `load' explains, its NOMESSAGE argument can always be > overridden by setting `force-load-message'. And I somewhat > doubt that a user really would like to have per-package > granularity for these (rather pesky) load messages. > > 2. Again rather personally, I don't think that it's necessary > to make this dependent on `noninteractive'. Just using > unconditional t for NOMESSAGE for all calls should be fine. > The main Debian hook in startup.el does the same, BTW: > > (if (load "debian-startup" t t nil) > (debian-startup debian-emacs-flavor)) I prefer to keep having this behavior to have a place where more info is available. This code is supposed to run under both Emacs and XEmacs and `force-load-message' seems not available for XEmacs. I also thought of early-init.el with debian-dict-common-debug enabled and `debian-ispell-load-nomessage' set as not it, but that is not always loaded and seems not available for XEmacs. > (I have tried signing up for Debian Salsa, hoping I could > provide feedback through the GitLab there, but I am not sure > whether me signing up is approved by the Salsa maintainers...) I personally prefer the BTS for bug related interactions, but that is a personal choice. Thanks again for your feedback. -- Agustin
