GNU bug report logs - #7769
Void variable nxml-mode-map when running nXhtml tests

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 7769 in the body.
You can then email your comments to 7769 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Lennart Borgman <lennart.borgman <at> gmail.com>
To: Emacs Bugs <bug-gnu-emacs <at> gnu.org>
Subject: Void variable nxml-mode-map when running nXhtml tests
Date: Sun, 2 Jan 2011 10:36:55 +0100

I found a problem with nxml-mode.el when testing nXhtml. If nXhtml is
not byte compiled it works fine, but when it is byte compiled some
tests fails with "void variable nxml-mode-map".

With an Emacs checkout from 2010-11-23 all tests pass, but with a
checkout from 2010-12-15 some tests fail as above.

I think the problem is caused by a small part of this change:

  2010-12-03  Glenn Morris  <rgm <at> gnu.org>

	* nxml/nxml-mode.el: Require rng-nxml.
	(rng-nxml-mode-init, nxml-enable-unicode-char-name-sets):
	Remove declarations.

This change moved the (provide 'nxml-mode) statement to the beginning
of nxml-mode.el. Since eval-after-load is run at the provide the
statement

(eval-after-load 'nxml-mode
  '(progn
     (define-key nxml-mode-map [C-M-left]  'nxml-backward-element)
     (define-key nxml-mode-map [C-M-right] 'nxml-forward-element)
     (define-key nxml-mode-map [C-M-up]    'nxml-backward-up-element)
     (define-key nxml-mode-map [C-M-down]  'nxml-down-element)))

in the nXhtml file nxhtml-mode.el will now be run before nxml-mode-map
is defined.

Message #10 received at 7769-done <at> debbugs.gnu.org (full text, mbox):

From: Glenn Morris <rgm <at> gnu.org>
To: 7769-done <at> debbugs.gnu.org
Subject: Re: Void variable nxml-mode-map when running nXhtml tests
Date: Fri, 7 Jan 2011 19:08:06 -0500

This kind of thing is explained in the doc-string of eval-after-load.
Use a string instead of a symbol for the FILE argument.

Message #13 received at 7769 <at> debbugs.gnu.org (full text, mbox):

From: Stefan Monnier <monnier <at> iro.umontreal.ca>
To: 7769 <at> debbugs.gnu.org
Cc: rgm <at> gnu.org
Subject: Re: bug#7769: Void variable nxml-mode-map when running nXhtml tests
Date: Mon, 10 Jan 2011 22:24:19 -0500

> This kind of thing is explained in the doc-string of eval-after-load.
> Use a string instead of a symbol for the FILE argument.

BTW, I've just installed a change in the trunk to fix this problem
(i.e. so that eval-after-load evals *after-load* even for features that
are provided at the beginning (or in the middle) of the file).


        Stefan

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.