diff options
| author | Craig Jennings <c@cjennings.net> | 2026-05-07 19:25:29 -0500 |
|---|---|---|
| committer | Craig Jennings <c@cjennings.net> | 2026-05-07 19:25:29 -0500 |
| commit | a41ef9774f6550da446a3ae8fbbcbcd5bf6c23c4 (patch) | |
| tree | 6afe628ac98e2e28c871ec5b78c91fddd2777252 /modules/custom-text-enclose.el | |
| parent | 93b033a313c5a7ad56293c192ef69b567d344178 (diff) | |
| download | dotemacs-a41ef9774f6550da446a3ae8fbbcbcd5bf6c23c4.tar.gz dotemacs-a41ef9774f6550da446a3ae8fbbcbcd5bf6c23c4.zip | |
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.
Diffstat (limited to 'modules/custom-text-enclose.el')
| -rw-r--r-- | modules/custom-text-enclose.el | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/modules/custom-text-enclose.el b/modules/custom-text-enclose.el index e93e6dea..ebfb44e9 100644 --- a/modules/custom-text-enclose.el +++ b/modules/custom-text-enclose.el @@ -21,8 +21,7 @@ ;;; Code: -;; cj/custom-keymap defined in keybindings.el -(eval-when-compile (defvar cj/custom-keymap)) +(require 'keybindings) ;; provides cj/custom-keymap (defun cj/--surround (text surround-string) "Internal implementation: Surround TEXT with SURROUND-STRING. |