Bug report + patch: add a few missing ;;;###autoload cookies

Mike FABIAN mfabian at suse.de
Mon Jul 3 11:18:20 EDT 2006


The custom-defines.elc files

/usr/share/xemacs/xemacs-packages/lisp/build/custom-defines.elc
/usr/share/xemacs/xemacs-packages/lisp/ps-print/custom-defines.elc

use some variables and functions which don't have autoload cookies.

This does still work if all the .el files are installed because if an
error happens while loading a custom-defines.el, XEmacs will try to
read the .el file again whare the defcustom or defgroup came from (see
the doc-string of the function
'custom-warn-when-reloading-necessary').

But as this doc-string says, one should not rely on that mechanism but
rather use appropriate autoload cookies.

Relying on that automatic mechanism also requires to have the .el
files installed (the .elc files are not enough). But most of the .el
files should not be required at runtime, not all users install
them. Those who don't install them will get an error message
when clicking on the menu:

    option -> advanced -> emacs

and leave the menu again. The lisp backtrace starts with:

    (1) (event/warning) Error during menu callback: (void-function (build-cvs-checkout-options-validate))
    Backtrace follows:

Adding the autoload cookies as in the attached patch fixes
the problem, even if the .el files are not installed clicking
on the menu 

    option -> advanced -> emacs

will work then.

For more details about this bug please see:

    http://bugzilla.novell.com/show_bug.cgi?id=183805

-------------- next part --------------
A non-text attachment was scrubbed...
Name: bugzilla-183805-missing-autoload-cookies.patch
Type: text/x-patch
Size: 1319 bytes
Desc: not available
Url : http://lists.xemacs.org/pipermail/xemacs-beta/attachments/20060703/ad143365/bugzilla-183805-missing-autoload-cookies.bin
-------------- next part --------------

-- 
Mike FABIAN   <mfabian at suse.de>   http://www.suse.de/~mfabian
?????????????


More information about the XEmacs-Beta mailing list