First Last Prev Next    No search results available      Search page      Enter new bug
Bug#: 155682
Alias:
Product:
Component:
Status: RESOLVED
Resolution: FIXED
Assigned To: Gunnar Wrobel <wrobel@gentoo.org>
Hardware:
OS:
Version:
Priority:
Severity:
Reporter: Matthew Kennedy (RETIRED) <mkennedy@gentoo.org>
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 155682 depends on: Show dependency tree
Show dependency graph
Bug 155682 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: 2006-11-19 08:44 0000
The emacs dependency should be "virtual/emacs" so that the two slots in
app-editors/emacs-cvs can be used :-)

Also, the build system will install into /usr/share/emacs/site-lisp/.  That
needs to be changed so that {ledger,timeclock}.el* install into
/usr/share/emacs/site-lisp/ledger.  50ledger-mode-gentoo.el should still go
into /usr/share/emacs/site-lisp/ though.

Calling elisp_src_install will result in the emacs .el/.elc being installed
twice.  If you can adjust the econf/einstall to honor
--lispdir=/usr/share/emacs/site-lisp/ledger, that's the best way.  I usually
don't call elisp_src_compile or elisp_src_install from an ebuild not in
app-emacs/.  Take a look at how dev-lang/gforth uses the eclass for an example.

And you need to make sure the new subdirectory,
/usr/share/emacs/site-lisp/ledger is added to the load-path.  So in your
50ledger-mode-gentoo.el, add this to the top: 

  (add-to-list 'load-path "@SITELISP@")

------- Comment #1 From Gunnar Wrobel 2006-11-20 01:29:09 0000 -------
Hi Matthew, thanks for the corrections. I fixed the ebuild according to your
suggestions :)

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