Quoting Rob Walker ([EMAIL PROTECTED]):
> Package: quake2-data
> Version: 13-0.1
> Severity: important
> 
> Following the recent NMU, the configure script quake2-data.config breaks when
> checking the previous version.  The script checks the previous version at line
> 74 with:
> 
>   if [ "$OLDVERSION" -lt "6" -o "$VALID" = "false" ]; then
> 
> If the NMU is installed, then oldversion is now 13-0.1 which is not valid
> input for the -lt operator and the script fails.


At first glance, it seems that this test could be removed. Version 6
is a prehistoric one and keeping this would mean that the package
handles direct upgrades from pre-woody versions to etch,
directly...which is not to be supported.

Proposed patch attached.

--- config.old  2006-11-19 08:32:29.298271550 +0100
+++ config      2006-11-19 08:33:27.154788578 +0100
@@ -44,12 +44,6 @@
        fi
 }
 
-OLDVERSION=$2
-
-if [ -z "$OLDVERSION" ]; then
-       OLDVERSION=6
-fi
-
 while [ "$STATE" != "0" ]; do
 
        case "$STATE" in
@@ -81,7 +75,7 @@
 
                        if [ "$INSTMETHOD" = "download shareware data" ]; then
                                VALID=`validmd5 $BASE/pak0.pak $sharepakmd5`
-                               if [ "$OLDVERSION" -lt "6" -o "$VALID" = 
"false" ]; then
+                               if [ "$VALID" = "false" ]; then
                                        db_input medium quake2-data/storage || 
true
                                        NEXTSTATE=4
                                else
@@ -91,7 +85,7 @@
                                PREVSTATE=1
                        elif [ "$POINTREL" = "true" -a "$INSTMETHOD" = "install 
from CD-ROM" ]; then
                                VALID=`validmd5 $BASE/pak1.pak $pointpakmd5`
-                               if [ "$OLDVERSION" -lt "6" -o "$VALID" = 
"false" ]; then
+                               if [ "$VALID" = "false" ]; then
                                        db_input medium quake2-data/storage || 
true
                                        NEXTSTATE=4
                                else

Attachment: signature.asc
Description: Digital signature

Reply via email to