Bug#527899: removal of package fails

2009-06-03 Thread Raphael Hertzog
On Tue, 02 Jun 2009, Chris Taylor wrote:
 This patch fixes the removal bug in sqlrelay. It simply removes all
 sqlrelay files and the directory from /etc.
 
 -Chris
 
 --- a/debian/sqlrelay.postrm
 +++ b/debian/sqlrelay.postrm
 @@ -19,7 +19,7 @@ set -e
  case $1 in
 remove|purge)
 rm -Rf /var/cache/sqlrelay
 -   rm -f /etc/sqlrelay*
 +   rm -rf /etc/sqlrelay*
 ;;

Removing configuration files on remove (instead of purge only) is a mistake
too...

Cheers,
-- 
Raphaël Hertzog

Contribuez à Debian et gagnez un cahier de l'admin Debian Lenny :
http://www.ouaza.com/wp/2009/03/02/contribuer-a-debian-gagner-un-livre/



--
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#527899: removal of package fails

2009-06-03 Thread Chris Taylor
On Wed, Jun 03, 2009 at 08:59:09AM +0200, Raphael Hertzog wrote:
 On Tue, 02 Jun 2009, Chris Taylor wrote:
  This patch fixes the removal bug in sqlrelay. It simply removes all
  sqlrelay files and the directory from /etc.
  
  -Chris
  
  --- a/debian/sqlrelay.postrm
  +++ b/debian/sqlrelay.postrm
  @@ -19,7 +19,7 @@ set -e
   case $1 in
  remove|purge)
  rm -Rf /var/cache/sqlrelay
  -   rm -f /etc/sqlrelay*
  +   rm -rf /etc/sqlrelay*
  ;;
 
 Removing configuration files on remove (instead of purge only) is a mistake
 too...
 
 Cheers,
 -- 
 Raphaël Hertzog
 

Oops, that was actually a typo, I meant to make it on purge only in the patch.

-Chris





--
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#527899: removal of package fails

2009-06-03 Thread Bernhard R. Link
* Raphael Hertzog hert...@debian.org [090603 09:10]:
 On Tue, 02 Jun 2009, Chris Taylor wrote:
  This patch fixes the removal bug in sqlrelay. It simply removes all
  sqlrelay files and the directory from /etc.
  
  -Chris
  
  --- a/debian/sqlrelay.postrm
  +++ b/debian/sqlrelay.postrm
  @@ -19,7 +19,7 @@ set -e
   case $1 in
  remove|purge)
  rm -Rf /var/cache/sqlrelay
  -   rm -f /etc/sqlrelay*
  +   rm -rf /etc/sqlrelay*
  ;;
 
 Removing configuration files on remove (instead of purge only) is a mistake
 too...

And the amount of what is removed looks extremly suspicious, too. This
really needs to be reduced to actual files, otherwise it will also
remove things like a user-generated /etc/sqlrelay.backup or things like
that

Hochachtungsvoll,
Bernhard R. Link



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#527899: removal of package fails

2009-06-02 Thread Chris Taylor
tag 527899 patch
thanks


This patch fixes the removal bug in sqlrelay. It simply removes all sqlrelay 
files and the directory from /etc.

-Chris

--- a/debian/sqlrelay.postrm
+++ b/debian/sqlrelay.postrm
@@ -19,7 +19,7 @@ set -e
 case $1 in
remove|purge)
rm -Rf /var/cache/sqlrelay
-   rm -f /etc/sqlrelay*
+   rm -rf /etc/sqlrelay*
;;
upgrade|failed-upgrade|abort-install|abort-upgrade|disappear)
;;



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of unsubscribe. Trouble? Contact listmas...@lists.debian.org



Bug#527899: removal of package fails

2009-05-09 Thread Holger Levsen
Package: sqlrelay
Version: 1:0.39.4-7
Severity: serious
User: debian...@lists.debian.org
Usertags: piuparts piuparts.d.o

Hi, 

during a test with piuparts I noticed your package fails to remove cleanly.

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

0m37.3s DEBUG: Starting command: 
['chroot', '/org/piuparts.debian.org/tmp/tmpbblZNJ', 'dpkg', '--remove', 
'--pending']
0m37.3s DUMP: 
  (Reading database ... 5835 files and directories currently installed.)
  Removing sqlrelay ...
  rm: cannot remove `/etc/sqlrelay': Is a directory
  dpkg: error processing sqlrelay (--remove):
   subprocess post-removal script returned error exit status 1
  Errors were encountered while processing:
   sqlrelay
0m37.3s DEBUG: Command failed (status=1), but ignoring error: 
['chroot', '/org/piuparts.debian.org/tmp/tmpbblZNJ', 'dpkg', '--remove', 
'--pending']
0m37.3s DEBUG: Starting command: 
['chroot', '/org/piuparts.debian.org/tmp/tmpbblZNJ', 'dpkg', '--purge', 
'sqlrelay']
0m37.4s DUMP: 
  (Reading database ... 5835 files and directories currently installed.)
  Removing sqlrelay ...
  rm: cannot remove `/etc/sqlrelay': Is a directory
  dpkg: error processing sqlrelay (--purge):
   subprocess post-removal script returned error exit status 1
  Errors were encountered while processing:
   sqlrelay


