pr fix autoload generation recursion error

[matsl]

What

Do not load and generate autoloads from hload-path while in batch
processing (such as byte compiling).

• Do not load and generate autoloads from hload-path in batch
• Remove compile dependency on loading autoloads

Why

When generating autoloads in Emacs 31 we get this error below. It does
not occur with older versions.

Eager macro-expansion failure

❯ make clean autoloads
  GEN      kotl/kotl-autoloads.el
  INFO     Scraping 66 files for loaddefs...
hibtypes:0: Warning: Not registering prefix "hy".  Affects: ("hyrolo-stuck-msg" "hyp-manual" "hyp-address" "hyp-source" "hywiki-word" "hywiki-existing-word")
loaddefs-gen: loading file hsys-youtube (for defact)
Loading /home/matsl/src/hyperbole/hsys-youtube.el (source)...
Warning (emacs): loaddefs-gen: load error
	(error "Eager macro-expansion failure: (error \"Eager macro-expansion failure: (error \\\"Recursive load\\\" \\\"/home/matsl/share/emacs/31.0.50/lisp/jka-compr.el.gz\\\" \\\"/home/matsl/share/emacs/31.0.50/lisp/jka-compr.el.gz\\\" \\\"/home/matsl/share/emacs/31.0.50/lisp/jka-compr.el.gz\\\" \\\"/home/matsl/share/emacs/31.0.50/lisp/jka-compr.el.gz\\\" \\\"/home/matsl/share/emacs/31.0.50/lisp/jka-compr.el.gz\\\" \\\"/home/matsl/share/emacs/31.0.50/lisp/emacs-lisp/pcase.el.gz\\\" \\\"/home/matsl/src/hyperbole/set.el\\\" \\\"/home/matsl/src/hyperbole/hact.el\\\" \\\"/home/matsl/src/hyperbole/hsys-youtube.el\\\" \\\"/home/matsl/src/hyperbole/hload-path.el\\\")\")")
  INFO     Scraping 66 files for loaddefs...done
  GEN      hyperbole-autoloads.el
  INFO     Scraping 13 files for loaddefs...
  INFO     Scraping 13 files for loaddefs...done
  GEN      kotl-autoloads.el
Initializing Hyperbole...
Initializing Hyperbole...done
  INFO     Scraping 14 files for loaddefs...
  INFO     Scraping 14 files for loaddefs...done

The recursive load seems to origin from hload-path.el having a call to
check if autoloads are generated and if not generate and load them.

This PR tries to solve this by limiting the generation and loading of
autoloads to an interactive Emacs.

Further, the byte compile errors that occurs due to not loading
autoloads are resolved by adding declare-function and defvar. Each
el-file is by that possible to compile without having to load the
autoloads.

Note

I though this was a good experiment for solving the loaddefs-gen issue
by decoupling byte compilation from autoloads generation.

It also brought my attention to that we generate autoloads from
hload-path which feels a bit odd. I mean for normal use the autoloads
are generated by the package manager or, when running from source, by our
make target. Anyway this PR does not change that. An interactive Emacs
behaves the same.

[rswgnu]

The hload-path autoload generation is from long ago as is that whole setup when Emacs could not find its own executable directory.Are you saying that the PR has all the defs needed to work if there are no autoloads?  If so, we should be able to go with this.-- BobOn May 13, 2026, at 5:54 PM, Mats Lidell ***@***.***> wrote: @matsl requested your review on: rswgnu/hyperbole#953 pr fix autoload generation recursion error. —Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because your review was requested.Message ID: ***@***.***>