[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#334345: invalid test expression in preinst



Package: x11-common
Version: 6.8.2.dfsg.1-9
Severity: normal
Tags: patch

Hi

The preinst script uses -eq for string comparsion, however correct is 
to use =. Otherwise you might get following error:

Preparing to replace x11-common 6.8.2.dfsg.1-8 (using .../x11-common_6.8.2.dfsg.1-9_all.deb) ...
[: 937: 6.8.2.dfsg.1-8: bad number

--
    Michal Čihař | http://cihar.com

-- System Information:
Debian Release: testing/unstable
  APT prefers unstable
  APT policy: (500, 'unstable'), (1, 'experimental')
Architecture: i386 (i686)
Shell:  /bin/sh linked to /bin/dash
Kernel: Linux 2.6.12
Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8)

Versions of packages x11-common depends on:
ii  debconf [debconf-2.0]         1.4.58     Debian configuration management sy
ii  debianutils                   2.15       Miscellaneous utilities specific t
ii  lsb-base                      3.0-9      Linux Standard Base 3.0 init scrip

x11-common recommends no packages.

-- debconf information:
  x11-common/experimental_packages:
--- /var/lib/dpkg/info/x11-common.preinst	2005-10-14 09:40:31.000000000 +0200
+++ /tmp/x11-common.preinst	2005-10-17 13:18:20.028585664 +0200
@@ -911,7 +911,7 @@
   # been modified from the stable sarge version. If they have been modified, we 
   # don't do anything, but if they are untouched then we remove them to let the 
   # upgrade continue without errors
-  if [ "$2" -eq "" ]; then
+  if [ "$2" = "" ]; then
     remove_conffile_prepare /etc/X11/Xresources/xfree86-common \
        a0f3911120c49038cb87607e3c545b90
     remove_conffile_prepare /etc/X11/Xsession \

Reply to: