Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 155682 - app-office/ledger: emacs dep should be virtual/emacs
Summary: app-office/ledger: emacs dep should be virtual/emacs
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High normal
Assignee: Gunnar Wrobel (RETIRED)
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-11-19 08:44 UTC by Matthew Kennedy (RETIRED)
Modified: 2006-11-20 01:29 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 Matthew Kennedy (RETIRED) gentoo-dev 2006-11-19 08:44:37 UTC
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 Gunnar Wrobel (RETIRED) gentoo-dev 2006-11-20 01:29:09 UTC
Hi Matthew, thanks for the corrections. I fixed the ebuild according to your suggestions :)