Hi,
On Wed, Mar 16, 2005 at 07:47:24PM +0100, Javier Fern�ndez-Sanguino Pe�a wrote:
> Yes, please, I cannot reproduce this. Can you please do this?
>
> # export DEBCONF_DEBUG=developer
> # apt-get install --reinstall checksecurity
Reinstalling works without errors. I did some debugging and came up with a
patch. The problem is this snippet in postinst:
if [ ! -d /var/log/setuid ]; then
mkdir -m 750 /var/log/setuid
chown root:adm /var/log/setuid || true
for file in /var/log/setuid.yesterday /var/log/setuid.today
/var/log/setuid.changes \
/var/log/setuid.changes.*; do
[ -e $file ] && mv $file /var/log/setuid
done
fi
The [ -e $file ] has a return code of 1 if the given files don't exist.
This return code seems to become the return code of the whole postinst
script, later. Here's my proposed fix, which works for me (tm):
--- postinst 2005-03-16 23:27:57.000000000 +0100
+++ postinst.new 2005-03-16 23:28:26.000000000 +0100
@@ -19,7 +19,9 @@
chown root:adm /var/log/setuid || true
for file in /var/log/setuid.yesterday /var/log/setuid.today
/var/log/setuid.changes \
/var/log/setuid.changes.*; do
- [ -e $file ] && mv $file /var/log/setuid
+ if [ -e $file ]; then
+ mv $file /var/log/setuid
+ fi
done
fi
HTH, Uwe.
--
Uwe Hermann <[EMAIL PROTECTED]>
http://www.hermann-uwe.de | http://www.crazy-hacks.org
http://www.it-services-uh.de | http://www.phpmeat.org
http://www.unmaintained-free-software.org | http://www.holsham-traders.de
