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 | 0805df5e6ac5aa4972600bbe3c01ec3d3c166592 (patch) | |
| tree | cef9429858f7bed10227ac350238161cebe18525 /docs/design | |
| parent | cfcb7cf0658a7d6b368e26e07d5ad5861af4ebf0 (diff) | |
| download | dotemacs-0805df5e6ac5aa4972600bbe3c01ec3d3c166592.tar.gz dotemacs-0805df5e6ac5aa4972600bbe3c01ec3d3c166592.zip | |
fix: load games-config via the malyon hook, not an autoload chain
The previous deferral (cfcb7cf0) 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 | |