After some more testing i've discovered another way to trigger a segfault by pressing SPACE while watching a replay. All in all this function is way to buggy and i think it's better to disable it for now.
I've updated the patch and also the README.Debian file so that the users know why this function has been disabled. My request for sponsorship is bug #693249
diff -u etw-3.6+svn140/debian/changelog etw-3.6+svn140/debian/changelog --- etw-3.6+svn140/debian/changelog +++ etw-3.6+svn140/debian/changelog @@ -1,3 +1,16 @@ +etw (3.6+svn140-4) unstable; urgency=low + + * New Maintainer. (Closes: #544922) + * Eat the Whistle will be maintained in a Git repository from now on. + Change the Vcs-fields in debian/control accordingly. + * debian/patches: + Add 0005-change-configuration-path.patch. + Save and load configuration in $HOME/.etw/ instead of + /usr/share/games/etw and stop failing silently. (Closes: #693244) + * Update README.Debian. + + -- Markus Koschany <a...@gambaru.de> Sun, 16 Nov 2012 19:41:55 +0100 + etw (3.6+svn140-3) unstable; urgency=low * Team upload. diff -u etw-3.6+svn140/debian/control etw-3.6+svn140/debian/control --- etw-3.6+svn140/debian/control +++ etw-3.6+svn140/debian/control @@ -2,12 +2,12 @@ Section: games Priority: optional Maintainer: Debian Games Team <pkg-games-de...@lists.alioth.debian.org> -Uploaders: Sam Hocevar <s...@debian.org>, Barry deFreese <bdefre...@debian.org> +Uploaders: Markus Koschany <a...@gambaru.de> Build-Depends: debhelper (>= 5.0), quilt, libgtk2.0-dev, libsdl1.2-dev, sharutils Standards-Version: 3.9.1 Homepage: http://www.ggsoft.org/etw/ -Vcs-Svn: svn://svn.debian.org/svn/pkg-games/packages/trunk/etw/ -Vcs-Browser: http://svn.debian.org/wsvn/pkg-games/packages/trunk/etw/?op=log +Vcs-Git: git://git.debian.org/pkg-games/etw.git +Vcs-Browser: http://git.debian.org/?p=pkg-games/etw.git;a=summary Package: etw Architecture: any diff -u etw-3.6+svn140/debian/README.Debian etw-3.6+svn140/debian/README.Debian --- etw-3.6+svn140/debian/README.Debian +++ etw-3.6+svn140/debian/README.Debian @@ -1,8 +1,26 @@ etw for Debian -------------- - The packaging should be rather clean, but there are still many bugs in -the game. Do not hesitate to report them directly to me. +Since version 3.6+svn140-4 you can save your configuration permanently in +$HOME/.etw/. Although Eat The Whistle has been developed since 1997 it is, like +other software, not free of bugs. Most notably the game crashes if you try +to view a replay which was recorded while playing in arcade mode or if you hit +SPACE during a replay again. + +As long as this behavior isn't fixed the replay function will be disabled. + +There are other well known bugs. Please refer for more information to +"bugs.txt" which is also located in /usr/share/doc/etw. Thus said the game +itself is stable and is made in the spirit of good old retro football +gaming. + +Nevertheless if you discover new bugs or if you think that they are related +specifically to Debian, don't hesitate to report them with the reportbug tool. + +The official README, EatTheWhistle.txt, contains more information about +controlling the players and other game functions. Don't be confused by some +remarks and always keep in mind that Eat The Whistle was originally developed +for the AMIGA platform! Useful excerpt from the old manual: @@ -33 +50,0 @@ - -- Sam Hocevar <s...@debian.org>, Wed, 15 Oct 2003 19:28:47 +0200 diff -u etw-3.6+svn140/debian/patches/series etw-3.6+svn140/debian/patches/series --- etw-3.6+svn140/debian/patches/series +++ etw-3.6+svn140/debian/patches/series @@ -1,3 +1,4 @@ +0005-change-configuration-path.patch 000_upstream_changelog.diff 100_pthread_linking.diff 200_datadir.diff only in patch2: unchanged: --- etw-3.6+svn140.orig/debian/patches/0005-change-configuration-path.patch +++ etw-3.6+svn140/debian/patches/0005-change-configuration-path.patch @@ -0,0 +1,32 @@ +From: Markus Koschany <a...@gambaru.de> +Date: Sat, 10 Nov 2012 14:22:44 +0100 +Subject: Change conf and replay path + +The default game directory of etw is /usr/share/games/etw. The game stores all +the configuration in there hence users couldn't use important functions of the +game in the past. This patch changes the behaviour. From now on the +configuration is saved permanently in $HOME/.etw/ +--- + etw/menu_config.c | 8 +++++++- + 1 files changed, 8 insertions(+), 1 deletions(-) + +--- a/etw/menu_config.c ++++ b/etw/menu_config.c +@@ -392,9 +392,15 @@ void load_config(FILE *f) + void read_menu_config(void) + { + FILE *f; ++ char path[1024]; ++ sprintf(path, "%s/%s", getenv("HOME"), ".etw/etw.cfg" ); + D(bug("Reading configuration...\n"/*-*/)); + +- f=fopen("etw.cfg"/*-*/,"r"); ++ f=fopen(path/*-*/,"r"); ++ ++ if (f == NULL) { ++ f=fopen("etw.cfg"/*-*/,"r"); ++ } + + newpitches=CheckNewPitches(); + +
signature.asc
Description: Digital signature
