Re: [PATCH] 2 modules, zsh/db, zsh/gdbm, bug-fixes

[Bart Schaefer <schaefer@xxxxxxxxxxxxxxxx>]

On Jun 22, 11:05am, Sebastian Gniazdowski wrote:
}
} PS. moddeps in zsh/db/gdbm (.mdd file) seems to not work right.

Hmm.   I had gdbm marked as "static" in config.modules, and after
applying your patch:

srcdir='../../zsh-5.0/Src' CFMOD='../config.modules' \
  /bin/sh ../../zsh-5.0/Src/mkbltnmlst.sh bltinmods.list
ERROR: linked-in module `zsh/db/gdbm' depends on `zsh/db'
make[2]: *** [bltinmods.list] Error 1

So I made them both static and then recompiled, but there is still a
problem; if I don't explicitly load zsh/db first, loading zsh/db/gdbm
crashes.

If I explicitly do "zmodload -d zsh/db/gdbm zsh/db" then the crash is
also avoided.  It appears that mkbltnmlst.sh only creates add_dep()
calls for modules that are loaded into the shell by default (load=yes
in config.modules).  This only matters if there is a boot_ dependency
rather than merely a builtins dependency, so e.g. it doesn't break for
zsh/zftp and zsh/net/tcp.