tags 547690 + patch thanks There is a test on ! -L in preinst, but if the file or directory doesn't exist at all, this also amounts to true. Hence there should be a test for existance. The attached patch deals with it.
Robert
diff -u splashy-0.3.13/debian/changelog splashy-0.3.13/debian/changelog --- splashy-0.3.13/debian/changelog +++ splashy-0.3.13/debian/changelog @@ -1,3 +1,9 @@ +splashy (0.3.13-5.1) unstable; urgency=low + + * Test for /etc/splashy/themes in preinst before mv (Closes: 547690) + + -- Robert Grimm <[email protected]> Sat, 28 Nov 2009 13:38:33 +0100 + splashy (0.3.13-5) unstable; urgency=low * Update 02_lsb-base-logging.sh_bug512951.diff patch: diff -u splashy-0.3.13/debian/splashy.preinst splashy-0.3.13/debian/splashy.preinst --- splashy-0.3.13/debian/splashy.preinst +++ splashy-0.3.13/debian/splashy.preinst @@ -17,7 +17,7 @@ # We don't want to mess with whatever themes were here already. # Just move them out of harms way: if [ -d /etc/splashy ]; then - if [ ! -L /etc/splashy/themes ]; then + if [ -e /etc/splashy/themes ] && [ ! -L /etc/splashy/themes ]; then if [ -e /etc/splashy/themes.old ]; then echo echo "/etc/splashy/themes.old already exist, please remove this directory or file"
