Re: Request for test/review - Alienblaster

[Hans de Goede <hdegoede@redhat.com>]

Hi,

On 06/06/2013 03:54 AM, Barry deFreese wrote:
> Hi folks,
>
> I have added a patch to alienblaster to create the game config file in the users home dir if it
> doesn't exist.  I am not a C++ expert by any stretch so I would appreciate some testing/review of my
> work if anyone can swing it.
>
> It is to close bug: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=476635
>
> I put my source package up here:  http://people.debian.org/~bdefreese/alienblaster/

Fedora uses a sh wrapper around the alienblaster binary to fix this issue, this would
also allow you do drop most other patches debian is carrying, although you will
want to keep the highscore patch to avoid people loosing there highscores.

#!/bin/bash
set -e

if [ ! -d ~/.alienblaster ]; then
  mkdir -p ~/.alienblaster
  ln -s /usr/share/alienblaster/images ~/.alienblaster/images
  ln -s /usr/share/alienblaster/sound ~/.alienblaster/sound
  cp -r /usr/share/alienblaster/cfg ~/.alienblaster
fi

cd ~/.alienblaster
exec alienblaster.bin "$@"


This works and is very KISS, alternatively if you're going to do a ton of patching
to fix all the assumptions that it does not have its data filed in the cwd, and that
that is rw, then I think it would be best to setup a new upstream for it and do the
work there, so that it can be shared with other distros.

Regards,

Hans