First Last Prev Next    No search results available      Search page      Enter new bug
Bug#: 173444
Alias:
Product:
Component:
Status: RESOLVED
Resolution: FIXED
Assigned To: GNU Emacs Team <emacs@gentoo.org>
Hardware:
OS:
Version:
Priority:
Severity:
Reporter: Hans Halvorson <hhalvors@princeton.edu>
Add CC:
CC:
URL:
Summary:
Status Whiteboard:
Keywords:

Filename Description Type Creator Created Size Actions
Create a New Attachment (proposed patch, testcase, etc.) View All

Bug 173444 depends on: Show dependency tree
Show dependency graph
Bug 173444 blocks:
Votes: 0    Show votes for this bug    Vote for this bug

Additional Comments: (this is where you put emerge --info)







View Bug Activity   |   Format For Printing   |   XML   |   Clone This Bug


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.

First Last Prev Next    No search results available      Search page      Enter new bug