Bug 304300 (agged woody) says:

----------------------------------------------------------------------
The postinst script for samba contains this errorneous code:

if [ -d /var/samba/ ]; then
        mv /var/samba/* /var/lib/samba/ 2>/dev/null || true
        rmdir /var/samba/
fi

Without actually checking that the directory contains the stuff that
should be moved to /var/lib/samba. That directory is commonly used for
the shared data (akin to /var/www). This code breaks such system.

The code should really do:

mkdir -p /var/lib/samba
for F in ntdrivers.tdb ntforms.tdb ntprinters.tdb secrets.tdb share_info.tdb 
wins.dat; do
        if [ -f /var/samba/$F ]; then
                mv /var/samba/$F /var/lib/samba
        fi
fi
grep /var/samba /etc/samba/smb.conf || rmdir /var/samba || true

The problem was already there in the previous update and was not fixed.
----------------------------------------------------------------------


The code is actually not really useful in post-sarge unless we want to
support upgrades from woody (I don't even remember whether /var/samba
was still used in woody...IIRC /var/lib/samba is already there).

So, I actually propose just dropping this piece of code and hereby
close this bug report.

Advices?




-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Reply via email to