Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 168212 - app-emacs/vm-7.19-r1 installs pixmaps in site-lisp directory
Summary: app-emacs/vm-7.19-r1 installs pixmaps in site-lisp directory
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: High normal (vote)
Assignee: Emacs project
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-02-24 08:39 UTC by Ulrich Müller
Modified: 2007-02-26 12:02 UTC (History)
0 users

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

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Ulrich Müller gentoo-dev 2007-02-24 08:39:11 UTC 
emerge app-emacs/vm-7.19-r1 creates the directory /usr/share/emacs/site-lisp/etc/vm and installs vm's pixmaps there. These directories (.../etc and .../etc/vm) are unnecessarity added by Emacs to its load-path.

Proposed fix: Pixmaps should go to /usr/share/pixmaps/vm (or maybe to /usr/share/emacs/pixmaps/vm).

(Variables vm-toolbar-pixmap-directory and vm-image-directory in 50vm-gentoo.el must be adapted, too.)
Comment 1 Christian Faulhammer (RETIRED) gentoo-dev 2007-02-26 08:14:19 UTC 
Fixed in -r2, I chose /usr/share/pixmaps/vm/.
Thanks.
Comment 2 Ulrich Müller gentoo-dev 2007-02-26 11:49:26 UTC 
Sorry, but I have to reopen this. The -r2.ebuild sets SITEFILE=51vm-gentoo.el,
but then does not use the variable in src_install():
    elisp-site-file-install "${FILESDIR}/50vm-gentoo.el"

It should be "${FILESDIR}/${SITEFILE}" instead.
Comment 3 Christian Faulhammer (RETIRED) gentoo-dev 2007-02-26 12:02:18 UTC 
Fixed...normally it should.  I haven't spotted it.