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 | 4a8c572ba6a64be997be072b44ef5ff62674d820 (patch) | |
| tree | 210e4e13238b4c73c4b62a3e4864827df64bec66 /docs/design | |
| parent | 55922a54652d74d047a41728e5ce7b2f86d9a3d7 (diff) | |
| download | dotemacs-4a8c572ba6a64be997be072b44ef5ff62674d820.tar.gz dotemacs-4a8c572ba6a64be997be072b44ef5ff62674d820.zip | |
fix: load games-config via the malyon hook, not an autoload chain
The previous deferral (03d8b587) 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')
| -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 3903d0ba9..fb883d701 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 | |