regards,
Holger
Start: 2009-04-18 04:12:36 UTC

Package: sqlrelay
Priority: optional
Section: misc
Installed-Size: 636
Maintainer: Debian QA Group packa...@qa.debian.org
Architecture: amd64
Version: 1:0.39.4-7
Depends: libxml2-utils, libc6 (= 2.7-1), libgcc1 (= 1:4.1.1), libncurses5 (= 5.6+20071006-3), libpcre3 (= 7.7), libreadline5 (= 5.2), librudiments0.31 (= 0.31), libsqlrelay-0.39 (= 1:0.39.4-7), libssl0.9.8 (= 0.9.8f-5), libstdc++6 (= 4.1.1), adduser
Suggests: sqlrelay-doc (= 1:0.39.4-7), sqlrelay-connection-daemon, sqlrelay-api
Filename: pool/main/s/sqlrelay/sqlrelay_0.39.4-7_amd64.deb
Size: 165442
MD5sum: fc10a30addfb71c96a64f6cc5a3ee13b
SHA1: 43d339196103dd88110949a5ffe5d43ca438920c
SHA256: 22cd558e8d9dd2c32cc418db5a46e98e18997c5bf3adc3e61a167795f8f329d7
Description: Database connection pooling, proxying and load balancing
 SQL Relay is a persistent database connection pooling, proxying and
 load balancing system for Unix and Linux supporting ODBC, Oracle,
 MySQL, mSQL, PostgreSQL, Sybase, MS SQL Server, IBM DB2, Interbase,
 Lago, MDB and SQLite with C, C++, Perl, Perl-DBD, Python, Python-DB, Zope,
 PHP, Ruby, Tcl and Java APIs, command line clients, a GUI configuration tool
 and extensive documentation.
 .
 This package contains the binaries. Adapters for non-free databases
 are not built by default.
Homepage: http://sqlrelay.sourceforge.net/
Tag: devel::lang:sql, implemented-in::c++, interface::daemon, network::load-balancing, network::server, role::program, uitoolkit::ncurses, use::proxying, works-with::db

Executing: sudo /org/piuparts.debian.org/sbin/piuparts --no-symlinks --scriptsdir /etc/piuparts/scripts/ --tmpdir /org/piuparts.debian.org/tmp -ad squeeze -b squeeze.tar.gz sqlrelay
Guessed: debian
0m0.0s INFO: --
0m0.0s INFO: To quickly glance what went wrong, scroll down to the bottom of this logfile.
0m0.0s INFO: FAQ available at http://wiki.debian.org/piuparts/FAQ
0m0.0s INFO: --
0m0.0s INFO: piuparts version 0.36~20090417 starting up.
0m0.0s INFO: Command line arguments: /org/piuparts.debian.org/sbin/piuparts --no-symlinks --scriptsdir /etc/piuparts/scripts/ --tmpdir /org/piuparts.debian.org/tmp -ad squeeze -b squeeze.tar.gz sqlrelay
0m0.0s INFO: Running on: Linux piatti 2.6.29.1-dsa-amd64 #2 SMP Sun Apr 5 00:16:11 CEST 2009 x86_64
0m0.0s DEBUG: Created temporary directory /org/piuparts.debian.org/tmp/tmpbblZNJ
0m0.0s DEBUG: Unpacking squeeze.tar.gz into /org/piuparts.debian.org/tmp/tmpbblZNJ
0m0.0s DEBUG: Starting command: ['tar', '-C', '/org/piuparts.debian.org/tmp/tmpbblZNJ', '-zxf', 'squeeze.tar.gz']
0m16.4s DEBUG: Command ok: ['tar', '-C', '/org/piuparts.debian.org/tmp/tmpbblZNJ', '-zxf', 'squeeze.tar.gz']
0m16.4s DEBUG: Created policy-rc.d and chmodded it.
0m16.4s DEBUG: Starting command: ['chroot', '/org/piuparts.debian.org/tmp/tmpbblZNJ', 'apt-get', 'update']
0m19.9s DUMP: 
  Get:1 http://ftp.fi.debian.org squeeze Release.gpg [197B]
  Get:2 http://ftp.fi.debian.org squeeze Release [78.3kB]
  Get:3 http://ftp.fi.debian.org squeeze/main Packages/DiffIndex [2038B]
  Get:4 http://ftp.fi.debian.org squeeze/main Packages [7353kB]
  Fetched 7434kB in 1s (5642kB/s)
  Reading package lists...
0m19.9s DEBUG: Command ok: ['chroot', '/org/piuparts.debian.org/tmp/tmpbblZNJ', 'apt-get', 'update']
0m19.9s DEBUG: Starting command: ['chroot', '/org/piuparts.debian.org/tmp/tmpbblZNJ', 'mount', '-t', 'proc', 'proc', '/proc']
0m19.9s DEBUG: Command ok: ['chroot', '/org/piuparts.debian.org/tmp/tmpbblZNJ', 'mount', '-t', 'proc', 'proc', '/proc']
0m19.9s DEBUG: