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

Bug#35460: cucipop: preinst remounts /usr without need



Package: cucipop
Version: 1.31-7

The preinst says

    if [ "$(mount | grep 'on /usr' | grep ro)" != "" ]
    then
	echo "Remounting the /usr filesystem read-write (forgetful, aren't we?)"
	mount -o remount,rw /usr
    fi

I don't understand why it does this (surely unpacking the .deb would
fail if /usr were read-only, and it is not each individual package's
responsibility to handle this situation), but at the least it should
get it right.  This code was tripped here because mount says

    /dev/hdc1 on /usr type ext2 (rw,errors=remount-ro)

and the code mistook the remount-ro.  If the code must stay, please at
least fix it up, perhaps like

    mount | awk '$3 == "/usr" && $6 ~ /^\(ro,/' 

or

    mount | perl -wane '$ok = 1 if $F[2] eq "/usr" && $F[5] =~ /^\(rw[,)]/;
			END { $? = 1 unless $ok }'

-- System Information
Debian Release: potato
Kernel Version: Linux jones 2.2.3-jones.2 #1 Sat Mar 13 12:25:33 EST 1999 i686 unknown

Versions of the packages cucipop depends on:
ii  netbase         3.14-1         Basic TCP/IP networking binaries
ii  netstd          3.07-7slink.3  Networking binaries and daemons for Linux
ii  libc6           2.1.1-0.1      GNU C Library: shared libraries
ii  libdb2          2.4.14-5       The Berkeley database routines (run-time fil


Reply to: