Bug 173444 - app-emacs/delicious-0.3 not compatible with all emacs versions
Bug#: 173444 Product:  Gentoo Linux Version: 2006.1 Platform: All
OS/Version: Linux Status: RESOLVED Severity: normal Priority: P2
Resolution: FIXED Assigned To: emacs@gentoo.org Reported By: hhalvors@princeton.edu
Component: Ebuilds
URL: 
Summary: app-emacs/delicious-0.3 not compatible with all emacs versions
Keywords:  
Status Whiteboard: 
Opened: 2007-04-05 10:15 0000
Description:   Opened: 2007-04-05 10:15 0000 
the delicious-0.3 ebuild contains a line:

local LOADPATH="/usr/share/emacs/22.0.50/lisp"

which causes a failure to compile for all versions of Emacs other than 22.0.50.
 But now emacs cvs is at version 22.0.60, so should this be changed?  Thanks.

Reproducible: Always

------- Comment #1 From Hans Halvorson 2007-04-05 10:17:17 0000 ------- 
(In reply to comment #0)
> the delicious-0.3 ebuild contains a line:
> 
> local LOADPATH="/usr/share/emacs/22.0.50/lisp"
> 
> which causes a failure to compile for all versions of Emacs other than 22.0.50.
>  But now emacs cvs is at version 22.0.60, so should this be changed?  Thanks.
> 
> Reproducible: Always
> 

oops, Emacs CVS is at version 22.0.96

------- Comment #2 From Ulrich Müller 2007-04-06 07:19:10 0000 ------- 
Using "elisp-comp" instead of "elisp-compile" should automatically take care of
the load-path. Replacing src_compile() in the ebuild by the following works for
me:

src_compile() {
    local FILES="delicioapi.el delicious.el"
    use planner && FILES="${FILES} planner-delicious.el"
    elisp-comp ${FILES} || die
}

Or, if you like one-liners: ;-)

src_compile() {
    elisp-comp delicio{api,us}.el $(use planner && echo planner-delicious.el)
}


(BTW, pkg_postinst and pkg_postrm are redundant, too, since they are the
standard actions of elisp.eclass.)

------- Comment #3 From Christian Faulhammer 2007-04-06 10:25:02 0000 ------- 
(In reply to comment #1)
> oops, Emacs CVS is at version 22.0.96

 No.  22.0.97

Thanks Ulrich, fixed.