Bug 34595 - xemacs-21.1.14-2 seems to have broken vm key bindings
Summary: xemacs-21.1.14-2 seems to have broken vm key bindings
Alias: None
Product: Red Hat Linux
Classification: Retired
Component: xemacs
Version: 7.0
Hardware: i386
OS: Linux
Target Milestone: ---
Assignee: Jens Petersen
QA Contact: Aaron Brown
Depends On:
TreeView+ depends on / blocked
Reported: 2001-04-04 00:32 UTC by Ronald Cole
Modified: 2007-04-18 16:32 UTC (History)
1 user (show)

Clone Of:
Last Closed: 2002-09-23 20:07:20 UTC

Attachments (Terms of Use)

Description Ronald Cole 2001-04-04 00:32:53 UTC
From Bugzilla Helper:
User-Agent: Mozilla/4.76 [en] (X11; U; Linux 2.2.17-14smp i686)

Under xemacs-21.1.12-3, running vm (M-x vm) and composing an email (m), C-c
C-w runs the command "mail-signature".  This is as expected.  It results in
the ~/.signature file being appended at the end of the buffer with a "\n--
" separator.

Applying the errata xemacs-21.1.14-2, starting vm and composing an email,
C-c C-w runs the command "insert-signature", which is wrong.  It causes the
~/.signature file to be inserted at point instead of at the end of the
buffer and doesn't insert the "\n-- " separator.

Reproducible: Always
Steps to Reproduce:
1. xemacs
2. M-x vm
3. m
4. type some email in the body and move the point to the middle of the text
5. C-c C-w

Actual Results:  ~/.signature was inserted at point with no separator.

Expected Results:  ~/.signature should be appended to the end of the
buffer, separated by "\n-- ".

Comment 1 Trond Eivind Glomsrxd 2001-04-04 15:54:23 UTC
That sounds more like "changed" than "a bug". If you're not happy with the new
binding, I suggest you modify this in your local .emacs

Comment 2 Ronald Cole 2001-04-04 20:57:38 UTC
But why would RedHat fiddle with the XEmacs distribution?  And why would RedHat
not document such a change?

Comment 3 Trond Eivind Glomsrxd 2001-04-04 21:03:04 UTC
We didn't change this, we just included the newest lisp bundle which it seems
upgraded this subpackage, the new subpackaging having this new keybinding.

We don't document the changes at the package level unless we do it ourselves.

Comment 4 Ronald Cole 2001-04-04 22:28:11 UTC
No, the vm-6.75 in the errata does not have that "new keybinding".  Look at the
elisp yourself.
Perhaps another package in the bundle you've included is clobbering that key
binding, since your suggestion of setting it myself in my .emacs:

(require 'sendmail)
(define-key mail-mode-map "\C-c\C-w" 'mail-signature)

didn't work around your "feature".  I still consider this a bug.

Comment 5 Ronald Cole 2001-04-17 22:00:53 UTC
Ok, I found the problem.  You *did* fiddle with the XEmacs distribution by
introducing the package "Wanderlust".  I note that "Wanderlust" isn't listed in
the file /usr/share/xemacs-21.1.14/etc/package-index.LATEST.pgp.

The file /usr/lib/xemacs/xemacs-packages/lisp/site-start.d/wl-init.el contains
the line (load "mime-setup"), which it gets from the "tm" package. 
Unfortunately, "tm" and "vm" don't get along here.  Also, unfortunate is that
RedHat didn't heed the Wanderlust MIME module suggestions on their web site
<http://www.gohome.org/wl/>, where they say: "Before installing Wanderlust,
please install either of the following MIME modules. SEMI is recommended because
it's more functional."

I haven't tested installing SEMI, but removing wl-init.el from site-start.d
solves the problem.  I suggest removing Wanderlust from the .rpm until such time
as it's listed as an offical XEmacs package in package-index.LATEST.pgp.

Comment 6 Trond Eivind Glomsrxd 2001-04-17 22:13:05 UTC
Semi is included in the package as well. Nakai, you added wl - please fix it not
to disrupt non-Japanese packages.

Comment 7 Ronald Cole 2001-04-17 22:39:20 UTC
No, semi is not included:

$ ls -l xemacs*
-rw-r--r--    2 root     root     20247307 Feb  7 13:28
-rw-r--r--    2 root     root      8629561 Feb  7 13:29
-rw-r--r--    2 root     root      1526587 Feb  7 13:31
$ for i in xemacs*; do rpm -qpl $i | grep -i semi; done

Comment 8 Trond Eivind Glomsrxd 2001-04-18 17:56:38 UTC
Sorry for that misunderstanding - it's part of the xemacs packages for 7.1, not
the errata packages. It was merged after the errata release

Comment 9 Ronald Cole 2001-05-10 23:26:12 UTC
Ok, I've tried 7.1.  Yes, it includes SEMI, but semi-init.el also loads
mime-setup from the tm package.  So removing both semi-init.el and wl-init.el
fixed the problem in 7.1 for me.

Comment 10 Jens Petersen 2002-11-11 13:11:00 UTC
semi is now included in the separate wl package.

Note You need to log in before you can comment on or make changes to this bug.