Cannot customize mumamo variables in Emacs 22.2.1

Bug #599726 reported by Dominique Quatravaux
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
nXhtml
Fix Committed
Medium
Unassigned

Bug Description

From a fresh loaded Emacs, typing M-x customize-variable <return> mumamo-major-modes , then clicking on "show value", results in a stack trace because module `ourcomments-widgets', which is necessary to interpret that variable's customization type, is not loaded. Adding the following to my .emacs after the loading of nxhtml/autostart.el fixes the issue:

  (require 'ourcomments-widgets)

Using Emacs 22.2.1 (Ubuntu Jaunty's emacs 22.2-0ubuntu2 package), nxhtml 2.08. Issue can be reproduced under M-x emacs-Q-nxhtml just as well (kudos for providing this debugging feature!).

Revision history for this message
lborgman (lennart-borgman) wrote :

Thanks for the bug report, Dominique.

I can not test this at the moment, my current checkout of Emacs seems a bit broken. I think this should work without an explicit require of ourcomments-widget since the functions involved should be autoloaded.

What version of nXhtml are you using? Could you test with a later version of Emacs if that is possible for you?

Changed in nxhtml:
importance: Undecided → Medium
status: New → In Progress
Revision history for this message
Dominique Quatravaux (dominique-quatravaux) wrote :

> I think this should work without an explicit require of ourcomments-widget since the functions involved should be autoloaded.

Widgets cannot be autoloaded. As you can see in Emacs' widget.el, the implementation of define-widget is to 'put stuff in the 'widget-type and 'widget-documentation entries of the symbol's plist. There is no way to make that happen on-demand; the only autoload feature that exists at all in Emacs-Lisp deals with the function cell of symbols.

> What version of nXhtml are you using?

nxhtml 2.08

> Could you test with a later version of Emacs if that is possible for you?

I just tested for Emacs 23 from Ubuntu Lucid, and I can reproduce the bug.

Revision history for this message
lborgman (lennart-borgman) wrote :

Hum, hum ... seems that you are right. I somehow hoped that define-widget should create a function too, but it does not.

Thanks for this. I have rearranged things a bit so that the things needed are loaded when autostart.el is loaded. This is now in the repository.

Changed in nxhtml:
status: In Progress → Fix Committed
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.