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/vc-config.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/vc-config.el')
| -rw-r--r-- | modules/vc-config.el | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/modules/vc-config.el b/modules/vc-config.el index 7865d0f4..c76e714e 100644 --- a/modules/vc-config.el +++ b/modules/vc-config.el @@ -14,6 +14,9 @@ ;;; Code: +(require 'user-constants) ;; provides code-dir +(require 'keybindings) ;; provides cj/custom-keymap + ;; ---------------------------- Magit Configuration ---------------------------- (use-package magit |