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

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


Attachments
Proposed changes to chess-2.0_beta5-r1.ebuild (diff,760 bytes, patch)
2007-02-27 11:20 UTC, Ulrich Müller
Details | Diff
Site-init file 51chess-gentoo.el (51chess-gentoo.el,598 bytes, text/plain)
2007-02-27 11:21 UTC, Ulrich Müller
Details

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-27 11:19:36 UTC
emerge app-emacs/chess-2.0_beta5-r1 installs chess's pixmaps and sounds in subdirectories of /usr/share/emacs/site-lisp. These directories are unnecessarily added by Emacs to its load-path.

Proposed fix: Pixmaps should go into /usr/share/pixmaps/chess, sounds into /usr/share/sounds/chess.

Another issue is that the site-init file does an unconditional "(require 'chess)" where an "autoload" would be sufficient. See commentary in chess.el:

;; To just get a chessboard up, put the following in your .emacs file:
;;   (add-to-list 'load-path "<the path to Emacs Chess>")
;;   (autoload 'chess "chess" "Play a game of chess" t)

Find attached my proposed changes to the ebuild and a new site-init file. It applies (and I have tested it) with chess-2.0_beta5, too, in case you want to maintain a separate stable branch.
Comment 1 Ulrich Müller gentoo-dev 2007-02-27 11:20:20 UTC
Created attachment 111397 [details, diff]
Proposed changes to chess-2.0_beta5-r1.ebuild
Comment 2 Ulrich Müller gentoo-dev 2007-02-27 11:21:14 UTC
Created attachment 111398 [details]
Site-init file 51chess-gentoo.el
Comment 3 Christian Faulhammer (RETIRED) gentoo-dev 2007-02-27 12:39:24 UTC
(In reply to comment #0)
> Find attached my proposed changes to the ebuild and a new site-init file. It
> applies (and I have tested it) with chess-2.0_beta5, too, in case you want to
> maintain a separate stable branch.

 Thanks.  Fixed in CVS.  I won't do the changes for the stable branch, it was broken for a long time and I wait until Emacs 22 goes stable.