Bug 184942 – games-action/slune doesn't install in /usr/games

• Status: RESOLVED UPSTREAM
• Product: Gentoo Linux

[Description Federico Ferri (RETIRED) 2007-07-11 10:44:57 UTC]

--- /usr/bin/
>>> /usr/bin/slune
>>> games-action/slune-1.0.11 merged.
>>> Recording games-action/slune in "world" favorites file...


it should install it in /usr/games/bin/slune instead (?)

also it doesn't provide a .desktop entry

[Comment 1 Mr. Bones. (RETIRED) 2007-07-11 16:22:51 UTC]

patches welcome.

[Comment 2 Federico Ferri (RETIRED) 2007-07-11 17:36:08 UTC]

Created attachment 124585 [details]
games-action/slune/slune-1.0.11.ebuild

here it is.
sorry, you have to diff it yourself cause I edited original ebuild in place :p

[Comment 3 Federico Ferri (RETIRED) 2007-07-11 21:29:11 UTC]

hmm ... forgot to add desktop entry


should add something like:

src_install() {
    make_desktop_entry ${PN} "Slune"
    prepgamesdirs
}

but I'm not sure which src_install is the default for python apps...sorry :S

[Comment 4 Mr. Bones. (RETIRED) 2007-10-15 02:24:34 UTC]

This patching only changes the location of the binary.  The data files should be moved as well for consistency.  locale files should be in /usr/share/locale

[Comment 5 Tomáš Chvátal (RETIRED) 2008-08-14 20:07:29 UTC]

There are two options for slune, put pyc objects into /usr/share/games wich will raise Warning or put it correctly and then i had to patch 3 files from when i start and it is still not working right... :(

This is what should be added to at least put slune into /usr/games directory structure:

inherit distutils games
...
src_unpack() {
    unpack ${A}
    cd "${S}"
    # fix data placement
    sed -i \
        -e "s:/usr/bin:${GAMES_BINDIR}:" \
        -e "s:/usr/share:${GAMES_DATADIR}:" \
        setup.cfg || die "patching setup.cfg failed"
}
this at least work but fixing that QA is something like rewriting most of the game...

[Comment 6 Mr. Bones. (RETIRED) 2008-09-30 06:14:26 UTC]

rewriting the game puts it fairly deep into upstream territory.  Marking UPSTREAM.  Hopefully, in a future release, the install will be more flexible.