Bug#689127: filler: do not delete /var/games/filler.ratings.gz while initializing /var/games/filler.ratings

[Andreas Beckmann]

Followup-For: Bug #689127
Control: tag -1 patch pending

Hi,

I just uploaded a NMU to DELAYED/1 to get this fixed in jessie, and I'll
take care of getting this unblocked.


Andreas
diff -u filler-1.02/debian/postinst filler-1.02/debian/postinst
--- filler-1.02/debian/postinst
+++ filler-1.02/debian/postinst
@@ -25,7 +25,7 @@
 case $1 in
 configure)
 if [ ! -f /var/games/filler.ratings ]; then
-	gzip -d /var/games/filler.ratings.gz
+	gzip -dc /var/games/filler.ratings.gz  /var/games/filler.ratings
 	chmod 0664 /var/games/filler.ratings
 	chgrp games /var/games/filler.ratings
 	fi
diff -u filler-1.02/debian/changelog filler-1.02/debian/changelog
--- filler-1.02/debian/changelog
+++ filler-1.02/debian/changelog
@@ -1,3 +1,11 @@
+filler (1.02-6.2) unstable; urgency=medium
+
+  * Non-maintainer upload.
+  * Do not delete the shipped /var/games/filler.ratings.gz while initializing
+/var/games/filler.ratings.  (Closes: #689127)
+
+ -- Andreas Beckmann a...@debian.org  Mon, 01 Dec 2014 21:57:47 +0100
+
 filler (1.02-6.1) unstable; urgency=low
 
   * Non-maintainer upload.
@@ -21,7 +29,7 @@
 filler (1.02-6) unstable; urgency=low
 
   * The main/ in the Section line of debian/control should be assumed.
-	
+
  -- James Damour (Suvarov454) suvarov...@users.sourceforge.net  Sat, 29 Apr 2006 14:07:00 -0400
 
 filler (1.02-5) unstable; urgency=low
@@ -32,7 +40,7 @@
   * Use correct temp directory in rules (debian/filler, not debian/tmp).
   * Updated DH_COMPAT of rules to 4 (from 1).
   * Eliminated unused dh_* lines from rules.
-	
+
  -- James Damour (Suvarov454) suvarov...@users.sourceforge.net  Wed, 11 Jan 2006 08:18:00 -0400
 
 filler (1.02-4) unstable; urgency=low

Bug#689127: filler: do not delete /var/games/filler.ratings.gz while initializing /var/games/filler.ratings

[Andreas Beckmann]

Package: filler
Version: 1.02-6.1
Severity: important
User: debian...@lists.debian.org
Usertags: piuparts

Hi,

during a test with piuparts I noticed your package deletes a shipped
file.
If the extraction of /var/games/filler.ratings.gz is changed to

gzip -dc /var/games/filler.ratings.gz  /var/games/filler.ratings

the shipped file will be preserved.


Not knowing filler at all, I assume filler.ratings is some kind of a
highscore file. Shouldn't it be kept around until postrm purge - like it
is done with configuration files and logfiles? (It's currently removed
during prerm remove). 


From the attached logfile, scroll to the end:

0m33.1s ERROR: FAIL: debsums reports modifications inside the chroot:
  debsums: missing file /var/games/filler.ratings.gz (from filler package)


Cheers,

Andreas


filler_1.02-6.1.log.gz
Description: GNU Zip compressed data