diff options
| author | Craig Jennings <c@cjennings.net> | 2026-05-15 02:31:43 -0500 |
|---|---|---|
| committer | Craig Jennings <c@cjennings.net> | 2026-05-15 02:31:43 -0500 |
| commit | 05f4c25130cdaaa16bc14bc1c36866b3e187ad6d (patch) | |
| tree | bead3e23781cbe5077e170db32658a5b955f9e71 /modules/org-webclipper.el | |
| parent | e4a957753d243aec03c304de3291476b3a226633 (diff) | |
| download | dotemacs-05f4c25130cdaaa16bc14bc1c36866b3e187ad6d.tar.gz dotemacs-05f4c25130cdaaa16bc14bc1c36866b3e187ad6d.zip | |
fix(system-commands): require keybindings at load time, not just compile time
The module had `(eval-when-compile (require 'keybindings))`, which
silences the byte-compiler but doesn't make `cj/custom-keymap'
available when the module is required. The top-level
`(keymap-set cj/custom-keymap "!" cj/system-command-map)' at the tail
of the file then fails with `void-variable cj/custom-keymap'.
Normal Emacs startup happened to work because `init.el' requires
`keybindings' before `system-commands'. But requiring the module in
isolation -- including from `make test-file
FILE=test-system-commands-resolve-and-run.el' -- blows up.
Fix: use a plain `(require 'keybindings)' so the load-time
dependency matches the load-time reference.
Diffstat (limited to 'modules/org-webclipper.el')
0 files changed, 0 insertions, 0 deletions
