Bug#303596: Emacs installation fails on vfat fs

[Stephen Gildea]

Package: emacs21-common
Version: 21.4a-1
Tags: patch

The postinst script does several chown and chmod operations on
subdirectories of /usr/local/share/.  If these fail, the script fails
(it runs with sh -e) and the package installation fails.  This is too
severe, as the chown/chmod failure can be innocuous.

I have my /usr/local/share mounted on a VFAT partition.  The VFAT file
system doesn't have the concept of file owner or modes.  The Linux kernel
reports all chown and chmod operations here as failing.

But failing to do a chown or chmod of these directories is not important
enough to cause package installation to fail on any file system.

I propose this patch to the postinst script to ignore errors on all
chown and chmod commands:

--- emacs21-21.4a/debian/emacs-common.postinst  2005-04-04 08:40:36.876721131 
-0400
+++ emacs21-common.postinst     2005-04-04 08:29:02.000000000 -0400
@@ -45,8 +45,8 @@
 then
   if mkdir ${parentdir}/${newdir} 2>/dev/null
   then
-    chown root:staff ${parentdir}/${newdir}
-    chmod 2775 ${parentdir}/${newdir}
+    chown root:staff "${parentdir}/${newdir}" || true
+    chmod 2775 "${parentdir}/${newdir}" || true
   fi
 fi
 
@@ -57,8 +57,8 @@
 then
   if mkdir ${parentdir}/${newdir} 2>/dev/null
   then
-    chown root:staff ${parentdir}/${newdir}
-    chmod 2775 ${parentdir}/${newdir}
+    chown root:staff "${parentdir}/${newdir}" || true
+    chmod 2775 "${parentdir}/${newdir}" || true
   fi
 fi
 
@@ -69,8 +69,8 @@
 then
   if mkdir ${parentdir}/${newdir} 2>/dev/null
   then
-    chown root:staff ${parentdir}/${newdir}
-    chmod 2775 ${parentdir}/${newdir}
+    chown root:staff "${parentdir}/${newdir}" || true
+    chmod 2775 "${parentdir}/${newdir}" || true
   fi
 fi
 
@@ -84,8 +84,8 @@
 then
   if mkdir ${parentdir}/${newdir} 2>/dev/null
   then
-    chown root:staff ${parentdir}/${newdir}
-    chmod 2775 ${parentdir}/${newdir}
+    chown root:staff "${parentdir}/${newdir}" || true
+    chmod 2775 "${parentdir}/${newdir}" || true
   fi
 fi
 
@@ -96,8 +96,8 @@
 then
   if mkdir ${parentdir}/${newdir} 2>/dev/null
   then
-    chown root:staff ${parentdir}/${newdir}
-    chmod 2775 ${parentdir}/${newdir}
+    chown root:staff "${parentdir}/${newdir}" || true
+    chmod 2775 "${parentdir}/${newdir}" || true
   fi
 fi
 



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