Steve Youngs <youngs(a)xemacs.org> wrote:
I just built XEmacs with your new module code (nice work, BTW) and
noticed a couple of problems:
- Autoloads are built for *everything* in ./modules
unconditionally. If I don't have/build/install ldap, I
don't want the ldap autoloads in
./modules/auto-autoloads.el. We should only build autoloads
for those modules that we are actually going to build and
install.
Yes, I've been aware of this for awhile. I've been pondering how to fix
it properly. I'm wondering if maybe the autoload forms for a given
module should be wrapped in something like
(when (file-exists-p "module.ell")
(autoload ...)
...
)
That way the autoloads depend only on the sources, not on a particular
configuration. On the other hand, this doesn't support installing new
modules later on that did not exist when the XEmacs binary was compiled.
On the gripping hand, that would still be a problem if auto-autoloads.el
contained only autoloads for modules installed at build time.
- With '--error-checking=all' I got an undefined
symbol error
while trying to connect to a PostgreSQL database...
"undefined symbol: error_check_pgconn"
Urk! That apparently means that ERROR_CHECK_TYPES was not defined
during compilation of the module code, but was for the main sources (see
lrecord.h). But how is that possible? That symbol is defined in
config.h, which is included first thing by postgresql.c! I don't get
it...
Can you take care of these, Jerry, or would you like me to?
Probably
be quicker if you do. :-)
Only if I understand the problem and have an actual solution. :-)
--
Jerry James
http://www.ittc.ku.edu/~james/