Bugzilla Bug 103586

In SlashEm v. 0.7E6 and forward, trying to save your game results in SlashEm
responding with an error message. SlashEm can't seem to find a certain file and
compression of the save file fails.
What happens is that SlashEm tries to find gzip to do the compression but looks
for it in the wrong place. This is due to a problem with the installation
procedure in the ebuilds. When updating "#define"-sections in the headers a
configuration file for RedHat9 systems is used which sets the path to gzip to
/usr/bin/gzip, but in gentoo systems the path should be /bin/gzip.
The solution is to put a corrected version of the configuration file in the
files/0.0.7E6- or files/0.0.7E7-directory inside slashem's portage directory,
thus including it in the configuration process.

Reproducible: Always
Steps to Reproduce:
1. Emerge >=games-roguelike/slashem-0.0.760
2. Run slashem and start a new game
3. Save

Actual Results:  
Saving fails

Expected Results:  
Saving works

Created an attachment (id=66753) [details]
A corrected version of the RedHat9 configuration file

Put this file in games-roguelike/slashem/files/0.0.7E6 and
games-roguelike/slashem/files/0.0.7E7 directories in the portage tree.
All it does is changing the path to gzip from /usr/bin/gzip to /bin/gzip

The proposed patch doesn't seem to have any effect.

Anyway, I thought that saving problem was fixed as bug 98318, but it looks like
I didn't test it good enough. There was still a problem which was result from
Gentoo definition of VAR_PLAYGROUND, which is obsolete define and shouldn't be
used when File Hierarchy Standard (FILE_AREAS) is used. Previously I thought
that this define just had no effect with FILE_AREAS, but now it seems it
actually confuses the game when dealing with savegames. There is another unused
define, LOCKDIR, which should be removed as well. Patch will follow soon which
removes obsolete defines. After applying the patch gentoo-paths.patch should be
removed from files/0.0.7E6. 

Created an attachment (id=67085) [details]
Remove obsolete VAR_PLAYGROUND and LOCKDIR defines

I'm going to guess that this is still valid with the 0.0.772 ebuild that I just
put into portage, correct?

Created an attachment (id=98890) [details]
Latest version of my patch

Saving problem is still there. This patch fixes the saving problem for latest
version and fixes the qt-support. All "Only in
/usr/portage/games-roguelike/slashem/files:" files mentioned in the patch
should be removed as obsolete. If you like a separate bug report for the
qt-fix, it is easy to remove qt-stuff from this patch.

Cool.  Fixed in CVS.