diff options
| author | Craig Jennings <c@cjennings.net> | 2026-06-21 02:16:42 -0400 |
|---|---|---|
| committer | Craig Jennings <c@cjennings.net> | 2026-06-21 02:16:42 -0400 |
| commit | a8571effadacd6f94291bfd57b7e58900a2e5816 (patch) | |
| tree | b0b91f29cb50ece1893379562803d235ba5895cb /docs/design/module-inventory.org | |
| parent | 8b2bc5187dd85178051139cfbf9dae29d1748d2b (diff) | |
| download | dotemacs-a8571effadacd6f94291bfd57b7e58900a2e5816.tar.gz dotemacs-a8571effadacd6f94291bfd57b7e58900a2e5816.zip | |
fix: load games-config via the malyon hook, not an autoload chain
The previous deferral (8b2bc518) autoloaded malyon to games-config, but games-config doesn't define malyon. It leaves the command to the malyon package, so M-x malyon loaded games-config, found malyon still undefined, and errored "Autoloading games-config.el failed to define function malyon". Emacs won't chain through a second autoload.
malyon and 2048-game autoload their own commands via package.el, so games-config should never own them. init.el now loads games-config via (with-eval-after-load 'malyon ...), and games-config just sets malyon-stories-directory when malyon loads. M-x malyon loads the package as a real command, then games-config applies its config.
The earlier batch check loaded the files by hand and missed the autoload failure. The new test resolves the autoload the way M-x does (autoload-do-load), so the real path is covered now.
Diffstat (limited to 'docs/design/module-inventory.org')
| -rw-r--r-- | docs/design/module-inventory.org | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/docs/design/module-inventory.org b/docs/design/module-inventory.org index 3903d0ba..fb883d70 100644 --- a/docs/design/module-inventory.org +++ b/docs/design/module-inventory.org @@ -205,7 +205,7 @@ flyspell-and-abbrev is the one Core-UX member (text-mode hooks). | =eshell-config= | 3 | D/P | eager | command | system-utils | add-hook, advice-add, package config | yes | | =eww-config= | 3 | D/P | eager | command | cl-lib | package config | yes | | =flyspell-and-abbrev= | 2 | C/P | eager | hook | cl-lib | mode-hook package config | yes | -| =games-config= | 4 | O | command | command | none | package config | yes | +| =games-config= | 4 | O | command | command | user-constants | package config | yes | | =gloss-config= | 4 | O/D/P | eager | command | none | package config | yes | | =httpd-config= | 4 | O/D/P | eager | command | none | package config | yes | | =jumper= | 4 | O/L | eager | command | cl-lib | jumper keymap | yes | |