[Bug 226800] Review Request: emacs-bbdb - email database for Emacs

[bugzilla at redhat.com]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: emacs-bbdb - email database for Emacs


https://bugzilla.redhat.com/show_bug.cgi?id=226800





------- Additional Comments From tromey at redhat.com  2007-09-09 01:21 EST -------
Hi.  Sorry I've been ignoring this once again :(
I think it would be fine for Jonathan to be the primary packager.

I looked at the newest spec file and I read through the Emacs
packaging draft.  Looking good!

Going back to the bbdb-autoloads.el question -- I think the reason that
this is in site-start.d is so that the autoloads will be evaluated at
Emacs startup (see site-lisp/site-start.el).  This means that the user
doesn't have to add an explicit require or anything to their .emacs to
start using BBDB.  I was wrong about this back in comment #10.  (FWIW
this is mentioned in the packaging guidelines.)

In my copy of the latest RPM, bbdb-autoloads.el ends up in site-lisp/bbdb/.
So, I think it should be moved again.

BTW ... BBDB and VM requiring each other seems pretty ugly!  Perhaps one or
the other could be fixed upstream.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.