Bug 960295

Summary:	Can't compose email vm-user-agent with emacs 24
Product:	[Fedora] Fedora	Reporter:	mpurtill
Component:	emacs-vm	Assignee:	Göran Uddeborg <goeran>
Status:	CLOSED ERRATA	QA Contact:	Fedora Extras Quality Assurance <extras-qa>
Severity:	unspecified	Docs Contact:
Priority:	unspecified
Version:	17	CC:	goeran, jonathan.underwood
Target Milestone:	---
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:	emacs-vm-8.1.2-2.fc17	Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2013-05-28 00:58:02 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:

Description mpurtill 2013-05-06 22:44:12 UTC

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.

Comment 1 Göran Uddeborg 2013-05-14 10:02:50 UTC

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

Comment 2 Göran Uddeborg 2013-05-14 11:44:04 UTC

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

Comment 3 mpurtill 2013-05-14 16:59:18 UTC

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.

Comment 4 Göran Uddeborg 2013-05-14 20:53:26 UTC

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.

Comment 5 mpurtill 2013-05-14 21:31:04 UTC

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

Comment 6 Göran Uddeborg 2013-05-15 20:13:30 UTC

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

Comment 7 mpurtill 2013-05-17 18:26:32 UTC

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

Comment 8 Göran Uddeborg 2013-05-17 21:20:18 UTC

Thanks for testing!  It seems to work for me too.  I'll go ahead and update the package in all branches.

Comment 9 Fedora Update System 2013-05-17 21:32:01 UTC

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

Comment 10 Fedora Update System 2013-05-17 21:53:04 UTC

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

Comment 11 Fedora Update System 2013-05-19 02:34:59 UTC

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

Comment 12 Fedora Update System 2013-05-19 20:51:50 UTC

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

Comment 13 Fedora Update System 2013-05-27 00:32:08 UTC

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.

Comment 14 Fedora Update System 2013-05-28 00:58:02 UTC

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.

Comment 15 Fedora Update System 2013-05-29 02:59:42 UTC

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.