Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 493954 - games.eclass should set --datarootdir=/usr/share
Summary: games.eclass should set --datarootdir=/usr/share
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Eclasses (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Gentoo Games
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-12-11 19:04 UTC by Michał Górny
Modified: 2014-01-25 17:44 UTC (History)
0 users

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


Attachments
games.eclass.patch (games.eclass.patch,349 bytes, patch)
2013-12-13 16:02 UTC, Julian Ospald
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Michał Górny archtester Gentoo Infrastructure gentoo-dev Security 2013-12-11 19:04:14 UTC
Modern autoconf has two data paths: datarootdir and datadir. The former is used for 'system global' data such as man pages, info files, .desktop files etc. The latter can be used for application-local data.

Since games.eclass overrides --prefix, default datarootdir ends up being /usr/games/share. I think the eclass should explicitly pass --datarootdir=/usr/share in order to fix .desktop file, pixmap installs etc.
Comment 1 Julian Ospald 2013-12-11 19:13:56 UTC
a problem is that we can probably not fix this without breaking a lot of stuff, can we?

I was suggesting to drop base.eclass inherit for EAPI=6 anyway... maybe we can fix this along with that?
Comment 2 Michał Górny archtester Gentoo Infrastructure gentoo-dev Security 2013-12-11 20:00:09 UTC
(In reply to Julian Ospald (hasufell) from comment #1)
> a problem is that we can probably not fix this without breaking a lot of
> stuff, can we?

What stuff is going to be broken? I doubt many games actually use datarootdir at all, not to mention using it incorrectly.

Old autoconf may be a problem but we can account for this with the usual '--help' check.
Comment 3 Julian Ospald 2013-12-11 20:05:23 UTC
(In reply to Michał Górny from comment #2)
> (In reply to Julian Ospald (hasufell) from comment #1)
> > a problem is that we can probably not fix this without breaking a lot of
> > stuff, can we?
> 
> What stuff is going to be broken? I doubt many games actually use
> datarootdir at all, not to mention using it incorrectly.
> 
> Old autoconf may be a problem but we can account for this with the usual
> '--help' check.

Then do a tinderbox run.

I will add this option locally to the eclass and do random testing.
Comment 4 Julian Ospald 2013-12-13 16:02:35 UTC
Created attachment 365230 [details, diff]
games.eclass.patch
Comment 5 Julian Ospald 2013-12-13 20:41:02 UTC
it seems the only one that actually "broke" was games-roguelike/angband... but the data files were misplaced in /usr/games/share anyway, so I fixed it

I did not test gnome games ebuilds, but afais they do their own econf thing anyway
Comment 6 Julian Ospald 2013-12-30 23:56:56 UTC
Can someone from games team comment? I feel this is safe to apply.
Comment 7 Julian Ospald 2014-01-12 21:44:04 UTC
I will apply this in two weeks unless someone has objections.
Comment 8 Julian Ospald 2014-01-25 17:44:46 UTC
+  25 Jan 2014; Julian Ospald <hasufell@gentoo.org> games.eclass:
+  set --datarootdir=/usr/share wrt #493954
+