From a41ef9774f6550da446a3ae8fbbcbcd5bf6c23c4 Mon Sep 17 00:00:00 2001 From: Craig Jennings Date: Thu, 7 May 2026 19:25:29 -0500 Subject: chore(modules): pass validate-modules in batch by adding requires `make validate-modules` had 19 module-load failures, all the same shape: a module references a symbol or feature owned by another module without saying so. Production was fine because init.el orders requires correctly. The batch target loads each module in isolation, though, and surfaces the gap. I added explicit `(require 'keybindings)` or `(require 'user-constants)` to each affected module. The requires are idempotent at runtime, so production load order is unchanged. For three optional packages (elpa-mirror, mu4e, org-contacts), I switched to `(require 'X nil t)` so the modules load cleanly when those packages aren't installed. The activation calls become no-ops in that case. `make validate-modules` now reports 0 failures. --- modules/custom-case.el | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'modules/custom-case.el') diff --git a/modules/custom-case.el b/modules/custom-case.el index a4e0c0e0..3f7ebc4d 100644 --- a/modules/custom-case.el +++ b/modules/custom-case.el @@ -16,7 +16,7 @@ ;; ;;; Code: -(eval-when-compile (defvar cj/custom-keymap)) ;; cj/custom-keymap defined in keybindings.el +(require 'keybindings) ;; provides cj/custom-keymap (defun cj/upcase-dwim () "Upcase the active region, or upcase the symbol at point if no region." -- cgit v1.2.3