Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 173444 - app-emacs/delicious-0.3 not compatible with all emacs versions
Summary: app-emacs/delicious-0.3 not compatible with all emacs versions
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High normal
Assignee: Emacs project
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-04-05 10:15 UTC by Hans Halvorson
Modified: 2007-04-06 10:25 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Hans Halvorson 2007-04-05 10:15:54 UTC
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 Hans Halvorson 2007-04-05 10:17:17 UTC
(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 Ulrich Müller gentoo-dev 2007-04-06 07:19:10 UTC
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 Christian Faulhammer (RETIRED) gentoo-dev 2007-04-06 10:25:02 UTC
(In reply to comment #1)
> oops, Emacs CVS is at version 22.0.96

 No.  22.0.97

Thanks Ulrich, fixed.