Cannot customize mumamo variables in Emacs 22.2.1

Bug #599726 reported by Dominique Quatravaux on 2010-06-29
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
nXhtml
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!).

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

> 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.

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  Edit
Everyone can see this information.

Other bug subscribers