[prev in list] [next in list] [prev in thread] [next in thread] 

List:       mandrake-cooker
Subject:    [Cooker] [PROPOSAL] use fstab-sync instead of drakupdate_fstab for removable devices
From:       FACORAT Fabrice <facorat.fabrice () laposte ! net>
Date:       2005-01-31 15:46:50
Message-ID: 200501311646.50395.facorat.fabrice () laposte ! net
[Download RAW message or body]

Actually when installing hal, you end up with drakupdate_fstab and fstab-sync 
trying both to managed removable HD ( USB HD/pendrive ).
I'm proposing to use fstab-sync for 2.6 kernels by default as fstab-sync seems 
to have more potential.

+ some fstab-sync features : 

1. new options could be easily add to fstab-sync when you want to add entries 
in fstab

2. fstab-sync by default use the media Name/Manufacturer name to make the 
mount point

3. fstab-sync is integrated with hal. If Gnome/KDE was integrated with D-BUS 
and hal was notifying the system via D-BUS, it could allow to have something 
very interesting ( XP like management with a notification message saying that 
the USB key has been discovered and mounted )

+ Actual issues with fstab-sync :

- no msec support ( i.e when security level > 3, users should not be able to 
mount USB devices )

- use /media instead of /mnt as root directory mountpoint

- add unsupported keyword for our mount package ( managed, pamconsole )


+ Some solutions :

- for msec support we could think about msec adding rules 
in /usr/share/hal/fdi/30osvendor/ or /usr/share/hal/fdi/95userpolicy/
when security level <= 3, then the rules specify users and umask=0 as mount 
options

- for mountpoint and unsopported keywords, this could be easily solved by 
modifying default policy ( or patching mount ) 
( /usr/share/hal/fdi/90defaultpolicy/storage-policy.fdi )
the following patch will modify fstab-sync so that :
	- it use /mnt as default root directory mount point instead of /media
	- it use kudzu and not managed as keyword to detect its entries
	- it doesn't use pamconsole option




-- 
Les optimistes pensent que nous vivons dans le meilleur des
mondes possibles, les pessimistes en sont intimement persuadés.
Pierre Desproges.

["hal-storage-policy_v2.patch" (text/x-diff)]

--- storage-policy.fdi	2005-01-27 17:50:25.000000000 +0100
+++ storage-policy.fdi.new	2005-01-31 16:45:36.607190629 +0100
@@ -5,12 +5,11 @@
   <!-- Default policies merged onto computer root object  -->
   <device>
     <match key="info.udi" string="/org/freedesktop/Hal/devices/computer">
-      <merge key="storage.policy.default.mount_root" type="string">/media</merge>
+    <merge key="storage.policy.default.mount_root" type="string">/mnt</merge>
       <merge key="storage.policy.default.use_managed_keyword" type="bool">true</merge>
-      <merge key="storage.policy.default.managed_keyword.primary" type="string">managed</merge>
-      <merge key="storage.policy.default.managed_keyword.secondary" type="string">kudzu</merge>
+      <merge key="storage.policy.default.managed_keyword.primary" type="string">kudzu</merge>
       <merge key="storage.policy.default.mount_option.noauto" type="bool">true</merge>
-      <merge key="storage.policy.default.mount_option.pamconsole" type="bool">true</merge>
+      <merge key="storage.policy.default.mount_option.pamconsole" type="bool">false</merge>
       <merge key="storage.policy.default.mount_option.exec" type="bool">true</merge>
     </match>
   </device>
@@ -19,7 +18,7 @@
     <!-- Whitelist bus types of storage devices we care about  -->
     <match key="info.category" string="storage">
       <match key="storage.bus" string="usb">
-	<merge key="storage.policy.should_mount" type="bool">true</merge>      
+       <merge key="storage.policy.should_mount" type="bool">true</merge>      
       </match>
       <match key="storage.bus" string="ide">
 	<merge key="storage.policy.should_mount" type="bool">true</merge>
@@ -163,11 +162,6 @@
 	      <merge key="volume.policy.mount_option.noatime" type="bool">true</merge>
 	    </match>
 	  </match>
-
-	  <!-- Use UTF-8 charset for vfat -->
-	  <match key="volume.fstype" string="vfat">
-	    <merge key="volume.policy.mount_option.iocharset=utf8" type="bool">true</merge>
-	  </match>
 	  
 	  <!-- whitelist of partition table id's, if from a msdos partition table -->
 	  <match key="volume.partition.msdos_part_table_type" exists="true">


[prev in list] [next in list] [prev in thread] [next in thread]