[pkg-kolab] Bug#817808: php-kolab: postrm is buggy, breaks upgrades

[Andreas Beckmann]

Package: php-kolab
Version: 0.6.0-2
Severity: serious
User: debian-qa at lists.debian.org
Usertags: piuparts

Hi,

during a test with piuparts I noticed your package fails to upgrade from
'sid'.
It installed fine in 'sid', then the upgrade to 'experimental' fails.

>From the attached log (scroll to the bottom...):

  Preparing to unpack .../php-kolab_1.0.2-1_amd64.deb ...
  Unpacking php-kolab (1.0.2-1) over (0.6.0-2) ...
  /var/lib/dpkg/info/php-kolab.postrm: 11: local: 1.0.2-1: bad variable name
  dpkg: warning: subprocess old post-removal script returned error exit status 2
  dpkg: trying script from the new package instead ...
  /var/lib/dpkg/tmp.ci/postrm: 11: local: 0.6.0-2: bad variable name
  dpkg: error processing archive /var/cache/apt/archives/php-kolab_1.0.2-1_amd64.deb (--unpack):
   subprocess new post-removal script returned error exit status 2
  /var/lib/dpkg/tmp.ci/postrm: 11: local: 0.6.0-2: bad variable name
  dpkg: error while cleaning up:
   subprocess new post-removal script returned error exit status 2
  Processing triggers for libc-bin (2.21-9) ...
  Processing triggers for systemd (229-2) ...
  Processing triggers for fontconfig (2.11.0-6.3) ...
  Processing triggers for udev (229-2) ...
  Errors were encountered while processing:
   /var/cache/apt/archives/php-kolab_1.0.2-1_amd64.deb

There seemes to be some quoting missing quoting ...

remove_module () {
        local step=$1 module=$2 inifile=$2.ini package=$3 args=$4
...
}

remove_module $1 kolab php-kolab "$*"
remove_module $1 kolabdummy php-kolab "$*"


Looks like you wanted 

remove_module $1 kolab php-kolab "$@"
remove_module $1 kolabdummy php-kolab "$@"


cheers,

Andreas
-------------- next part --------------
A non-text attachment was scrubbed...
Name: php-kolab_1.0.2-1.log.gz
Type: application/gzip
Size: 38702 bytes
Desc: not available
URL: <http://lists.alioth.debian.org/pipermail/pkg-kolab-devel/attachments/20160310/fceae9a7/attachment-0001.bin>