Description of problem: Doing C-x m, when I have user agent set to vm-user-agent, crashes Version-Release number of selected component (if applicable): emacs.x86_64 1:24.1-7.fc17 emacs-vm.x86_64 0:8.1.2-1.fc17 How reproducible: Every time Steps to Reproduce: 1. emacs -q 2. M-x load-library vm 3. M-: (setq mail-user-agent 'vm-user-agent) 4. C-x m Actual results: Crash as above. With debugging enabled, the traceback is: Debugger entered--Lisp error: (wrong-number-of-arguments #[(&optional to subject other-headers continue switch-function yank-action send-actions) "<gibberish omitted>" [continue to subject buffer mail-header-separator other-headers vm-session-initialization vm-continue-composing-message vm-mail-internal format "message to %s" vm-truncate-roman-string 20 re-search-forward "^" "$" beginning-of-line 32 delete-char -1 58 ": " 10 " " mail-position-on-field "To" "Subject" mail-text switch-to-buffer apply push-mark run-hooks mail-citation-hook mail-yank-hooks vm-mail-yank-default make-local-variable mail-send-actions switch-function yank-action send-actions] 7 nil nil] 8) vm-compose-mail(nil nil nil nil nil nil nil nil) compose-mail(nil nil nil nil) call-interactively(compose-mail nil nil) Expected results: VM compose mail buffer. Additional info: According to the bug <https://bugs.launchpad.net/vm/+bug/705393>, vm needs version 8.2.0b or later when used with emacs 24.
I see the problem. I also see reasons not to change this. At least not in this stage of F17's life cycle. 8.2 is not completely compatible with 8.1, and I don't want to break anyone's usage in an update. While this particular feature doesn't work, a lot of VM's functionality still is usable. One could argue we should upgrade to 8.2 in rawhide and possibly F19. I've not yet done that, since 8.2.0b is still officially a beta release. I have been waiting for the final release. Admittedly, it seems to take quite a long time for it to appear. Hm...
To know what to expect, I sent a question to the vm mailing list: http://lists.nongnu.org/archive/html/viewmail-info/2013-05/msg00000.html
FYI, I was able to fix the problem by just adding an additional argument (ignore-variable) to vm-compose-mail, like this (in /usr/share/emacs/site-lisp/vm/vm.el): (defun vm-compose-mail (&optional to subject other-headers continue switch-function yank-action send-actions ignore-variable) After byte-compiling that, I was able to send email with vm. I haven't found any other problems.
Oh, it's that simple a patch! (I never looked at what Uday had done.) Let me also try it out a bit, and if it works for me too, I'll build an update.
(In reply to comment #4) > Oh, it's that simple a patch! (I never looked at what Uday had done.) Just to clarify, I'm not sure what the actual fix in the next vm version was either. This was just my quick hack so I could send email.
I see. I took a look at what Uday had actually done. He had added a bit more code, actually doing something with the new argument. But it was still a very small patch. And with a few lines of offset, it applied to 8.1.2 too. So I'm trying to use that instead. It seems to work fine initially. If you wish to check your test cases too, my local build is available at ftp://ftp.uddeborg.se/pub/bz960295/emacs-vm-8.1.2-2.fc17.x86_64.rpm
(In reply to comment #6) > If you wish to check your test cases too, my local build is > available at > ftp://ftp.uddeborg.se/pub/bz960295/emacs-vm-8.1.2-2.fc17.x86_64.rpm The vm in that rpm seems to work for me, thanks.
Thanks for testing! It seems to work for me too. I'll go ahead and update the package in all branches.
emacs-vm-8.1.2-2.fc17 has been submitted as an update for Fedora 17. https://admin.fedoraproject.org/updates/emacs-vm-8.1.2-2.fc17
emacs-vm-8.1.2-4.fc18 has been submitted as an update for Fedora 18. https://admin.fedoraproject.org/updates/emacs-vm-8.1.2-4.fc18
Package emacs-vm-8.1.2-2.fc17: * should fix your issue, * was pushed to the Fedora 17 testing repository, * should be available at your local mirror within two days. Update it with: # su -c 'yum update --enablerepo=updates-testing emacs-vm-8.1.2-2.fc17' as soon as you are able to. Please go to the following url: https://admin.fedoraproject.org/updates/FEDORA-2013-8550/emacs-vm-8.1.2-2.fc17 then log in and leave karma (feedback).
emacs-vm-8.1.2-6.fc19 has been submitted as an update for Fedora 19. https://admin.fedoraproject.org/updates/emacs-vm-8.1.2-6.fc19
emacs-vm-8.1.2-4.fc18 has been pushed to the Fedora 18 stable repository. If problems still persist, please make note of it in this bug report.
emacs-vm-8.1.2-2.fc17 has been pushed to the Fedora 17 stable repository. If problems still persist, please make note of it in this bug report.
emacs-vm-8.1.2-6.fc19 has been pushed to the Fedora 19 stable repository. If problems still persist, please make note of it in this bug report.