Bug 835318 - bbdb fails to add new e-mail addresses because emacs-24 changes run-hook-with-args return value
Summary: bbdb fails to add new e-mail addresses because emacs-24 changes run-hook-with...
Alias: None
Product: Fedora
Classification: Fedora
Component: emacs-bbdb
Version: 17
Hardware: All
OS: Linux
Target Milestone: ---
Assignee: Göran Uddeborg
QA Contact: Fedora Extras Quality Assurance
Depends On:
TreeView+ depends on / blocked
Reported: 2012-06-26 00:40 UTC by John Heidemann
Modified: 2012-07-19 09:18 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Last Closed: 2012-07-19 09:18:49 UTC

Attachments (Terms of Use)
patch to fix run-hook-with-args API change in emacs-24 triggered by bbdb (657 bytes, patch)
2012-06-26 00:43 UTC, John Heidemann
no flags Details | Diff

Description John Heidemann 2012-06-26 00:40:46 UTC
Description of problem:
(defun bbdb-canonicalize-address (net)


but as of 24.0.50 that no longer returns a value:


as a result, bbdb no longer records new e-mail addresses :-(

Version-Release number of selected component (if applicable):

How reproducible:

Steps to Reproduce:
1.Run emacs and browse a new message (a message that bbdb has never seen)
in your mail reader that uses bbdb.
(For me, mh-e.)
2. Execute something that should discover the e-mail address in the current message,
like M-x bbdb/mh-pop-up-bbdb-buffer
3. be sad when the buffer shows a name, but no bbdb-snarf'ed e-mail address
Actual results:
new bbdb record created with a name, but no net: field

Expected results:
a new bbdb record with both a name and an e-mail address grabbed from the message

Additional info:

discusses the problem and identifies emacs-24's helpfully documented change in the run-hook-with-args command

I have a fix that I will post in a followup.

Comment 1 John Heidemann 2012-06-26 00:43:25 UTC
Created attachment 594317 [details]
patch to fix run-hook-with-args API change in emacs-24 triggered by bbdb

Comment 2 Göran Uddeborg 2012-07-04 21:51:47 UTC
Thanks for your report.  I'll try to take care of this; I am a new co-maintainer of the emacs-bbdb package.

Since you have investigated the issue, have you also checked if the upstreams are aware already?  Or should we point it out them (him)?

Comment 3 John Heidemann 2012-07-05 00:22:09 UTC
I attempted to post the bug to the bbdb-info mailing list, but my post was held for moderator approval (even after I joined the list).  If you know a better way to upstream the bug, please advise.

Comment 4 Göran Uddeborg 2012-07-05 15:37:13 UTC
I have no special contacts with upstreams, but I did manage to send a message to the bbdb-info mailing list after subscribing.  I don't know why it worked for me but not for you, but there is a short report there now.

Your patch didn't quite apply, so I had to adjust it somewhat.  I hope I got it right, but please do test the update I'll push shortly.  I don't normally use bbdb, so the testing I can do myself is a bit rudimentary, to put it mildly.

Comment 5 Fedora Update System 2012-07-05 15:57:08 UTC
emacs-bbdb-2.35-8.fc17 has been submitted as an update for Fedora 17.

Comment 6 John Heidemann 2012-07-05 19:55:22 UTC
Thanks, can you post the updated patch (mentioned in comment #4) so I can try it in parallel with the new package coming down the line?

Comment 7 Göran Uddeborg 2012-07-05 20:20:58 UTC
I guess the best testing is if you pick up the packages directly via https://admin.fedoraproject.org/updates/emacs-bbdb-2.35-8.fc17

Comment 8 John Heidemann 2012-07-05 21:45:40 UTC
Yes, I went there. It's Much Less Than Obvious which of the several links gets me to the patch, or even the source or binary of the package itself.  (I think I've now found the src.rpm, but whew...a maze of twistly URLs, all alike.)

Comment 9 Fedora Update System 2012-07-05 23:38:22 UTC
Package emacs-bbdb-2.35-8.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-bbdb-2.35-8.fc17'
as soon as you are able to.
Please go to the following url:
then log in and leave karma (feedback).

Comment 10 Göran Uddeborg 2012-07-07 07:50:41 UTC
> It's Much Less Than Obvious which of the several links gets me to the patch

Um, right.  I could have put in more direct URL:s I guess.  I'll try to think about that next time.  (This package is available in testing now, so for this time it doesn't matter any more.)

Comment 11 Fedora Update System 2012-07-19 09:18:49 UTC
emacs-bbdb-2.35-8.fc17 has been pushed to the Fedora 17 stable repository.  If problems still persist, please make note of it in this bug report.

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